>i am not familiar with the detailed current handling of configuration in<br>>the session, but IMHO this should only be required once per session<br><br>Great idea, Sebastian!<br><br>Fetching settings from db only once per session would be vital in decreasing server load and further increasing performance. <br>
<br>I believe Marc Delisle also suggested this.  <br><br>I will see how I can go about implementing this.  Thanks!<br><br>Also, I found a minor bug in my code.  Extracted db value for LoginCookieValidity turns out to be non-INT type by default.  So, if you have tried running the script, you may have realized that your session breaks in less than expected time.  <br>
Multiplying db value by 1 should do the trick.<br><br>--------------------------------------------------<br>Best regards,<br>Zeeshan Mughal<br>Email:  <a href="mailto:zeeshanmughal@ieee.org">zeeshanmughal@ieee.org</a><br>
Web:  <a href="http://www.zixan.info">http://www.zixan.info</a><br><br><br>On Thu, Aug 13, 2009 at 2:08 AM, Sebastian Mendel <<a href="mailto:lists@sebastianmendel.de">lists@sebastianmendel.de</a>> wrote:<br>><br>
> On 13.08.2009 07:21, Zeeshan M. wrote:<br>> >  >you should try to replace this .... with a single function call, e.<br>> > g.: $GLOBALS['cfg']['PropertiesNumColumns'] =<br>> > PMA_permUsrPrefGet('PropertiesNumColumns');<br>
> ><br>> > This would help put the code in one central location.  This is probably<br>> > better than having the code scattered all over.<br>><br>> yes<br>><br>> > But I don't think it<br>
> > would help reduce the amount of code being written, if it were one of<br>> > the goals.  Some of the settings are not directly applied to global<br>> > config vars so they have to go through conditional statements.<br>
><br>> btw. why do you not just apply all user settings at once?<br>><br>> i am not familiar with the detailed current handling of configuration in<br>> the session, but IMHO this should only be required once per session<br>
><br>> even if this is not an option, in most cases you set only one user<br>> option and this could be done with a single function call<br>><br>><br>><br>><br>> i would suggest include the user pref class in common.inc.php and apply<br>
> all user settings at once, additionally i would move all related code<br>> into the class (except the initial call of course)<br>><br>> Michal, what do you think?<br>><br>><br>> --<br>> Sebastian Mendel<br>
><br>> ------------------------------------------------------------------------------<br>> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day<br>> trial. Simplify your report design, integration and deployment - and focus on<br>
> what you do best, core application coding. Discover what's new with<br>> Crystal Reports now.  <a href="http://p.sf.net/sfu/bobj-july">http://p.sf.net/sfu/bobj-july</a><br>> _______________________________________________<br>
> Phpmyadmin-devel mailing list<br>> <a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>> <a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
<br>