#54 confusing setup screen

[Ed Davis]

on the user edit (or create new user page) churchinfo/UserEditor.php
bottom half, set permission.
imho, the current value column should just be the current value displayed, not a T or F choice.
as is the permission choice column doesn't do anything, but changing the current value resets the value. which i would expect the permission column was actually designed to do.

From post in help forum with name "confusing setup screen"

[Fred]

Logged In: YES
user_id=1752306
Originator: NO

Ed,

Actually there is a bug in SettingsIndividual.php, line 112

if(!($ucfg_permission || $_SESSION['bAdmin']))

should be

if(!(($ucfg_permission == 'TRUE') || $_SESSION['bAdmin']))

With this, the permission column allows admin to specify which field is customizable, and only fields with permission set to 'TRUE' will appear on the "My User Setting" screen and allows user to specify their own values. The original code mistakenly allows all checking to pass so all fields, with permission or not, are then customizable.

Fred

[USAF_Pride]

Logged In: YES
user_id=1530314
Originator: NO

Link to forum topic
https://sourceforge.net/forum/message.php?msg_id=4183633