Very useful module, and thanks for contributing it! Alas, I must report a bug today, but I'm contributing a patch that fixes it also.

(Note: I'm not 100% sure of the version. I checked out from CVS using branch tag DRUPAL-5 as of 2007-04-06.)

This may be related to issue #79365, because it occurs when there are spaces in the category name, but I'm reporting it as a separate bug because I'm getting a different message than what was reported in #79365.

The error is "Only variables may be passed by reference" in line 431 of article.module.

The problem seems to be the nested call of current(module_invoke(.....)) at this line. I changed the code so that the module_invoke() call puts its results into a scratch variable, which is then passed to current(), and this removes the error. The page appears to be working correctly now, even with the spaces in the category name. This may be a fix for #76365 as well, but I haven't tested that.

Please feel free to contact me if you need me to test something else in my sandbox site.

Kind regards,
Syscrusher

CommentFileSizeAuthor
article.module_0.patch610 bytessyscrusher

Comments

msameer’s picture

Version: 5.x-2.2 » 5.x-1.x-dev
Assigned: Unassigned » msameer
Status: Active » Closed (won't fix)

The branch you used is obsolete. Sorry for your time. Please use the latest stable release.

We are now using the term IDs instead of the term names.