The new build seems to give everyone a "Quota by role" link in the navigation, and it gives everyone access/permissions to the new "Administer module" page, even if they don't have permission in Access control.

Thanks

Comments

introfini’s picture

introfini commented

Hi,

That's a bug. You have to change the line #53 to

'access' => user_access('administer quota by role'),

introfini

tj2653’s picture

tj2653 commented

That seemed to fix the access issue, but I tried to uncheck the "Yes, override the default behaviour, I want to redirect the user to my own error page" checkbox with my admin user, but it won't save the new setting now.

lazyboy’s picture

lazyboy commented

tj2653

You can download a new release for the module (http://drupal.org/project/quota_by_role), that where I fixed that problem (the access control). Make sure to disable the module, then re-enable (no need to uninstall).

Regards
lazyboy

lazyboy’s picture

lazyboy commented

Thanks introfini for the ad-hoc support :) Appreciate it.

tj2653’s picture

tj2653 commented

Thanks lazyboy...any ideas about the problem where the "Yes, override the default behaviour, I want to redirect the user to my own error page" stays checked even after I unchecked it and tried Saving that setting? I tried disabling the module, and completely uninstalling it, but the appears that that setting doesn't delete and stays checked. I believe it was originally set to unchecked, but ever since I checked it, I can't get back.

lazyboy’s picture

lazyboy commented

Hi tj2653

Yeah, I've found the problem. I forgot to add code to remove the variable when you untick the checkbox... Committed it to CVS, and it's already available for download.

Regards
lazyboy

tj2653’s picture

tj2653 commented

Sorry to be a pain, but unless I'm going crazy, 5.x-7.2 seems to be identical to 5.x-7.1.

lazyboy’s picture

lazyboy commented
Assigned: Unassigned » lazyboy
Status: Active » Fixed

Hello All

I've finally managed to commit the code to cvs, and have the right version appear on the front page. Download latest version and test :)

Regards
lazyboy

tj2653’s picture

tj2653 commented

The problem with not being able to uncheck the "Yes, override the default behaviour..." box looks like it's fixed in the latest build. Thanks :)

lazyboy’s picture

lazyboy commented
Status: Fixed » Closed (fixed)

Hi tj2653

Sweet! Then I'm going to close the issue.

Regards
lazyboy