By wrapping the form in an if() instead of using #access, it's much harder for other modules to alter the form visibility. The upcoming patches change the form to use #access, and the untested D7 patch fixes a bug in calling user_access() without passing in the current account.

Comments

deviantintegral’s picture

Status: Active » Needs review
StatusFileSize
new3.37 KB
new2.5 KB

Here are two patches ready for git am.

berdir’s picture

Looks good, minor suggestion about the commit message, if you're using format-patch anyway. Start the commit message with "Issue #...." instead of "#" because the latter will be recognised as a comment when creating the commit message in an editor. Also, you can also directly add your name to it, for example: "Issue #111685 by deviantintegral: ...".

Then I don't need to edit the commit message at all :)

deviantintegral’s picture

I usually use --cleanup=whitespace, or -m '#123: my awesome message.'

Were you editing the commits, and should I reroll with any changes? If you were planning on committing them as-is you shouldn't need to edit the commit message.

berdir’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev

Commited the 7.x, updated commit message as described above.

berdir’s picture

mrf’s picture

Issue tags: +6.x-1.1

Looking at issues to include in 6.x-1.1

mrf’s picture

Status: Needs review » Fixed

Committed to 6.x-1.x-dev

Status: Fixed » Closed (fixed)
Issue tags: -6.x-1.1

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