getSettings should not return undefined settings, it should set it to NULL
Bug #932577 reported by
Adrian Liechti
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
PHPDevShell |
Fix Released
|
Low
|
TitanKing |
Bug Description
When i try to make a configuration to set some settings, but the settings do not exist at this time. I must set the settings Like this to handle it:
$setting = $this->
$setting2 = array('cfg1'=>NULL, 'cfg2'=>NULL);
$setting = array_merge(
Otherwise, it destroys the layout of the page, when i do not handle it like this.
Changed in phpdevshell: | |
status: | New → Confirmed |
importance: | Undecided → Low |
assignee: | nobody → TitanKing (titan-phpdevshell) |
milestone: | none → 3.1.4-stable |
Changed in phpdevshell: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I would say that your script breaks when you're trying to use your settings variables.
The normal working of PHP is to emit a warning when the code is using an undefined variable, which may be caught by the framework as an error (depending on your configuration).
It's actually a good thing (your code should use undefined variables and should not generate warnings), and the way you dealt with it is good programming.
I think it's not a problem with the framework.