Seems the permission "manage MERCI" has been changed to "administer MERCI", but "manage MERCI" still being used in code. For example line 91 in merci.module
'access arguments' => array('manage MERCI'),
Because of this no one except for user 1 will be able to access the Current Inventory page at admin/merci/manage/current_inventory .
Or am I missing something here?
Using Drupal 6.24 with MERCI dev from februari 8.

Comments

darrick’s picture

darrick commented
Status: Active » Needs review

I've commited a fix here: http://drupalcode.org/project/merci.git/commit/cb781c7

hurricane66’s picture

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

Patch in #1 works. Perhaps a separate permission for the inventory parts should be added in a future release, i.e. "manage inventory" or something.

darrick’s picture

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

Thanks for testing. I will look into making the permissions more practical in the next major version upgrade. I have no plans to continue any major development on the D6 branch beyond bug fixes. For D7 the plan is get a 2.x stable release out before work begins 7.x-3.x.

Status: Fixed » Closed (fixed)

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