Fixed php82 bug

fballiano · fballiano · commit 9b48a81e5235 · 2023-08-15T10:41:23.000+02:00
diff --git a/app/code/core/Mage/Widget/controllers/Adminhtml/WidgetController.php b/app/code/core/Mage/Widget/controllers/Adminhtml/WidgetController.php
@@ -34,10 +34,11 @@ public function indexAction()
     {
         // save extra params for widgets insertion form
         $skipped = $this->getRequest()->getParam('skip_widgets');
-        $skipped = Mage::getSingleton('widget/widget_config')->decodeWidgetsFromQuery($skipped);
+        if (is_string($skipped)) {
+            $skipped = Mage::getSingleton('widget/widget_config')->decodeWidgetsFromQuery($skipped);
+        }
 
         Mage::register('skip_widgets', $skipped);
-
         $this->loadLayout('empty')->renderLayout();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -34,10 +34,11 @@ public function indexAction()`
`34`	`34`	`{`
`35`	`35`	`// save extra params for widgets insertion form`
`36`	`36`	`$skipped = $this->getRequest()->getParam('skip_widgets');`
`37`		`- $skipped = Mage::getSingleton('widget/widget_config')->decodeWidgetsFromQuery($skipped);`
	`37`	`+ if (is_string($skipped)) {`
	`38`	`+ $skipped = Mage::getSingleton('widget/widget_config')->decodeWidgetsFromQuery($skipped);`
	`39`	`+ }`
`38`	`40`
`39`	`41`	`Mage::register('skip_widgets', $skipped);`
`40`		`-`
`41`	`42`	`$this->loadLayout('empty')->renderLayout();`
`42`	`43`	`}`
`43`	`44`