<br><br><div class="gmail_quote">On Sat, Aug 18, 2012 at 1:08 AM, 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">

Thilina Buddika Abeyrathna a écrit :<br>
<div class="im">> On Fri, Aug 17, 2012 at 6:35 PM, Rouslan Placella <<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>>wrote:<br>
><br>
>> On 17/08/2012 13:17, Thilina Buddika Abeyrathna wrote:<br>
>><br>
>>><br>
>>> On Fri, Aug 17, 2012 at 4:31 PM, Rouslan Placella <<a href="mailto:rouslan@placella.com">rouslan@placella.com</a><br>
>>> <mailto:<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>>> wrote:<br>
>>><br>
>>>     On 17/08/2012 12:00, Thilina Buddika Abeyrathna wrote:<br>
>>>      ><br>
>>>      ><br>
>>>      > On Fri, Aug 17, 2012 at 1:42 PM, Rouslan Placella<br>
>>>     <<a href="mailto:rouslan@placella.com">rouslan@placella.com</a> <mailto:<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>><br>
</div>>>>      > <mailto:<a href="mailto:rouslan@placella.com">rouslan@placella.com</a> <mailto:<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>>>**><br>
<div class="im">>>> wrote:<br>
>>>      ><br>
>>>      >     Hi all,<br>
>>>      ><br>
>>>      >     I've found some obviously broken HTML tags on<br>
>>>     tbl_structure.php and<br>
>>>      >     I assume that this is due to the recent refactoring of this<br>
>>>     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<br>
>>>     clickable.<br>
>>>      >     Screenshot attached.<br>
>>>      ><br>
>>>      >     Another thing that I noticed in the new *_structure code are<br>
>>>     the new<br>
>>>      >     function names. Not sure if it's just me, but after a 2<br>
>>>     minute look<br>
>>>      >     I found all of these functions where IMO the names can be<br>
>>>     improved:<br>
>>>      ><br>
</div>>>>      >     PMA___**getHtmlForPrintViewAndDataDict**__ionaryLinks - too<br>
<div class="im">>>> long?<br>
>>>      >     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<br>
<div><div class="h5">>>> camel case<br>
>>>      ><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>
>>>     The bug is an unclosed div tag at line 1831 in structure.lib.php<br>
>>><br>
>>> Thanks  Rouslan.<br>
>>><br>
>> Hi Thilina,<br>
>><br>
>> There seems to be a whole bunch of other bugs on the table structure page<br>
>> that have to do with the dropdown menu. It would be great if you could look<br>
>> at this asap as these have caused breakages in my branch.<br>
>><br>
>> To reproduce, create a table with the following definition:<br>
>><br>
>> CREATE TABLE IF NOT EXISTS `gfdfgd` (<br>
>>   `a` int(11) NOT NULL,<br>
>>   `s` linestring NOT NULL,<br>
>>   `f` text NOT NULL,<br>
>>   PRIMARY KEY (`a`)<br>
>> ) ENGINE=MyISAM DEFAULT CHARSET=latin1 ROW_FORMAT=COMPACT;<br>
>><br>
>> Then, please look at the attached screenshots.<br>
>> In screenshot-1, the fulltext link is missing from the text column.<br>
>> In screenshot-2, both the fulltext and spatial links are missing.<br>
>> In screenshot-3, you can see an unwanted link in the top left and a<br>
>> fulltext link for a spatial column.<br>
>> In screenshot-4, you can see an spatial link for a text column.<br>
>><br>
>> I didn't do a bisect...<br>
>><br>
>><br>
> Thank you very much  Rouslan. I'll look at those issues.<br>
<br>
</div></div>I see that Thilina has fixed most of these problems, except that the<br>
More drop-down is missing some of the links and has incorrect other<br>
links (for example, a linestring column should not have a link to add a<br>
FULLTEXT index).<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>Fixed in cd2177ec4b0fa58be20ca563cc7bcbd645126b93 commit. </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>