I found a discrepency in the code which was causing the ordering to fail.

Here is a patch to fix it.

CommentFileSizeAuthor
#3 myaccount_alter_order.patch1.2 KBkleinmp
myaccount_alter.patch1.22 KBwilco

Comments

wilco’s picture

wilco commented
Status: Active » Needs review

Upgrading issue.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Needs review » Postponed (maintainer needs more info)

Looking at your patch as far as I can see it just deletes some lines and adds the same exact ones back in. Am I missing something?

kleinmp’s picture

kleinmp commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new myaccount_alter_order.patch1.2 KB

It looks like wilco moved the code that does the reordering to be outside of the 'if' statement on line 102:
if (variable_get($cat_class .'_'. str_replace(" ", "", $key), 1) == 0) {

I tried it out and the patch worked for me.

*Ignore the attachment to this comment*

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Thanks wilco and matt.

kleinmp’s picture

kleinmp commented

This patch has been committed.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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