* I'm not sure we agreed on the list that there was a need to enable/disable each setting one by one. * Allowing sysadmins to enable/disable each setting does not make code either more complex, or trivial to write. I think that it would give them more options certainly; do they really need these options or not differs from person to person. What are your thoughts?
*I agree with Herman's reaction, so it could be $cfg['Servers'][$i]['**userprefs']['enable'] = true|false;*
Sounds good. May be $cfg['Servers'][$i]['enable_userprefs'] would be better since enable is the only sub-array of "userprefs"? -------------------------------------------------- Best regards, Zeeshan Mughal Email: zeeshanmughal@ieee.org Web: http://www.zixan.info
On Fri, Jul 17, 2009 at 12:58 PM, Marc Delisle marc@infomarc.info wrote:
Zeeshan M. a écrit :
Hi,
I added configuration vars for all the settings currently under consideration. Please let me know if I am following the convention correctly, and/or if there is anything else.
I decided to get this done first as I would need to access these vars for user interfaces.
Zeeshan, I don't understand why you are using a syntax like $cfg['perm_storage'][$i]['font_size'] = TRUE;
As you are using $i it means this is a setting per server, right? So it should be, IMO, something like
$cfg['Servers'][$i]['userprefs']['font_size'];
which would follow the other $cfg['Servers'][$i] settings. And perm_storage is not meaningful, again IMO.
I agree with Herman's reaction, so it could be $cfg['Servers'][$i]['userprefs']['enable'] = true|false;
By the way, I'm not sure we agreed on the list that there was a need to enable/disable each setting one by one.
-- Marc Delisle http://infomarc.info
Enter the BlackBerry Developer Challenge This is your chance to win up to $100,000 in prizes! For a limited time, vendors submitting new applications to BlackBerry App World(TM) will have the opportunity to enter the BlackBerry Developer Challenge. See full prize details at: http://p.sf.net/sfu/Challenge _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel