[Phpmyadmin-devel] GSOC 2012 - Refactoring: Insert/edit, Privileges, Operations, Structure
Thilina Buddika Abeyrathna
thilinaabeyrathna at gmail.com
Fri Mar 23 02:57:04 CET 2012
On Sat, Mar 17, 2012 at 9:30 PM, Thilina Buddika Abeyrathna <
thilinaabeyrathna at gmail.com> wrote:
> On Sat, Mar 17, 2012 at 8:38 PM, Marc Delisle <marc at infomarc.info> wrote:
>> Le 2012-03-17 10:30, Thilina Buddika Abeyrathna a écrit :
>> > Hi Marc,
>> > I did a feature request and uploaded the patch file  to Sourceforge.
>> > it is relevant to Context. Therefor I think it will review after the
>> > deadline. So I like to much familiar with that Refactoring project.
>> > When I implementing above feature I went through the tbl_change.php
>> file. I
>> > need to familiar with other files also which is relevant to above
>> > So if you have any idea, please let me know, I am grateful if I could
>> > to know your suggestions.
>> Hi Thilina,
>> I have reviewed your patch.
>> About refactoring, I have an idea. In the master branch,
>> tbl_structure.php, look at lines 199-209. Can you suggest some
>> refactoring of these lines?
> Thanks Marc. I'll look at that file.
Sorry for the delay to reply. I had a mid semester exam. So I look at the
tbl_structure.php file and I have some suggestion for refactoring of line
First thing is it is better to have an underscore for id field
<th id="th_<?php echo ++$i; ?>"></th>
Second one is,
when using the if condition or for loop with html, it is better to use this
<php? if(condition) : ?>
<?php endif; ?>
Thilina Buddika Abeyrathna,
Department of Computer Engineering,
Faculty Of Engineering,
University of Peradeniya,
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Developers