<br><br><div class="gmail_quote">On Fri, Aug 17, 2012 at 4:31 PM, Rouslan Placella <span dir="ltr"><<a href="mailto:rouslan@placella.com" target="_blank">rouslan@placella.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On 17/08/2012 12:00, Thilina Buddika Abeyrathna wrote:<br>
><br>
><br>
> On Fri, Aug 17, 2012 at 1:42 PM, Rouslan Placella <<a href="mailto:rouslan@placella.com">rouslan@placella.com</a><br>
</div><div class="im">> <mailto:<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>>> wrote:<br>
><br>
>     Hi all,<br>
><br>
>     I've found some obviously broken HTML tags on tbl_structure.php and<br>
>     I assume that this is due to the recent refactoring of this page. I<br>
>     couldn't find where the issue is in the code, but it manifests<br>
>     itself in the "more" dropdown as links there are now not clickable.<br>
>     Screenshot attached.<br>
><br>
>     Another thing that I noticed in the new *_structure code are the new<br>
>     function names. Not sure if it's just me, but after a 2 minute look<br>
>     I found all of these functions where IMO the names can be improved:<br>
><br>
</div>>     PMA___getHtmlForPrintViewAndDataDict__ionaryLinks - too long?<br>
<div class="im">>     PMA_SortableTableHeader - starts with a capital unlike others<br>
>     PMA_getAliasAndTruename - incorrect camel case<br>
>     PMA_getStuffForEnginetable - meaningless + incorrect camel case<br>
</div>>     PMA___getHtmlForCheckAlltableColumn - incorrect camel case<br>
<div class="im">>     PMA_getHtmlForSomeLinks - meaningless<br>
>     PMA_getHtmlForRowStatstableRow - incorrect camel case<br>
>     getHtmlForRowStatsTable - no PMA_ prefix<br>
</div>>     PMA___getHtmlForDistincValueAction - spelling: "Distinct"<br>
>     PMA___getHtmlForActionsIntableStruct__ure - incorrect camel case<br>
<div class="im">><br>
>     Bye,<br>
>     Rouslan<br>
><br>
><br>
> Hi  Rouslan,<br>
> Thank you for spent time to look at my code. And I'll improve those<br>
> function names,<br>
> And I'll look at the mentioned bug also.<br>
<br>
</div>The bug is an unclosed div tag at line 1831 in structure.lib.php<br>
<br></blockquote><div>Thanks  Rouslan.</div></div><div><br></div>-- <br><font face="georgia, serif">Regards.</font><br><br><font face="'trebuchet ms', sans-serif" color="#660000">Thilina Abeyrathna</font><font face="georgia, serif"><br>

</font><div><font face="georgia, serif" size="1">Gtalk : thilinaabeyrathna</font></div><div><font face="georgia, serif" size="1">skype: thilinabuddika88</font></div><div><font face="georgia, serif"><font size="1"><a href="http://thilinaa.wordpress.com" target="_blank">thilinaa.wordpress.com</a> <br>

</font><br></font></div><br>