Adding a new Model is sorted incorrectly in Treelist

[highnote]

Volkswagen is coming out with a new model for 2008 called Neeza.  I added this model to the Volkswagen Make in the Make/Model Tree List Edit screen and then moved it just above the Passat so that the selection would be in alphabetical order for the seller to easily find.  However, the treelist pull-down (when adding a listing) still has it at the bottom of the list as it seems to be sorting on the ID rather than the order specified in the "Edit Tree" menu in the Admin Console.  The Tree List is correctly ordered when doing a search.  The problem only exists when adding or editing a listing as the owner.

If anyone is so inclined, please duplicate this issue and post your results.
Thanks and regards,
Lou

[Rodion]

highnote ,
I've reproduce this issue. We'll fix it in a few day and make the patch.

[makr1]

Hello, i also had this problem too, looking forward to a patch.
Is there any way when adding a new Make or Model it will automatically be place in alphabetical order, because using the up and down arrows take extremely long especially for moving a new Make up the whole list?

[Rodion]

makr1 ,
Yes, you're right it's not really easy to manage tree values. We'll try to improve tree type interface in the new version.

[Rodion]

The patch for incorrect sorting of tree values is ready and could be found here: http://forum.iautosoftware.com/index.php/topic,233.0.html

[highnote]

The functionality is now fixed for add/edit record, however, the order is now incorrect for the search menu.

In the function requesthtml_treelist_list(), the ORDER BY statement was taken out of the line in this patch.  I presume this is why the ordering is now incorrect for the search menus.   It should be put back in for the treelist to be correctly ordered.  (In exactly the same way the add/edit record was fixed in the function inputhtml_treelist_list().)

$sql = "SELECT * FROM `".$column['column']."` ORDER BY `order`";

I changed this line and the ordering is now correct in my configuration.  Please see the attachment for the updated file.

Rodion,
Please verify if what I suspect is true and ensure this gets integrated back into the main code stream for future releases.
Thanks,
Lou

[Rodion]

highnote ,
the changes you've made seems to be correct.

We've also fixed this bug in searched form in our patch.