Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP8 problem with each() function #1553

Closed
S0FTWEX opened this issue Apr 17, 2021 · 2 comments
Closed

PHP8 problem with each() function #1553

S0FTWEX opened this issue Apr 17, 2021 · 2 comments
Labels

Comments

@S0FTWEX
Copy link
Contributor

S0FTWEX commented Apr 17, 2021

Preconditions (*)

  1. OpenMage 20.0.8
  2. PHP 8.0.3

Steps to reproduce (*)

  1. In Admin Panel, go to System -> Configuration ->Advanced -> Admin
  2. Page goes white blank with this error in log:
PHP Fatal error:  Uncaught Error: Call to undefined function each() in /app/code/core/Mage/Adminhtml/Model/System/Config/Source/Admin/Page.php:127
Stack trace:
#0 /app/code/core/Mage/Adminhtml/Model/System/Config/Source/Admin/Page.php(118): Mage_Adminhtml_Model_System_Config_Source_Admin_Page->_buildMenuArray()
#1 /app/code/core/Mage/Adminhtml/Model/System/Config/Source/Admin/Page.php(118): Mage_Adminhtml_Model_System_Config_Source_Admin_Page->_buildMenuArray()
#2 /app/code/core/Mage/Adminhtml/Model/System/Config/Source/Admin/Page.php(42): Mage_Adminhtml_Model_System_Config_Source_Admin_Page->_buildMenuArray()
#3 /app/code/core/Mage/Adminhtml/Block/System/Config/Form.php(463): Mage_Adminhtml_Model_System_Config_Source_Admin_Page->toOptionArray()
#4 /app/code/core/Mage/Adminhtml/Block/System/Config/Form.php(229): Mage_Adminhtml_Block_System_Config_Form->initFields()

Solution

It would be great to finally merge pull request #1391 (or at least the files from this PR with each() functions inside).

@S0FTWEX S0FTWEX added the bug label Apr 17, 2021
@fballiano
Copy link
Contributor

fballiano commented Apr 26, 2021

#1391 has been merged and released in 20.0.10 so i think we can close this issue.

Edit: I've tested 20.0.10 on php8 and it doesn't cause this error.

@S0FTWEX
Copy link
Contributor Author

S0FTWEX commented Apr 27, 2021

I agree, issue can be closed, problem has been fixed after merging #1391.

@S0FTWEX S0FTWEX closed this as completed Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants