Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.This patch reworks hook_menu_alter() for compatibility wiht other modules using taxonomy/term/% like Taxonomy Breadcrumb. Using this path instead of taxonomy/term/[tid] also prevents performance issues with large browsable indexes.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | hook_menu_alter_02.patch | 4.36 KB | xano |
| #10 | 494408_vocabindex_menu_alter.patch | 7.82 KB | MGN |
| #1 | hook_menu_alter_01.patch | 4.96 KB | xano |
| hook_menu_alter_00.patch | 4.93 KB | xano |
Comments
Comment #1
xanoCode comment love.
Comment #2
zarudnyi commentedBreadcrumb start to work, but two items not.
1) Instead term title, e. g. "Drugs", it show "Taxonomy term"
2) Node not showing, "There are currently no terms in this vocabulary."
O_o
Comment #3
xano1) Good one. This is a bug I will fix tomorrow.
2) Where, how and what? This message should only be shown if you visit an index page for a vocabulary that contains no terms.
Comment #4
zarudnyi commentedThanks for help, Xano
For 2) Right, the message shown on every index page that contains no terms, but in that page i whant to see node with this term. I mean if you have some node with "term", you can't brouse for it becose it not shown on page for it "term"
Comment #5
xanoI'm not sure if I fully understand you. Index pages are the pages you define a path for at Administer > Site building Vocabulary Index. At those pages you will never ever see nodes, just terms or a message that says the vocabulary this page is for doesn't contain any terms. For browsable indexes Vocabulary Index overrides
taxonomy/term/%for certain terms to display their child terms. If a term contains no other terms, you should see the default Taxonomy term page at those URLs, otherwise Vocabulary Index should list those terms.Comment #6
zarudnyi commentedSorry, i made mistake, i did't enable taxonomy/term/% views. Now i do but have "access denied" and warning: array_merge() [function.array-merge]: Argument #2 is not an array in domain\sites\all\modules\taxonomy_breadcrumb\taxonomy_breadcrumb.module on line 104. messages.
Comment #7
xanoThat's a bug in Taxonomy Breadcrumb, not in Vocabulary Index.
Comment #8
zarudnyi commentedOk, btw thanks a lot for your help.
Comment #9
Defender-1 commentedTried to apply this patch, but TortoiseSVN tells me that
"Could not retrieve revision Jun 2009 19:54:16 -0000 1.1.2.5.2.83 of the file Z:\home\sessii.net\www\sites\all\modules\vocabindex\vocabindex.module.
Patching is not possible!"
:-(
Comment #10
MGN commentedI manually patched the latest dev version. Everything seems to work fine, but I haven't really done a thorough review. Still, I thought I'd attach an updated patch to help move this issue along.
Comment #11
xanoCommitted to 6.x-2.x-dev. Ports to 5.x-2.x-dev and 7.x-2.x-dev will follow. The patch applies to 6.x-2.x-dev and is intended for future reference.
Comment #12
xanoVocabulary Index 5.x-y.z is no longer supported.
Comment #13
xano