Make options parameter optional in User#allowed_to_globally?
|Assignee:||Jean-Baptiste Barth||% Done:|
|Category:||Permissions and roles|
It's optional in
allowed_to? so it would make sense to make it optional in
Patch is attached.
#2 Updated by Jean-Baptiste Barth over 3 years ago
- Status changed from New to Closed
- Assignee set to Jean-Baptiste Barth
- Target version set to 2.6.0
Yes obviously. This method is not used anywhere in the Redmine code base anymore, but I find lots of usages in plugins on Github (https://github.com/search?o=desc&q=allowed_to_globally&s=indexed&type=Code)
On a side note Jan: I'm trying to spend some time reviewing/merging patches these days, and obviously 4 years old patches with not even a reply are ridiculous. If you want to drop me an email with things you think I can merge directly, let me know (or we can chat about that on freenode eventually, nick=salvor but my IRC client is at $WORK so I'm only behind on weekdays).