[drupal-devel] [bug] New pager code broken taxonomy pagers

[Morbus Iff]

Issue status update for http://drupal.org/node/23687

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Morbus Iff
 Reported by:  Morbus Iff
 Updated by:   Morbus Iff
 Status:       patch

Which part of comment.module? I hope you don't think the crazy "thread
1.1.2/, 1.2" stuff is /better/.




Morbus Iff



Previous comments:
------------------------------------------------------------------------

May 26, 2005 - 14:44 : Morbus Iff

I've not yet looked into it, but I doubt I'll be able to fix it (the
original taxonomy/term pager code was a hack that moshe taught me
about). In a nutshell, though, the recent pager cleanup in head has
broken the term pagers for vocabularies.




------------------------------------------------------------------------

May 26, 2005 - 15:21 : Morbus Iff

Attachment: http://drupal.org/files/issues/_p_23687_taxopager.patch (1.88 KB)

Patch attached.




------------------------------------------------------------------------

May 27, 2005 - 02:05 : Steven

Why can't this code use pager_query() and query the database directly?
If free tagging vocabularies can get so big, it seems a very
inefficient thing to fetch the entire list.




------------------------------------------------------------------------

May 27, 2005 - 02:05 : Steven

Note: if you need to fetch a list of tags ordered by hierarchy in one
query, you should look at how comment.module does it.