Project

General

Profile

Actions

Patch #12078

closed

User#allowed_to? should return true or false

Added by Anonymous over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Low
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

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 ;).


Files

0001-Replace-incorrect-.detect-call-by-.any.patch (1.13 KB) 0001-Replace-incorrect-.detect-call-by-.any.patch Patch changing .detect to .any? Anonymous, 2012-10-11 16:31
Actions #1

Updated by Etienne Massip over 11 years ago

  • Category set to Code cleanup/refactoring
Actions #2

Updated by Jean-Philippe Lang over 11 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.

Actions

Also available in: Atom PDF