<br><br><div class="gmail_quote">On Tue, May 15, 2012 at 10:37 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Le 2012-05-15 09:42, Atul Pratap Singh a écrit :<br>
<div><div class="h5">> On Wed, May 9, 2012 at 2:28 AM, Atul Pratap Singh <<br>
> <a href="mailto:atulpratapsingh05@gmail.com">atulpratapsingh05@gmail.com</a>> wrote:<br>
><br>
>><br>
>><br>
>> On Wed, May 9, 2012 at 1:57 AM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>> wrote:<br>
>><br>
>>> Le 2012-05-08 16:18, Atul Pratap Singh a écrit :<br>
>>>> On Tue, May 8, 2012 at 3:24 PM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
>>> wrote:<br>
>>>><br>
>>>>> Le 2012-05-08 02:44, Atul Pratap Singh a écrit :<br>
>>>>> (...)<br>
>>>>>><br>
>>>>>> So, to move HTML to lib/include files, slicing proper reusable HTML<br>
>>>>> blocks<br>
>>>>>> from the script should be done, followed by moving it to a lib/include<br>
>>>>>> file, and then calling the include file at proper places. Is this the<br>
>>>>> way,<br>
>>>>>> I should proceed ?<br>
>>>>>><br>
>>>>><br>
>>>>> Hi Atul,<br>
>>>>> it depends: do you plan to create a class for this, or use global<br>
>>>>> functions?<br>
>>>>><br>
>>>><br>
>>>> For an example using a class, look at libraries/Menu.class.php. The<br>
>>>>> _getBreadcrumbs() function returns some HTML and is called afterwards<br>
>>>>> with an "echo".<br>
>>>>><br>
>>>>> You should have just one library file that will contain all your<br>
>>>>> refactored functions for tbl_select.php.<br>
>>>>><br>
>>>>> Marc, yes I would like to form a class. As there is already a library<br>
>>> for<br>
>>>> "tbl_select.php" i.e. "tbl_select.lib.php" and it contains some<br>
>>> functions,<br>
>>>> Should I refactor it and convert it into a class file for<br>
>>> "tbl_select.php"<br>
>>>> and "tbl_zoom_select.php" ?<br>
>>>><br>
>>><br>
>>> Yes, good idea.<br>
>>><br>
>> Thanks Marc, will show you a few commits in a day or two :).<br>
>><br>
> Hi Marc,<br>
> Sorry for the delay. I have done some refactoring in tbl_select.php. Part<br>
> of the script responsible for displaying table search form(which was<br>
> earlier done using HTML-PHP mix) has now been converted into functions<br>
> displaying HTML. Please checkout [0].<br>
><br>
> [0]<br>
> <a href="https://github.com/zixtor/phpmyadmin/commit/36d56fb2ea56e47b3979cfb07ff3ad5a1b4f1b2f" target="_blank">https://github.com/zixtor/phpmyadmin/commit/36d56fb2ea56e47b3979cfb07ff3ad5a1b4f1b2f</a><br>
<br>
</div></div>Thanks Atul, I started to have a look and added my comments directly to<br>
github.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
</div></div><br>------------------------------------------------------------------------------<br>
Live Security Virtual Conference<br>
Exclusive live event will cover all the ways today's security and<br>
threat landscape has changed and how IT managers can respond. Discussions<br>
will include endpoint security, mobile security and the latest in malware<br>
threats. <a href="http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/" target="_blank">http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/</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>Hi Marc,<br><div><span style="background-color:rgb(251,251,251);color:rgb(51,51,51);font-family:Helvetica,arial,freesans,clean,sans-serif;font-size:13px;line-height:20px">I have now converted the SQL query execution part of tbl_select.php into functions. </span>Please checkout [0]</div>

<div><br></div><div>[0] <a href="https://github.com/zixtor/phpmyadmin/commit/360e2063ea271030d3042e3d0d2b6b5b2eccc938">https://github.com/zixtor/phpmyadmin/commit/360e2063ea271030d3042e3d0d2b6b5b2eccc938</a><br clear="all">

<div><br></div>-- <br>Atul Pratap Singh<div>Senior Undergraduate Student</div><div>Institute Of Technology,<br>Banaras Hindu University, Varanasi,<br>India.</div><br>
</div>