I'm currently bugfixing a Drupal 5 site. There, author_pane and de_stemmer are installed. As author_pane backported the Drupal6 preprocess functionality, that is searching for hook_preprocess(). As de_stemmer uses de_stemmer_preprocess(), this causes author_pane calling this function. But by this, it "destroys" the given preprocess-$variables.

The attached patch solved this problem, bysimply renaming de_stemmer_preprocess() to _de_stemmer_preprocess().

CommentFileSizeAuthor
de_stemmer.patch894 bytesderhasi

Comments

schildi’s picture

schildi commented
Assigned: Unassigned » schildi

I'm a bit surprised. Looking for "hook_preprocess" I found that this hook is available for D7 only. How can it be part of a back port from D6 to D5 ?
On the other side, all functions used internally only should be prefixed by an underscore character. I will check this when upgrading the stemmer to D7 next time.

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

template_preprocess is what was backported. Basically, I backported D6 style theming to keep maintaining two branches at once sane. Bit of deja vu here... I'm pretty sure I had to work around this problem with the porter_stemmer module. Ended up putting an exception in or something... Not real sure. It's been a long time since I've looked at 5.x.

Michelle

schildi’s picture

schildi commented
Status: Needs review » Closed (fixed)