[MAGNOLIA-5149] MgnlUser#inGroup(String) is very likely to always return false Created: 03/Jul/13 Updated: 04/Jul/13 Resolved: 03/Jul/13 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | None |
| Affects Version/s: | 4.5.9, 5.0 |
| Fix Version/s: | 4.5.10, 5.0.1 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Daniel Lipp | Assignee: | Daniel Lipp |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Description |
|
MgnlUser#inGroup(String) is very shake. When debugging I found that in many cases it'll return false where as it should actually return true. Looks like the reason is because values of various properties are collected and later interpreted as uuids. In case they actually are we might be lucky - else we get RepositoryExceptions that are kind of swallowed and we simply return false. |