<br><br><div class="gmail_quote">On Fri, Aug 17, 2012 at 1:42 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">

Hi all,<br>
<br>
I've found some obviously broken HTML tags on tbl_structure.php and I assume that this is due to the recent refactoring of this page. I couldn't find where the issue is in the code, but it manifests itself in the "more" dropdown as links there are now not clickable. Screenshot attached.<br>


<br>
Another thing that I noticed in the new *_structure code are the new function names. Not sure if it's just me, but after a 2 minute look I found all of these functions where IMO the names can be improved:<br>
<br>
PMA_<u></u>getHtmlForPrintViewAndDataDict<u></u>ionaryLinks - too long?<br>
PMA_SortableTableHeader - starts with a capital unlike others<br>
PMA_getAliasAndTruename - incorrect camel case<br>
PMA_getStuffForEnginetable - meaningless + incorrect camel case<br>
PMA_<u></u>getHtmlForCheckAlltableColumn - incorrect camel case<br>
PMA_getHtmlForSomeLinks - meaningless<br>
PMA_getHtmlForRowStatstableRow - incorrect camel case<br>
getHtmlForRowStatsTable - no PMA_ prefix<br>
PMA_<u></u>getHtmlForDistincValueAction - spelling: "Distinct"<br>
PMA_<u></u>getHtmlForActionsIntableStruct<u></u>ure - incorrect camel case<br>
<br>
Bye,<br>
Rouslan<br><br></blockquote><div><br></div><div>Hi  Rouslan,</div><div>Thank you for spent time to look at my code. And I'll improve those function names,</div><div>And I'll look at the mentioned bug also.</div></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>