Hello,

i have a problem with taxonomy module. I don't use views to display lists for terms, only taxonomy module. Also i use i18n module (not sure if this is related).

I created a vocabulary (with localized terms). When someone visits a taxonomy/term page he correctly sees all related nodes in selected language (detected by path prefix). But pager creates more pages!

F.e. I have a taxonomy/term page with 12 nodes in language A and 12 in language B. Taxonomy/term page lists correctly only selected language nodes, but pager creates 3 pages. The first page contains the 10 first nodes, the second (taxonomy/term/x/?page=1) contains the remaining two and the third (taxonomy/term/x/?page=2) contains nothing. I'm guessing that page calculates 24 nodes (12 per language).

Is there something i miss about configuration or is this a bug?

Thank you, in advance and best wishes for the new year.

ps. Please, excuse my terrible English.

CommentFileSizeAuthor
#4 i18n-taxonomy-pager-1392024-4.patch734 byteswebflo

Comments

balagan’s picture

I have similar problem. I am also using taxonomy and i18n for translating taxonomy terms by the way (still not sure it is the problem). I have some 190 hierarchical terms. The pager shows 2 pages, although the first page (../taxonomy/dictionary/list) lists all the 190 terms, and the second page (../taxonomy/dictionary/list?page=1) only shows the second half of the terms.

xjm’s picture

Project: Drupal core » Internationalization
Version: 7.10 » 7.x-1.x-dev
Component: taxonomy.module » Taxonomy

i18n does a lot of custom stuff, so moving this issue to that queue unless someone can reproduce it with just core.

jose reyero’s picture

It looks like we are query altering the main query (adding language condition) but not the count query.

webflo’s picture

Status: Active » Needs review
StatusFileSize
new734 bytes

Here is a patch. I think we should open a Drupal core issue. A proper query tag would be helpful tag ...

calculus’s picture

webflo it worked! Thank you. Will this patch be committed?

kustolovic’s picture

Tested. the patch solved the problem

no2e’s picture

Status: Needs review » Reviewed & tested by the community

I'm not sure about this whole process, but because calculus and kustolovic tested it, the status can be set to RTBC, so it could be committed, right?

webflo’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +Needs tests

Commit 07f6d61 on 7.x-1.x
Thanks!

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