ContentExtractor: Fix invalid argument type

jtojnar · jtojnar · commit b10b40e5c949 · 2023-02-16T15:30:29.000+01:00
This was tripping up selfoss: date_parse(): Passing null to parameter #1 ($datetime) of type string is deprecated [vendor/j0k3r/graby/src/Extractor/ContentExtractor.php:722] [vendor/j0k3r/graby/src/Extractor/ContentExtractor.php:722] date_parse() [vendor/j0k3r/graby/src/Extractor/ContentExtractor.php:541] Graby\Extractor\ContentExtractor->validateDate() [vendor/j0k3r/graby/src/Graby.php:374] Graby\Extractor\ContentExtractor->process() [vendor/j0k3r/graby/src/Graby.php:181] Graby\Graby->doFetchContent() [src/spouts/rss/fulltextrss.php:97] Graby\Graby->fetchContent() [src/spouts/rss/fulltextrss.php:74] spouts\rss\fulltextrss->getFullContent() [src/spouts/Item.php:117] spouts\rss\fulltextrss->spouts\rss\{closure}() [workbench.php:112] spouts\Item->getContent()
diff --git a/src/Extractor/ContentExtractor.php b/src/Extractor/ContentExtractor.php
@@ -713,12 +713,13 @@ public function getNextPageUrl()
     /**
      * Validate and convert a date to the W3C format.
      *
-     * @param string $date
+     * @param string|null $date
      *
      * @return string|null Formatted date using the W3C format (Y-m-d\TH:i:sP) OR null if the date is badly formatted
      */
     public function validateDate($date)
     {
+        $date = (string) $date;
         $parseDate = (array) date_parse($date);
 
         // If no year has been found during date_parse, we nuke the whole value

Original file line number	Diff line number	Diff line change
`@@ -713,12 +713,13 @@ public function getNextPageUrl()`
`713`	`713`	`/**`
`714`	`714`	`* Validate and convert a date to the W3C format.`
`715`	`715`	`*`
`716`		`- * @param string $date`
	`716`	`+ * @param string\|null $date`
`717`	`717`	`*`
`718`	`718`	`* @return string\|null Formatted date using the W3C format (Y-m-d\TH:i:sP) OR null if the date is badly formatted`
`719`	`719`	`*/`
`720`	`720`	`public function validateDate($date)`
`721`	`721`	`{`
	`722`	`+ $date = (string) $date;`
`722`	`723`	`$parseDate = (array) date_parse($date);`
`723`	`724`
`724`	`725`	`// If no year has been found during date_parse, we nuke the whole value`