After an out-of-the-box installation of the taxonomy access module certain nodes are displayed more than once on summary pages for certain users. In my case the superuser isn't bothered with this *feature*, but ie. anonymous users are. The problem seems to be related to the book module. In my setup each book has its own taxonomy and the forums are categorized into a separate taxonomy. The summary page of the forums doesn't repeat its nodes, whereas the one of the books does.

Does the patch that comes with the taxonomy access module solve this problem? Does the book module need a patch too? If so, when will those patches be incorporated into the officially downloadable versions of these modules?

Please don't tell me that I have to dive into the code myself , because I don't want to end up with a personally forked Drupal to which I can't apply any official upgrades and patches anymore without loosing my own code changes. Moreover, I don't want to have to remember which patches I made for every fresh Drupal installation I intend to perform.

Comments

sami_k’s picture

sami_k commented

this should be fixed by patch from nysus.

auctmore’s picture

auctmore commented

Do you mean the patch for the taxonomy access module or is there a patch for the book module too?

auctmore’s picture

auctmore commented

After applying the taxonomy.patch to the taxonomy.module this is what I get when trying to look at my site (which is in Dutch) :

Warning: array_key_exists(): The second argument should be either an array or an object in /home/wdbsiha/public_html/drupal/includes/module.inc on line 89

Warning: array_key_exists(): The second argument should be either an array or an object in /home/wdbsiha/public_html/drupal/includes/module.inc on line 89

Warning: array_key_exists(): The second argument should be either an array or an object in /home/wdbsiha/public_html/drupal/includes/module.inc on line 89

Warning: array_key_exists(): The second argument should be either an array or an object in /home/wdbsiha/public_html/drupal/includes/module.inc on line 89

Warning: array_key_exists(): The second argument should be either an array or an object in /home/wdbsiha/public_html/drupal/includes/module.inc on line 89

Warning: Cannot modify header information - headers already sent by (output started at /home/wdbsiha/public_html/drupal/includes/module.inc:89) in /home/wdbsiha/public_html/drupal/includes/bootstrap.inc on line 258

Warning: Cannot modify header information - headers already sent by (output started at /home/wdbsiha/public_html/drupal/includes/module.inc:89) in /home/wdbsiha/public_html/drupal/includes/bootstrap.inc on line 259

Warning: Cannot modify header information - headers already sent by (output started at /home/wdbsiha/public_html/drupal/includes/module.inc:89) in /home/wdbsiha/public_html/drupal/includes/bootstrap.inc on line 267

Warning: Cannot modify header information - headers already sent by (output started at /home/wdbsiha/public_html/drupal/includes/module.inc:89) in /home/wdbsiha/public_html/drupal/includes/bootstrap.inc on line 275

Has this got to do with Drupal's cache? If so, how do I avoid this?

sami_k’s picture

sami_k commented

Do this:

restore taxonomy.module
get rid of your current taxonomy_access module
get rid of the table related with the taxonomy module.

The get this version of the taxonomy_access module: http://drupal.org/files/projects/taxonomy_access-cvs.tar.gz

Install it and add the table to your database

Don't enable it right away.

Get the nysus patch, the final one and run it on the module. Alternatively you can use the module that I provided and replace your current taxonomy_access module with that.

Patch the taxonomy module with the patch in the cvs archive. Make the little change that nysus suggests. That should solve your problem.

sami_k’s picture

sami_k commented

Nysus' patch/module: http://drupal.org/node/13597

sami_k’s picture

sami_k commented

Sorry, get rid of the table related with the taxonomy_access module! not the taxonomy module!

sami_k’s picture

sami_k commented

Ignore above and do what this says: http://drupal.org/node/13902

pyromanfo’s picture

pyromanfo commented

Let me know if the latest CVS combined with nysus's instructions fixes it.

pyromanfo’s picture

pyromanfo commented
Anonymous’s picture

(not verified) commented