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.

Sign up for GitHub

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

Jump to bottom

When Using withPrefix Without Mappers on the Base, a PHP Notice is Generated #7

Closed
jimdelois opened this issue Jun 14, 2016 · 0 comments
Closed

When Using withPrefix Without Mappers on the Base, a PHP Notice is Generated #7

jimdelois opened this issue Jun 14, 2016 · 0 comments
Labels
bug

Comments

@jimdelois
Copy link
Contributor 

NOTICE: PHP message: PHP Warning:  Invalid argument supplied for foreach() in /app/vendor/improvframework/configuration/src/Configuration.php on line 130

This occurs because the private Configuration::$mappers variable is not correctly initialized to an array. When there are no mappers added to the base Configuration object (which effectively converts it to an array), then the iterating of this null variable causes a PHP warning.
@jimdelois jimdelois added the bug label Jun 14, 2016
jimdelois pushed a commit to jimdelois/configuration that referenced this issue Jun 14, 2016
Bugfix: Initializing the Configuration's Mappers Property
bbf7c9c 
Resolves improvframework#7
jimdelois pushed a commit to jimdelois/configuration that referenced this issue Jun 14, 2016
Bugfix: Initializing the Configuration's Mappers Property
b16712e 
Resolves improvframework#7
@jimdelois jimdelois mentioned this issue Jun 14, 2016
Merged
jimdelois added a commit to jimdelois/configuration that referenced this issue Sep 25, 2019
@jimdelois
Bugfix: Initializing the Configuration's Mappers Property
3ce6e4e 
Resolves improvframework#7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jimdelois