13 Aug
2009
13 Aug
'09
9:08 a.m.
On 13.08.2009 07:21, Zeeshan M. wrote:
yes
you should try
to replace this .... with a single function call, e.
g.:
$GLOBALS['cfg']['PropertiesNumColumns'] =
PMA_permUsrPrefGet('PropertiesNumColumns');
This would help put the code in one central location. This is probably
better than having the code scattered all over. But I don't think it
would help reduce the amount of code being written, if it were one of
the goals. Some of the settings are not directly applied to global
config vars so they have to go through conditional statements.
btw. why do you not just apply all user settings at once?
i am not familiar with the detailed current handling of configuration in
the session, but IMHO this should only be required once per session
even if this is not an option, in most cases you set only one user
option and this could be done with a single function call
i would suggest include the user pref class in common.inc.php and apply
all user settings at once, additionally i would move all related code
into the class (except the initial call of course)
Michal, what do you think?
--
Sebastian Mendel