There are times when I want a module to display nodes in alphabetical order by ascending title rather than by created date descending. So I hacked my taxonomy.module to let me send sort field and sort order paramaters to my taxonomy_select_nodes function calls. I've blogged it on urlgreyhot.

So I want to know if this is a bad thing? If not, can CVS HEAD be modified to include this change?

Comments

tdailey’s picture

Thanks for the sample code, I could use this as well.

My practice on modifying modules has been to create a new function whereever possible. It makes upgrades easier. So, I'll probably take your code and create taxonomy_select_nodes_sort or something. Of course, getting it into the CVS code would be great too.

jibbajabba’s picture

Sounds like good idea to create a new function. I hadn't thought of doing that, but it makes sense. I hope this could get incorporated into CVS if it makes sense. Seems like it wouldn't add any overhead, no?

nedjo’s picture

Thanks for the tips. I didn't find a project issue for this so I've created one and posted a patch based on your work here. See:

http://drupal.org/node/10839.

boreg’s picture

I posted my solution here - u dont have to patch the module..its theme based:

http://drupal.org/node/10820#comment-1452074