Skip to content

#91: Missing class Zend_Filter_Input saving a slider in M2.4.6 / PHP 8.1#92

Open
mageprashant wants to merge 1 commit intomageplaza:2.4from
mageprashant:2.4
Open

#91: Missing class Zend_Filter_Input saving a slider in M2.4.6 / PHP 8.1#92
mageprashant wants to merge 1 commit intomageplaza:2.4from
mageprashant:2.4

Conversation

@mageprashant
Copy link

#91: Missing class Zend_Filter_Input saving a slider in M2.4.6 / PHP 8.1

@codedge
Copy link

codedge commented Nov 9, 2023

This has been fixed in release 4.x, so this PR can be closed.

@Urmi0311
Copy link

Urmi0311 commented May 8, 2024

Hello
I am getting the below error while saving the slider as I have a core module version of 1.5.7 and I have installed this module in Magento-2.4.5p3

Error: Class "Magento\Framework\Filter\Filter
Input" not found in /var/www/html/magep3/app/code/Mageplaza/BannerSlider/Controller/Adminhtml/Slider/Save.php:191

I am attaching screenshots of the composer.json of mageplaza

Screenshot from 2024-05-08 14-58-30
Screenshot from 2024-05-08 15-06-22

Copy link

@pszeszko pszeszko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe something more general ?

if (class_exists('\Magento\Framework\Filter\FilterInput')) {
            $inputFilter = new \Magento\Framework\Filter\FilterInput(
                ['from_date' => $this->_dateFilter,],
                [],
                $data
            );
        } else {
            $inputFilter = new \Zend_Filter_Input(
                ['from_date' => $this->_dateFilter,],
                [],
                $data
            );
        }

@kristapsbriks
Copy link

This is still an issue - cannot save banners on release 4.1.1 @codedge

@kristapsbriks
Copy link

@namvq2001 fix was implemented in 8d63343
then presumably accidentally reverted in ab47955

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants