I added a few new features to this module I needed for a project of mine. They don't interfere within its normal functionality but enhace it with some new ones:

1.- New block to show the 'link operations' links, instead at the bottom of the node.
2.- New display settings to choose whether to see all the other parents, children and link operations links under each node. Default to yes.
3.- New hook_perm with a 'create child' permission. I needed this to be able to create a child node without having to have permission to edit the whole parent node.

I enclose the patch in case you like to include these in your module.

Thanks.

CommentFileSizeAuthor
relativity_2.patch4.3 KBrobertgarrigos

Comments

robertgarrigos’s picture

Assigned: Unassigned » robertgarrigos
darius’s picture

Thanks for the contribution. Regarding #2:

2.- New display settings to choose whether to see all the other parents, children and link operations links under each node. Default to yes.

Isn't this functionality redundant, since in version 5.0 we can now set the appropriate weight to 0 to not show the link (Display options)?

Darius

JohnG-1’s picture

thanks robert - I like the sound of this. I haven't applied the patch yet - I'll have to do it manually :(

#2 Darius : if I understand correctly, the current 'Display Settings' do not affect the display of 'Link Operations' (and quite rightly).

robertgarrigos’s picture

darius, it's just a way for the user to choose whether to see those link boxes or not. I'm using this module within og.module, thus every child node already appears under the group node. I don't need those link boxes appear under each child node. If not set the original functionality applies.

robertgarrigos’s picture

Status: Active » Needs review
darius’s picture

Title: new block, settings and hook_perm added » new block, settings and hook_perm patch
Status: Needs review » Fixed

Thanks for the patch. I added the new "Link operations" block in issue http://drupal.org/node/133219. One can now set the "link operations" weight to 0 to not display the box at all.

It seems that the permissions system needs a complete overhaul... We'll leave this part for the future.

Anonymous’s picture

Status: Fixed » Closed (fixed)