I get tired of checking all the permissions in admin/user/access, so I added a javascript checkbox in the column header to fix it. I know this isn't done exactly correctly, but I wasn't sure how to do it. Here are the changes:

  • In user.module around line 2013, when it goes through the user roles, it returns a header cell with the class col-select. The function also calls theme_column_select_header_cell (which doesn't actually return anything, because otherwise the settings get set the same number of times as there are roles. I don't know how to fix this).
  • theme.inc around line 1170 added theme_column_select_header_cell
  • added column_select.js to misc. there's probably a more elegant way to do this but I can't figure it out.

If anyone would like to help get this patch right, I would appreciate it.

CommentFileSizeAuthor
colselect.patch3.82 KBbennybobw

Comments

pancho’s picture

pancho
Location UTC+2 🇪🇺 EU
commented
Version: 6.x-dev » 7.x-dev
Category: task » feature

This is a feature request, so D7.

Jaza’s picture

Jaza commented
Version: 7.x-dev » 8.x-dev
Status: Needs work » Closed (fixed)

Closing this issue for 2 reasons:

1. It's been inactive for 2 years
2. I personally think adding this would reduce usability: the good of having convenience for selecting all permissions in a given role, is outweighed by the bad of this being a security risk (too easy for users to assign all permissions to a given role) and of adding extra complexity to the UI; also, in D7 we have the admin role, which has all permissions by default, thus making this even less necessary

If anyone wants to pick up on this, feel free to re-open in 8.x-dev.