I just tested this (great ;) module, I took primary links into it and the order of the items was wrong. So I searched a little and found, that the menu_links.weight and menu_links.plid had no value (there was no changing weight action, I simple added them one by one). So, I think, it would be right to have "ml.mlid" in the "order by" clause (in footermap_get_menu) - it worked for me.

CommentFileSizeAuthor
#2 890358-d7.patch922 bytesmradcliffe
#1 890358-d7.patch1.61 KBmradcliffe
#1 890358-d6.patch932 bytesmradcliffe
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe’s picture

Version: 6.x-1.6 » 7.x-1.x-dev
Status: Active » Needs review
FileSize
932 bytes
1.61 KB

The default sort order is by menu weight and then link title. I want to make things a bit efficient so I'm adding plid on there. I think adding title would be appropriate.

mradcliffe’s picture

FileSize
922 bytes

Err, the d7 patch had another change in it. Whoops.

mradcliffe’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.