<br><br><div class="gmail_quote">On Sun, May 29, 2011 at 4:31 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Le 2011-05-29 03:11, Ammar Yasir a écrit :<br>
<div class="im">>> Ammar,<br>
>> it's OK to have two components but please refactor common code into<br>
>> functions and put them in a new libraries/tbl_select.lib.php that will<br>
>> be called from both components; otherwise it will be difficult to maintain.<br>
>><br>
>> An example of needed refactoring: the "Gets the list and number of<br>
>> fields" section.<br>
>><br>
>> I did refactoring in two areas:<br>
> 1) As you proposed, Getting the list and number of fields.<br>
> 2) The where clause generation in the query-building part.<br>
><br>
> Should we also do for the HTML part? Some part of that (for example the<br>
> foreign keys display part) is common but also has HTML content embedded.<br>
> Apart from that since the zoom-search form takes the field as input so rest<br>
> of the form is based on that constraint. So I was thinking that the HTML<br>
> part can stay as it is.<br>
><br>
</div>Ammar,<br>
<br>
The foreign keys display part is a good example of needed refactoring.<br>
<br>
I don't know how much HTML refactoring you are willing to do, but some<br>
HTML is easy to refactor, for example:<br>
- the section that sets $titles['Browse']<br>
- the beginning of the table with headers Column, Type, etc<br>
</blockquote><div></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> <br></blockquote><div>Parts refactored are:<br>- Setting titles (titles['Browse']])<br>

- getting fields list (getting column details)<br>
- setting sub-tabs (Table search and Zoom search)<br>- setting table header of QBE display<br>
- displaying foreign data <br>
- search criteria input elements (HTML content generation for the input elements)<br> </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

In libraries/tbl_select.lib.php, please describe for each function the<br>
parameters and return values; see an example in<br>
libraries/server_synchronize.lib.php.<br>
<br>
About the syntax, please use spaces around operators. Instead of<br>
$i-1<br>
use<br>
$i - 1<br>
<br>
There is a case of undefined variable. To reproduce, on sakila.actor,<br>
Search, Table search, last name Wilson, Go:<br>
<br>
Notice in ./libraries/tbl_select.lib.php#129<br>
Undefined variable: w<br>
<br>
Backtrace<br>
<br>
./tbl_select.php#329: PMA_tbl_search_getWhereClause(<br>
string ,<br>
string last_update,<br>
string timestamp,<br>
string ,<br>
string =,<br>
boolean ,<br>
)<br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="h5">Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
------------------------------------------------------------------------------<br>
vRanger cuts backup time in half-while increasing security.<br>
With the market-leading solution for virtual backup and recovery,<br>
you get blazing-fast, flexible, and affordable data protection.<br>
Download your free trial now.<br>
<a href="http://p.sf.net/sfu/quest-d2dcopy1" target="_blank">http://p.sf.net/sfu/quest-d2dcopy1</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>
</div></div></blockquote></div><br>