We need to hide the machine name field form editors. Right now the machine name field is visible and editable for everyone who has permissions to edit the node.

CommentFileSizeAuthor
#4 defaultcontent_1160250_4.drushmake.patch951 bytesjamsilver
#2 defaultcontent-n1160250.patch959 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review

How's about this? It adds the 'edit node machine name' permission and only shows the form field if you have that permission.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new defaultcontent-n1160250.patch959 bytes

Oh wait, the patch itself would be useful =)

smk-ka’s picture

smk-ka commented
Status: Needs review » Reviewed & tested by the community

Looks good!

jamsilver’s picture

jamsilver commented
StatusFileSize
new defaultcontent_1160250_4.drushmake.patch951 bytes

Just for my own drush make purposes more than anything - here is the same again without the prefix. Patch looks good!

mrfelton’s picture

mrfelton commented

Looks good, works for me. Thanks.

Mike Dodd’s picture

Mike Dodd commented

Works good for me, thanks

czigor’s picture

czigor commented

Works for me too.

Anonymous’s picture

Anonymous (not verified) commented

Patched my defaultcontent.module and tested it. The patch works for me and should be committed.

goron’s picture

goron commented

I'm using this patch now. Works well.

pcho’s picture

pcho commented
Assigned: Unassigned » pcho
Status: Reviewed & tested by the community » Fixed

Applied DamienMckenna's patch after further review.

Status: Fixed » Closed (fixed)

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