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:

  1. $user->uid == 0 is not needed because you have user_access('access claim page') that already checks the permissions, which is fine and works. (if i let anonymous users have access, this prevents them)
  2. !page is not needed because you have your $types_to_annotate function that checks which content types are allowed. (if my content type is not a page, ie a custom content type, it won't show up.)

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

jordana’s picture

Also I added a wrapper div so can be easily themed:
ln 205:

'#value' => "<div class='claimthis'>".l('Claim This '. $node->type .'.', 'node/' . $node->nid . '/claim')."</div>",

Don't know if it helps or not, but I find it useful =)

jordana’s picture

Status: Active » Needs review
gauravkumar87’s picture

Assigned: Unassigned » gauravkumar87
Status: Needs review » Needs work