#430 Error in edit_globals.php line 219 ?

[Amiel Elboim]

Hi.
Does somebody know whether there's an error in line 219 of interface/super/edit_globals.php) ?

        if(fldvalue){

Now the variable missing ‘$’ and without condition...

[ViSolve]

Hi Amiel,

Yes line 219 in interface/super/edit_globals.php has an error. Adding the $ symbol here if($fldvalue) will resolve the issue.

Thanks
OpenEMR Customization/Support Team,
ViSolve Inc
services@visolve.com | Phone: 408-850-2243
Demo's @ ViSolve Demo Library

[Brady Miller]

Hi,

To ensure do not introduce a bug in a crucial place, recommend doing some analysis on this. Appears to be added here:
https://github.com/openemr/openemr/commit/50d7a3c8b6054114db727111265c37f7ed60a207#diff-728da937d36bc012590e867af490fe15

-brady

[Oleg Sverdlov]

This feature prevented entering empty parameters into global configuration.

[Brady Miller]

Hi,
There are lots of globals that are empty, though (see the Portal tab).
-brady

[Oleg Sverdlov]

In here https://github.com/openemr/openemr/pull/49 he had tried to fix that error.

[Oleg Sverdlov]

PR 134 https://github.com/openemr/openemr/pull/134 fixes this problem and optimizes time needed to save globals.