<br>Hi,<br><i><br>Why do you introduce syntax errors (calendar.php)?<br></i>
<br>I think I edited it by mistake, I'll fix it.<br><i><br>
Why do you change config defaults (libraries/config.default.php)?<br></i><div id=":18f" class="ii gt">
<br>I had to set pma db var equal to "phpmyadmin" as for some reasons it was not set to "phpmyadmin" by default.<br><i><br>
Why do you duplicate lot of information in permUsrPref class (user<br>
name, table name, database link,...)?</i><br><br>Such parameters are set only once in the constructor.  Can you please elaborate with a minimal example what do you mean by duplication?<br>
<br>
<i>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.</i><br clear="all"><br>I thought each setting would have different test cases, that's why I set up conditional statements separately.  May be I can reduce code by generalizing test cases, but I am not sure if it would be a probable solution.  What are your thoughts?<br>
<br>Thanks!<br></div>--------------------------------------------------<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><div class="gmail_quote">On Wed, Jul 29, 2009 at 4:59 AM, Michal Čihař <span dir="ltr"><<a href="mailto:michal@cihar.com">michal@cihar.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi<br>
<br>
Dne Tue, 28 Jul 2009 03:24:25 -0400<br>
"Zeeshan M." <<a href="http://zeeshan.jp" target="_blank">zeeshan.jp</a>@<a href="http://gmail.com" target="_blank">gmail.com</a>> napsal(a):<br>
<div class="im"><br>
> I uploaded a revision for patch #<br>
</div>> <goog_1248753684487>2825187<<a href="https://sourceforge.net/tracker/download.php?group_id=23067&atid=377410&file_id=336915&aid=2825187" target="_blank">https://sourceforge.net/tracker/download.php?group_id=23067&atid=377410&file_id=336915&aid=2825187</a>>at<br>

> <a href="http://sourceforge.net" target="_blank">sourceforge.net</a>.<br>
<br>
Why do you introduce syntax errors (calendar.php)?<br>
<br>
Why do you change config defaults (libraries/config.default.php)?<br>
<br>
Why do you duplicate lot of information in permUsrPref class (user<br>
name, table name, database link,...)?<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 option.<br>
<font color="#888888"><br>
--<br>
        Michal Čihař | <a href="http://cihar.com" target="_blank">http://cihar.com</a> | <a href="http://phpmyadmin.cz" target="_blank">http://phpmyadmin.cz</a><br>
</font><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" target="_blank">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" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
<br></blockquote></div><br>