Order relevant user/role/access forms by Role Weight

Awesome, thank you. I think you're right - shall we assume that if one has enabled this module, then re-ordering the Access Control page is desired, without option? We don't have an admin/settings/role_weights page, so adding one for this does seem a bit daft.

Will check the patch out - thanks again.

Patch works a treat - will commit this to 5.x unless I hear any complaints re: no settings page?

Thanks again.

Just a thought, but if we have no explicit setting for this re-ordering on user access or user roles forms, then I'd like the re-ordering to be done across the board - ie we'd need a solution for ordering the user roles form (#114037: Order admin_user_roles by weight?) too.

Merging #114037: Order admin_user_roles by weight? here, as per comment #3. Merged patch soon.

Not sure I like the weight actually being shown on the admin_user_roles form, but this merged patch should do what we want.

Better patch - no weight is shown on the admin_user_roles form now but its still sorted. Also added a system variable 'role_weights_reorder_forms' which allows for the sorting on admin_user_roles form and the user_admin_perm form to be toggled.

I didn't want to add an entire settings page just for this system variable - I guessed that if the user was able to manage roles and permissions, then adding a setting to the $conf variable to ENABLE sorting of these forms by role weight should suffice?

So, if you test this patch, add:

  # role weights re-order admin forms setting
  $conf = array(
    'role_weights_reorder_forms' => TRUE,
  );

to the $conf array in your settings.php file.

Will be committing this and some other changes very soon for the next 5.x-1.x release - please shout now if you would like a settings page for this!

Minor change - if no weight is set (NULL) we assume a weight of zero. Without this, when new roles were added on admin_user_roles, no weight is set and so the sorting fails - the new role appeared at the top of the list.

Committed to 5.x-1.x-dev and HEAD.