Fixed strip_tags(): Passing null to parameter #1 in Catalog/Model/Product/Option/Type/File.php (OpenMage#3775)

kiatng · web-flow · commit 207b7736d94b · 2024-02-08T10:49:02.000Z
diff --git a/app/code/core/Mage/Catalog/Model/Product/Option/Type/File.php b/app/code/core/Mage/Catalog/Model/Product/Option/Type/File.php
@@ -545,7 +545,8 @@ protected function _unserializeValue($value)
      */
     public function getPrintableOptionValue($optionValue)
     {
-        return strip_tags($this->getFormattedOptionValue($optionValue));
+        $value = $this->getFormattedOptionValue($optionValue);
+        return $value === null ? '' : strip_tags($value);
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -545,7 +545,8 @@ protected function _unserializeValue($value)`
`545`	`545`	`*/`
`546`	`546`	`public function getPrintableOptionValue($optionValue)`
`547`	`547`	`{`
`548`		`- return strip_tags($this->getFormattedOptionValue($optionValue));`
	`548`	`+ $value = $this->getFormattedOptionValue($optionValue);`
	`549`	`+ return $value === null ? '' : strip_tags($value);`
`549`	`550`	`}`
`550`	`551`
`551`	`552`	`/**`