Merge pull request #229 from alexdebril/feature/missing-alias

Feature/missing alias
alexdebril · Oct 3, 2019 · fe3f720 · fe3f720
2 parents 12ec103 + 49c59f7
commit fe3f720
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/src/FeedIo/Standard/Atom.php b/src/FeedIo/Standard/Atom.php
@@ -75,7 +75,7 @@ public function buildFeedRuleSet() : RuleSet
             ->add(new LinkNode())
             ->add(new PublicId('id'))
             ->add(new Language('lang'))
-            ->add($this->getModifiedSinceRule('updated'))
+            ->add($this->getModifiedSinceRule('updated'), ['published'])
         ;
 
         return $ruleSet;

diff --git a/tests/live-feeds/StandardFeedsTest.php b/tests/live-feeds/StandardFeedsTest.php
@@ -115,25 +115,16 @@ protected function getUrls()
             'http://feeds.feedburner.com/dailyjs',
             'http://feeds.feedburner.com/HighScalability',
             'http://feeds.feedburner.com/symfony/blog',
-            'http://feeds.mashable.com/Mashable',
             'http://feeds.slate.com/slate',
             'http://feeds.wired.com/wired/index',
             'http://feeds2.feedburner.com/blogspot/Egta',
             'http://feeds2.feedburner.com/LeJournalduGeek',
             'http://feeds2.feedburner.com/Webappers',
-            'http://liberation.fr.feedsportal.com/c/32268/fe.ed/rss.liberation.fr/rss/44/',
-            'http://liberation.fr.feedsportal.com/c/32268/fe.ed/rss.liberation.fr/rss/54/',
-            'http://liberation.fr.feedsportal.com/c/32268/fe.ed/rss.liberation.fr/rss/latest/',
             'http://linuxfr.org/journaux.atom',
             'http://php.net/feed.atom',
-            'http://pipes.yahoo.com/pipes/pipe.run?_id=647030be6aceb6d005c3775a1c19401c&_render=rss',
             'http://rss.lemonde.fr/c/205/f/3050/index.rss',
             'http://rss.slashdot.org/Slashdot/slashdot',
-            'http://rue89.feedsportal.com/c/33822/f/608948/index.rss',
-            'http://what-if.xkcd.com/feed.atom',
             'http://www.debian.org/News/news',
-            'http://www.gizmodo.fr/feed',
-            'http://www.larvf.com/rss/article/10135',
             'http://www.lemonde.fr/sciences/rss_full.xml',
             'http://www.lemonde.fr/technologies/rss_full.xml',
             'http://www.metalorgie.com/feed/news',