The list at admin/content/nodequeue should be sorted for humans.

CommentFileSizeAuthor
#6 nodequeue.module.624288.patch830 bytesjaydub
#5 624288.nodequeue.admin-list-case.patch928 bytesbdurbin
#4 624288.nodequeue.admin-list-case.patch956 bytesbdurbin
#1 624288.nodequeue.admin-list-case.patch699 bytesjoachim

Comments

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 624288.nodequeue.admin-list-case.patch699 bytes

One-line patch :)

ezra-g’s picture

ezra-g commented
Status: Needs review » Needs work

Looks good except that we should use drupal_strtolower.

ezra-g’s picture

ezra-g commented

If you want to patch it then great, otherwise I can get it later today probably.

bdurbin’s picture

bdurbin commented
Status: Needs work » Needs review
StatusFileSize
new 624288.nodequeue.admin-list-case.patch956 bytes

Picking up this thread. Changed to drupal_strtolower and made sure the secondary sort (which also uses the title) was case-insensitive as well.

bdurbin’s picture

bdurbin commented
StatusFileSize
new 624288.nodequeue.admin-list-case.patch928 bytes

Oops. Botched that last patch. Re-rolling.

jaydub’s picture

jaydub commented
StatusFileSize
new nodequeue.module.624288.patch830 bytes

Minor but I'd forgo setting the intermediate variable and just call drupal_strtolower() twice

ezra-g’s picture

ezra-g commented
Status: Needs review » Patch (to be ported)

Committed - thanks!
http://drupal.org/cvs?commit=471842

ezra-g’s picture

ezra-g commented
Status: Patch (to be ported) » Fixed

Committed to HEAD. Thanks!
http://drupal.org/cvs?commit=474702

Status: Fixed » Closed (fixed)

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