Fixes 32 - Method getDateModified of RSS reader doesn't iterate over …

…different formats Signed-off-by: Frank Brückner <[email protected]>
laminas · Dec 22, 2020 · fe4b481 · fe4b481
1 parent 9d127b9
commit fe4b481
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 12 deletions.
diff --git a/src/Reader/Feed/Rss.php b/src/Reader/Feed/Rss.php
@@ -206,23 +206,22 @@ public function getDateModified()
                         DateTime::RSS,
                         DateTime::RFC822,
                         DateTime::RFC2822,
-                        null,
                     ];
                     foreach ($dateStandards as $standard) {
-                        try {
-                            $date = DateTime::createFromFormat($standard, $dateModified);
+                        $date = DateTime::createFromFormat(
+                            $standard,
+                            $dateModified
+                        );
+                        if ($date instanceof DateTime) {
                             break;
-                        } catch (\Exception $e) {
-                            if ($standard === null) {
-                                throw new Exception\RuntimeException(
-                                    'Could not load date due to unrecognised format'
-                                    . ' (should follow RFC 822 or 2822): ' . $e->getMessage(),
-                                    0,
-                                    $e
-                                );
-                            }
                         }
                     }
+                    if (! $date) {
+                        throw new Exception\RuntimeException(
+                            'Could not load date due to unrecognised'
+                            . ' format (should follow RFC 822 or 2822).'
+                        );
+                    }
                 }
             }
         }

diff --git a/test/Reader/Feed/RssTest.php b/test/Reader/Feed/RssTest.php
@@ -19,6 +19,7 @@
  */
 class RssTest extends TestCase
 {
+    /** @var string */
     protected $feedSamplePath;
 
     protected $expectedCats = [];
@@ -2178,6 +2179,22 @@ public function dateModifiedProvider(): array
         ];
     }
 
+    public function testGetDateModifiedShouldThrowExceptionForInvalidDate(): void
+    {
+        $feed = Reader\Reader::importString(
+            file_get_contents(
+                $this->feedSamplePath . '/datemodified/plain/invalid.xml'
+            )
+        );
+
+        $this->expectException(Reader\Exception\RuntimeException::class);
+        $this->expectExceptionMessage(
+            'Could not load date due to unrecognised format (should follow RFC 822 or 2822).'
+        );
+
+        $feed->getDateModified();
+    }
+
     /**
      * Get Hubs (Unencoded Text)
      *

diff --git a/test/Reader/Feed/_files/Rss/datemodified/plain/invalid.xml b/test/Reader/Feed/_files/Rss/datemodified/plain/invalid.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<rss version="2.0">
+    <channel>
+        <!-- missing year -->
+        <lastBuildDate>Sat, 07 Mar 08:03:50 +0000</lastBuildDate>
+    </channel>
+</rss>