[Phpmyadmin-devel] Bug in Ajax table Structure column Change

Aris Feryanto aris_feryanto at yahoo.com
Fri Aug 19 09:46:03 CEST 2011


Hi,

On 19 Agu 2011, at 13:57, Thilanka Kaushalya <lgtkaushalya at gmail.com> wrote:


Hi Marc,
>
>
>I found the place where this issue generates. When we try click on Table->Structure->Column->Change link it goes to the tbl_alter.php file and within that the "append_fields_form" is process by using tbl_properties.inc.php file. At line number 325 in tbl_properties.inc.php "PMA_getSupportedDatatypes(true, $type_upper);" function is call to get the table type select options. 
>
>
>You can find at line 3118 in "PMA_getSupportedDatatypes(true, $type_upper);" the option selection is set as 
>if ($selected == $value) {
>                    $retval .= "<option selected='selected'>$value</option>";
>                } else {
>                    $retval .= "<option>$value</option>";
>                }
> But if you use an echo you can see that $selected contains VARCHAR(45)as the value and $value contains VARCHAR as the value. So when we try to edit the tables which are having VARCHAR type columns it does not select the value at the options. 
>
>
>I can modify the logic by splitting the $selected by "(" and solve the issue. Is there any better solution you can suggest. If there any please let me know. Thank you. 
>

I did a bisecting in the official branch (using "git bisect" command) and found that 48149c261c633baf517455079a2524a9a9aac6a0 (message: "Use already calculated values", by Michal) is the first bad commit. Hope this helps.


--
Aris Feryanto




More information about the Developers mailing list