Unless you are on uid 1, any role can't manage simple acces.
Below code on function simple_access_permission

    'Administer simple access' => array(
      'title' => t('Administer Simple Access'),
      'description' => t('Allow access to administration pages for simple access.'),
    ),

Should be

    'manage simple access' => array(
      'title' => t('Administer Simple Access'),
      'description' => t('Allow access to administration pages for simple access.'),
    ),
CommentFileSizeAuthor
#1 simple_access-n887108.patch477 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-2.0-alpha1 » 7.x-2.0-beta1
Status: Active » Needs review
StatusFileSize
new simple_access-n887108.patch477 bytes

This problem still exists in beta1, so here's a patch.

emok’s picture

emok commented

The patch works. Would be nice if it could be committed, since issue remains in -dev at this date. (Don't know if I'm allowed to set status to RTBC.) I did a string search in the module's files and found no other reference to the incorrect permission name.

Note, that after applying the patch you need to rebuild the menu (e.g. submit the Modules page without any change) and give your administrator(s) the new permission (go to /admin/people/permissions, check "Administer Simple Access" and save). They should then see some links from admin/config/simple-access/.

rt_davies’s picture

rt_davies commented

I can also confirm that the patch works. @DamienMcKenna: thanks for posting it!

Note for others who find this: I had to uninstall (not just disable) and reinstall the module for the patch to take effect.

flocki’s picture

flocki commented

The patch works for me as well. For me, disabling and re-enabling the module did the trick to make the patch work.

joelstein’s picture

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

Works for me!

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

This must be committed and rolled into a release. Over a year is just silly.

dianacastillo’s picture

dianacastillo commented

I agree I was looking all over for how to administer simple access, and following these steps solved it for me! Why isnt this in the last release?

gordon’s picture

gordon commented
Status: Reviewed & tested by the community » Fixed

Fixed in dev.

dianacastillo’s picture

dianacastillo commented
Status: Fixed » Needs work

When you say fixed in dev what does that mean? I dont see the fix in any of the releases listed on this page http://drupal.org/project/simple_access

gordon’s picture

gordon commented
Status: Needs work » Fixed

This has been committed to the dev version which is packed up once every day. You can check this out via the git repository.

I will release a new version once I have done some testing and confirmed that the bugs have been fixed.

dianacastillo’s picture

dianacastillo commented

I think that many drupal users don't connect with git and they don't know how to check that out. At least some information about that should be put on the page where people read about this module, it would save them a lot of time.

gordon’s picture

gordon commented

As I said the the dev version http://drupal.org/node/1083258 gets repacked once every 24 hours (after a commit). I missed the window yesterday and should get repacked today.

dianacastillo’s picture

dianacastillo commented

oh ! okay , thanks a lot for explaining that.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 1602501 on 7.x-2.x, 8.x-3.x by gordon: 
    Issue #887108 by DamienMcKenna | sylv3st3r: Fixed Wrong permission for...
laceysanderson’s picture

laceysanderson commented
Issue summary: View changes

You really need to release a new version or at least tell users on your main page to use the dev version. The fact that normal users can't currently configure this module even though the fix has been committed for over a year is.... *shakes head*

gordon’s picture

gordon commented

FYI the only change since the last release is only cosmetic making sure all the code is standards compilant. So I was not interesting in creating a new release.

I am working on the D8 version now and want to put my energy into that.

vulfox’s picture

vulfox commented

Maybe add a notice to the module description on the project page that there is currently some problems with the 'stable' release?

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

For those of you that are seeking faster response times in fixing issues, you might wanna give Backdrop CMS a go. This module is already ported and the fix for this issue is included in its latest stable release: https://backdropcms.org/project/simple-access