Hi,<br><br>> Configuration changes should go to config.inc.php, these are *defaults*.<br>> You copy them from configuration, you should rather use directly<br>> configuration data in your code.<br><br>Sure, I'll make these changed. :)<br>
<br>> Having defined list of variables with descriptions and functions to<br>> call to verify input is correct. It would make code more readable even<br>> when there will be more options.<br><br>Can you also please look over the test cases to see if more restrictions can be applied<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, Jul 30, 2009 at 5:20 AM, Michal Čihař <<a href="mailto:michal@cihar.com">michal@cihar.com</a>> wrote:<br>><br>> Hi<br>><br>> please configure your mailer to quote properly (and do not use html<br>
> mails), this way it is quite hard to track your replies.<br>><br>> Dne Wed, 29 Jul 2009 13:55:52 -0400<br>> "Zeeshan M." <<a href="http://zeeshan.jp">zeeshan.jp</a>@<a href="http://gmail.com">gmail.com</a>> napsal(a):<br>
><br>> > *<br>> > Why do you change config defaults (libraries/config.default.php)?<br>> > *<br>> > I had to set pma db var equal to "phpmyadmin" as for some reasons it was not<br>> > set to "phpmyadmin" by default.<br>
><br>> Configuration changes should go to config.inc.php, these are *defaults*.<br>><br>> > *<br>> > Why do you duplicate lot of information in permUsrPref class (user<br>> > name, table name, database link,...)?*<br>
> ><br>> > Such parameters are set only once in the constructor.  Can you please<br>> > elaborate with a minimal example what do you mean by duplication?<br>><br>> You copy them from configuration, you should rather use directly<br>
> configuration data in your code.<br>><br>> > *In server_perm_storage.php there is a huge code duplication, please<br>> > have just one piece code to handle setting the value and do not copy it<br>> > for every available optiofn.*<br>
> ><br>> > I thought each setting would have different test cases, that's why I set up<br>> > conditional statements separately.  May be I can reduce code by generalizing<br>> > test cases, but I am not sure if it would be a probable solution.  What are<br>
> > your thoughts?<br>><br>> Having defined list of variables with descriptions and functions to<br>> call to verify input is correct. It would make code more readable even<br>> when there will be more options.<br>
><br>> --<br>>        Michal Čihař | <a href="http://cihar.com">http://cihar.com</a> | <a href="http://phpmyadmin.cz">http://phpmyadmin.cz</a><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><br>