Closed (fixed)
Project:
Permissions API
Version:
6.x-2.5
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
18 Feb 2010 at 15:38 UTC
Updated:
23 Mar 2010 at 14:40 UTC
Unwittingly I used a string as the role I wanted to grant, instead of an array and got a foreach error. Some suggestions:
* Check for non-arrays if you are going to be doing a foreach
* Support a string
* Provide a default type in the function definition
* Better documentation
Thanks for the great module.
Comments
Comment #1
ebeyrent commentedUgh, that's embarrassing. You're absolutely correct - there needs to be better error checking.
Thanks for the feedback!
Comment #2
ebeyrent commentedFixed in the 2.6 release