By jibbajabba on
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
Thanks for the sample code, I
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.
That's a good idea
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?
Project issue created
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.
theme based solution
I posted my solution here - u dont have to patch the module..its theme based:
http://drupal.org/node/10820#comment-1452074
Squelle Group, s.r.o. - Drupal web development, consulting and training.