I have a need to add an extra permission. delete own tags. This will allow finer access control. In our case we do not want to let users delete their tags after they have created one.

CommentFileSizeAuthor
#2 community-tags-module-836926-3.patch492 bytessyakely
#2 community-tags-pages-836926-3.patch674 bytessyakely
#1 community-tags-module-836926-2.patch22.9 KBsyakely
#1 community-tags-pages-836926-2.patch674 bytessyakely

Comments

syakely’s picture

syakely commented
StatusFileSize
new community-tags-pages-836926-2.patch674 bytes
new community-tags-module-836926-2.patch22.9 KB

I think it would only take a few changes to the .module and .pages.inc files to make this happen.
Please wait for new patch, the module patch attached is bad.
thanks for making this module.
SY.

syakely’s picture

syakely commented
StatusFileSize
new community-tags-pages-836926-3.patch674 bytes
new community-tags-module-836926-3.patch492 bytes

Try these if you are considering adding this function.
thanks
SY.

entrigan’s picture

entrigan commented

What use case would require a user to not be able to delete their own tags?

erik.ahlswede’s picture

erik.ahlswede commented

One use case could be so that tags aren't constantly changing on a node. It can be helpful if a different user relies on those tags for a search.

If a tag is incorrect, or inappropriate, an admin can remove the tag.

Erik

syakely’s picture

syakely commented

For us the client was worried that a community tag used by the community to do searches with might change too much.

Just gives another permission level the client felt they needed/wanted. May not fit into the intended use of this module.

entrigan’s picture

entrigan commented

That makes sense, I will probably add this tonight.

entrigan’s picture

entrigan commented

the patches seem incomplete (they do not apply because I think you modified an already modified file). But also even if the $delete is added there will need to be some handler in the JS file to not display the delete option.

syakely’s picture

syakely commented

Yes, I did not know some of my other co-workers hacked up this module already. Sad face here.

entrigan’s picture

entrigan commented

hmm, ok well tell your co-workers to submit patches too :)

Hopefully this feature will make it into 2.x

chaps2’s picture

chaps2 commented
Version: 6.x-1.0-beta3 » 6.x-2.x-dev
Status: Active » Fixed

This use case is covered by other features - particularly the separation of node terms and community tags via the community tags node synchronisation module. Try 2.x-dev and if you still need an extra permission - please reopen!

Status: Fixed » Closed (fixed)

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