Reviewed & tested by the community
Project:
Menu HTML
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
8 Jun 2010 at 20:23 UTC
Updated:
10 Nov 2019 at 23:06 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
gmclelland commentedBump, any thoughts on this. Thanks
Comment #2
alan d. commentedDouble this. This could be considered a security issue!
Comment #3
mzwyssig commentedSubscribing. This would be very useful.
Comment #4
mzwyssig commentedAttached is a patch that checks if user has "administer nodes" permissions.
Comment #5
ClaireC commentedThe patch works for me ! Thanks.
Comment #6
gagarine commentedI think it would be great to commit this...
Comment #7
gagarine commentedComment #8
gagarine commentedI'm going to make a permission like "Use html for menu link". If the user don't have the permission but can edit the menu link the menu link should go back to no HTML. The other possibility is than when a menu link use HTML user without the permission can't edit them. Otherwise it's a security problem...
To be clear, the patch #4 doesn't work. An admin can change the link to "HTML" and a user without the permission can still edit the content and add bad stuff...
Comment #9
webadpro commentedGreat point guys.
Would you guys say to simply not allow to edit the element completely or simply deny access to edit the title, but the user could always change the path, etc.?
Comment #10
gagarine commentedI'm for let the user edit what he can and just disable the title field.
Comment #11
gagarine commentedComment #12
drupov commentedHas this patch been commited?
Comment #13
robinsonsarah01 commentedAttached is a patch that adds a "use html for menu titles" permission to this module. If a user can administer menus but isn't allowed to use HTML in those menus, the checkbox for "Allow HTML" will not appear; if HTML is enabled, the link title will be disabled and the user will not be able to edit it.
Comment #14
gagarine commentedThis patch look good thanks! Someone was able to test it?
Comment #15
shortspoken commentedI just tested the patch and it works as advertised. Thank you @robinsonsarah01! Please commit.
Comment #16
irankhosravi commentedhello how to logo in center of menu
Comment #17
irankhosravi commentedhelp
Comment #18
alan d. commented@mehrwebstay
You are very unlikely to get help by jumping into an unrelated thread, especially a project issue on a completely unrelated topic (aka this is to prevent users from inserting HTML).
Try https://www.drupal.org/forum/22 or http://drupal.stackexchange.com/ and be a bit more verbose about what you have tried and what isn't working. :)
Comment #19
irankhosravi commentedComment #20
markabur commentedLooks good, thank you! RTBC
Comment #21
chris matthews commentedWould it be possible to commit the patch in #13 to 7.x-1.x-dev and tag a 7.x-1.1 release?
Comment #22
chris matthews commented@maintainers, can the patch in #13 be committed and pushed to a 7.x-1.1 release?