It seems currently a user needs "administer site configuration" permission to edit the option sets.

We needed to introduce a separate permission for a demo site to provide non-admin users to edit the option sets. This might be needed for other usecases too..

function galleria_demo_permission() {
  return array(
    'administer galleria configuration' => array(
      'title' => t('Administer galleria configuration'),
      'description' => t('Bla'),
    ),
  );
}

function galleria_demo_menu_alter(&$items) {
  //debug($items);
  $items['admin/config/media/galleria']['access arguments'] = array('administer galleria configuration');
  $items['admin/config/media/galleria/add']['access arguments'] = array('administer galleria configuration');
  $items['admin/config/media/galleria/edit/%galleria_optionset']['access arguments'] = array('administer galleria configuration');
  $items['admin/config/media/galleria/delete/%galleria_optionset']['access arguments'] = array('administer galleria configuration');
}

Shall we add a separate permission?

Comments

kroimon’s picture

This has already been done in the latest -dev build (more specifically, in commit a1dd48ac).

/**
 * Implements hook_permission().
 */
function galleria_permission(){
  return array(
    'administer galleria' => array(
      'title' => t('Administer the Galleria module'),
    ),
  );
}

/**
 * Implements hook_menu().
 */
function galleria_menu() {
...
    'access arguments' => array('administer galleria'),
...
}
miro_dietiker’s picture

Status: Active » Closed (works as designed)

Thanks, that's great! I didn't check the dev..? hmmm ;-) sorry