When extracting the terms, another method is `taxonomy_get_tree`, which will order the items according to the weight in the taxonomy. This should be preferable to `entity_load` which returns a list alphabetically (or possibly by term id).

CommentFileSizeAuthor
#4 interdiff.txt636 bytesefpapado
#4 webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-4.diff826 bytesefpapado
#3 webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-3.diff787 bytessvenryen
#2 webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-2.patch742 bytessvenryen

Comments

svenryen created an issue. See original summary.

svenryen’s picture

svenryen commented
Status: Active » Needs review
StatusFileSize
new webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-2.patch742 bytes

Patch solves the issue.

svenryen’s picture

svenryen commented
StatusFileSize
new webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-3.diff787 bytes

A bug was found in the patch. Here's an updated version.

efpapado’s picture

efpapado commented
StatusFileSize
new webpush-sort-taxonomy-terms-by-tree-position-instead-of-by-alphabet-3056378-4.diff826 bytes
new interdiff.txt636 bytes

Just a variable renaming, to be more consistent :)

  • efpapado committed 5de062e on 7.x-1.x 
    Issue #3056378 by svenryen, efpapado: Sort taxonomy terms by tree...
efpapado’s picture

efpapado commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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