In the current version of openfed, the role 'Builder' does not have the 'administer site configuration' permission, this should be possible otherwise the site's name cannot be changed (which a Builder certainly should be able to do)

I think we might also want to give this to the "Configurator" role

Comments

bart.hanssens’s picture

Assigned: bart.hanssens » stefan.r
Issue tags: +release1.3
stefan.r’s picture

The administer site configuration role is used in Drupal 7 as a superuser permission. It is used in contrib modules to identify administrators and in core it allows one access to not only the Site information page but to many other config pages.

Upon discussion with bdeclercq the only requirement for the Configurator and Builder roles is to access the Site information page. As such we could instead create a new permission "edit site information" and use a hook_menu_alter to give these users access to the Site information page.

bart.hanssens, sound good to you?

stefan.r’s picture

Title: Builder/Configurator roles should be able to 'administer site configuration' » Builder/Configurator roles should be able to edit Site Information
stefan.r’s picture

Title: Builder/Configurator roles should be able to edit Site Information » New module: Custom Permissions
stefan.r’s picture

Upon discussion with bart.hanssens we will handle this using the Custom Permissions module

stefan.r’s picture

Title: New module: Custom Permissions » New module: Custom Permissions (Builder/Configurator roles should be able to edit Site Information)
stefan.r’s picture

Status: Active » Needs review

This has been set up in the 1.3 branch, needs further testing.

stefan.r’s picture

Status: Needs review » Closed (won't fix)

Reverted changes. Upon further discussion builder and configurator will not be able to access Site Information and instead they will request for Fedict to do this in a ticket.

  • Commit 27679d7 on 7.x-1.x, 7.x-1.7 by stefan.r:
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...
  • Commit e8a5ef1 on 7.x-1.x, 7.x-1.7 by stefan.r:
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...

  • Commit 27679d7 on 7.x-1.x, 7.x-1.8 by stefan.r:
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...
  • Commit e8a5ef1 on 7.x-1.x, 7.x-1.8 by stefan.r:
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...

  • stefan.r committed 27679d7 on 1.8
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...
  • stefan.r committed e8a5ef1 on 1.8
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...

  • stefan.r committed 27679d7 on
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...
  • stefan.r committed e8a5ef1 on
    Issue #2127881 by bdeclerc: New module: Custom Permissions (Builder/...