[4.4] Fix strpos(): Passing null is deprecated (#42773)

QuyTon · Quy · laoneo · web-flow · commit 0aba9c897011 · 2024-02-09T21:36:20.000+01:00
* Fix deprecated warning

* Better code.

Co-authored-by: Allon Moritz &lt;allon.moritz@digital-peak.com&gt;

---------

Co-authored-by: Quy &lt;quy@fluxbb.org&gt;
Co-authored-by: Allon Moritz &lt;allon.moritz@digital-peak.com&gt;
diff --git a/plugins/content/fields/src/Extension/Fields.php b/plugins/content/fields/src/Extension/Fields.php
@@ -68,7 +68,7 @@ public function onContentPrepare($context, &$item, &$params, $page = 0)
         }
 
         // Prepare the full text
-        if (property_exists($item, 'fulltext') && strpos($item->fulltext, 'field') !== false) {
+        if (!empty($item->fulltext) && strpos($item->fulltext, 'field') !== false) {
             $item->fulltext = $this->prepare($item->fulltext, $context, $item);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public function onContentPrepare($context, &$item, &$params, $page = 0)`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`// Prepare the full text`
`71`		`- if (property_exists($item, 'fulltext') && strpos($item->fulltext, 'field') !== false) {`
	`71`	`+ if (!empty($item->fulltext) && strpos($item->fulltext, 'field') !== false) {`
`72`	`72`	`$item->fulltext = $this->prepare($item->fulltext, $context, $item);`
`73`	`73`	`}`
`74`	`74`	`}`