User#allowed_to? should return true or false
|Assignee:||Jean-Philippe Lang||% Done:|
The attached patch fixes a minor bug in User.allowed_to?
Specifically, that method should return a boolean, but it actually sometimes returns a Role instead. This is because .detect is used when really .any? should be used.
I am not aware of this causing any serious issues, but I noticed this during debugging and was greatly confused when allowed_to? suddenly returned a Role ;).
#2 Updated by Jean-Philippe Lang almost 7 years ago
- Subject changed from Replace incorrect uses of .detect by .any? to User#allowed_to? should return true or false
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Target version set to 2.2.0
Committed in r10614. Another change was required so that
User#allowed_to? return true or false and tests were updated.