Needs work
Project:
Node Ownership
Version:
6.x-1.0
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
4 Jun 2011 at 21:58 UTC
Updated:
17 Mar 2012 at 11:56 UTC
Thanks for writing this module. Saved me a ton of work =)
One problem though:
I had a problem having the "Claim This" show up on pages.
Turns out in your .module file there were too many requisites to the if statement in the hook_nodeapi.
Changed on ln 200:
if ($user->uid == 0 || !$page || !$types_to_annotate[$node->type] || $node->uid==$user->uid || !user_access('access claim page')) {
to
if (!$types_to_annotate[$node->type] || $node->uid==$user->uid || !user_access('access claim page')) {
Reasons for change:
This will solve the problem of not having it show up in the content, ie the problem in issue #1162434: How does this appear in the content type?
Thanks
Comments
Comment #1
jordanaAlso I added a wrapper div so can be easily themed:
ln 205:
Don't know if it helps or not, but I find it useful =)
Comment #2
jordanaComment #3
gauravkumar87 commented