[MGNLUI-2168] MoveInsidePossibilityPredicate sometimes allows wrong operations. Created: 27/Sep/13 Updated: 10/Sep/14 Resolved: 09/Sep/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | content app |
| Affects Version/s: | None |
| Fix Version/s: | 5.1 |
| Type: | Bug | Priority: | Major |
| Reporter: | Eric Hechinger | Assignee: | Aleksandr Pchelintcev |
| 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
|
||||||||||||||||
| Date of First Response: | |||||||||||||||||
| Sprint: | 5.1 - Final | ||||||||||||||||
| Description |
|
Currently Move Action is too permissive... For example in page app it is possible to move pages under other pages without any check. MoveDialogPresenter uses that constraint, but the predicate that validates possibility of moving item inside the selected one sometimes grants permission before the constraint is used. It assumes that the destination node is root because the getParent() call for JcrNodeAdapter returns null (see |
| Comments |
| Comment by Eric Hechinger [ 27/Sep/13 ] |
|
take for example Unfortunately, the move dialog allows to do it. |
| Comment by Antti Hietala [ 08/Sep/14 ] |
|
Reopened. Check also the Move before and Move after actions. They allow an editor to move a page to a location where creating the page would not be allowed. |