I think this is because it's using hook_nodeapi('load') instead of hook_nodeapi('view') - taxonomy/term/323 on the development site for an example. Switching to $op == 'view' might help with body vs. teaser option as well.

CommentFileSizeAuthor
#2 restricted_content_alter-448404.patch925 bytescatch

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Seems like a good change. I'm trying to remember the reason why I used hook_nodeapi('load')... I think it was so I could adjust the variables in the preprocess function. I'll check on this.

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new restricted_content_alter-448404.patch925 bytes

Usering 'alter' actually seems OK here since we're replacing stuff added during view. Here's a patch for that. Could maybe do it by setting a low module weight but other modules might do the same thing.

sbart76’s picture

sbart76 commented

This patch works for me as well. I am using node_images module and experienced hiding only text in the teaser and not the images. Thanks!

giorgosk’s picture

giorgosk
Location Chios, Greece 🇬🇷
commented
Status: Needs review » Reviewed & tested by the community

Patch works for me !!

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Fixed in CVS. Thanks!

Status: Fixed » Closed (fixed)

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