Ensure null value not passed to str_replace (#2901)

OpenMage · Jan 9, 2023 · 390a79c · 390a79c
1 parent 37fbac1
commit 390a79c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/app/code/core/Mage/Core/Model/Resource/Config.php b/app/code/core/Mage/Core/Model/Resource/Config.php
@@ -90,7 +90,7 @@ public function loadToXml(Mage_Core_Model_Config $xmlConfig, $condition = null)
             if ($r['scope'] !== 'default') {
                 continue;
             }
-            $value = str_replace($substFrom, $substTo, $r['value']);
+            $value = str_replace($substFrom, $substTo, (string)$r['value']);
             $xmlConfig->setNode('default/' . $r['path'], $value);
         }
 
@@ -107,7 +107,7 @@ public function loadToXml(Mage_Core_Model_Config $xmlConfig, $condition = null)
             if ($r['scope'] !== 'websites') {
                 continue;
             }
-            $value = str_replace($substFrom, $substTo, $r['value']);
+            $value = str_replace($substFrom, $substTo, (string)$r['value']);
             if (isset($websites[$r['scope_id']])) {
                 $nodePath = sprintf('websites/%s/%s', $websites[$r['scope_id']]['code'], $r['path']);
                 $xmlConfig->setNode($nodePath, $value);
@@ -136,7 +136,7 @@ public function loadToXml(Mage_Core_Model_Config $xmlConfig, $condition = null)
             if ($r['scope'] !== 'stores') {
                 continue;
             }
-            $value = str_replace($substFrom, $substTo, $r['value']);
+            $value = str_replace($substFrom, $substTo, (string)$r['value']);
             if (isset($stores[$r['scope_id']])) {
                 $nodePath = sprintf('stores/%s/%s', $stores[$r['scope_id']]['code'], $r['path']);
                 $xmlConfig->setNode($nodePath, $value);