Adrian's form api tree property fix (http://drupal.org/node/35630) has broken the user permissions page.

The permissions were all in a ['checkboxes'] sub-array, but they are not anymore.

Should they be in a sub-array eg ['permissions'] ?

Patch changes the permissions form _execute hook for the current situation.

CommentFileSizeAuthor
user-admin-perm-execute-fix.patch621 bytesasimmonds
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community

This is my bad. In Bryght SVN, this was in but I changed back 'cos the page did not work. Now I know that it was intended for good 'cos Adrian fixed this tree thing earlier just that got somehow lost :( Again, I suspect gremlin involvement :)

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks agian.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)