Hi,
Thanks for this module, it does (almost) everything I need.
Here is a small patch to permit or deny roles to clone nodes on a per-content-type basis - that is, you can allow members of a role to clone one or more content types but not all.
The change is simple, and should be backwards compatible.

CommentFileSizeAuthor
#3 node-clone-per-content-type-perms.patch2.48 KBjames.cartledge
per-content-type-perms.patch1.15 KBjames.cartledge

Comments

pwolanin’s picture

pwolanin commented

hmm, while this makes sense - will also lead to a proliferation of permissions on the permissions page - guess that's ok, if ugly.

james.cartledge’s picture

james.cartledge commented

pwolanin - that's true. For my use case this functionality is needed, so I just have to live with that.
Would it be preferable if there was a setting for this, defaulting to the current configuration, or is that too much noise on the settings screen?

james.cartledge’s picture

james.cartledge commented
StatusFileSize
new node-clone-per-content-type-perms.patch2.48 KB

Not sure if this is better - I've added a checkbox in the settings screen to make the change optional.

csc4’s picture

csc4 commented

I'd like something similar to this, I don't want to let users edit a node of a particular type, but I do want to let them clone them and I think I'd need this patch to be able to do that?

pwolanin’s picture

pwolanin commented

@csc4 - right now clone is a global permission, so users can essentially clone any node if they can view it and they can create a node of that type with that format. This would lock things down more - though it's mostly a usability/workflow question I think, since users could still manually copy/paste content to create content like existing content.

csc4’s picture

csc4 commented

Thanks for the clarification - I hadn't realised that the clone was only 

if they can view it _and_ they can create a node of that type with that format

seems obvious now you say it! and that does help a bit.

I do still like the idea of tying the workflow - but perhaps that's the key, maybe this should be in workflow rather than the base module, then those wanting the simple case can keep it and we inveterate complicators can complcate things.

Clone is hugely superior to cut and paste - one of my core modules I always install! - thanks for your work on it.

korzh-nick’s picture

korzh-nick commented

Is it possible to use it in drupal 7?

pwolanin’s picture

pwolanin commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)