Bug #48214
Moving of nodes across levels should not show the same node twice
| Status: | Resolved | Start date: | 2013-05-15 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Sebastian Kurfuerst | % Done: | 100% |
|
| Category: | - | |||
| Target version: | Base Distribution - 1.0 beta 1 |
Description
If having a node structure in live workspace, and MOVING a node into another
parent node, this is copied into your personal workspace.
In neos, the moved node is then shown twice: First from the user workspace (correct), and from the live workspace (wrong!).
It only breaks for fetching collections of nodes, but that's what we do in Neos all the time.
Associated revisions
[BUGFIX] Fix moving of nodes across workspace levels
Adds a check for moved nodes in your workspace while fetching
child nodes and also introduces dql to make this check faster.
Also implements test cases which show this behavior.
Change-Id: I8700ce155cf929eddb5a046d6362209f796e6b81
Fixes: #48214
Reviewed-on: https://review.typo3.org/20782
Reviewed-by: Christian Mueller
Tested-by: Christian Mueller
[BUGFIX] fix regression introduced with moving-nodes-fix
Related: #48214
Change-Id: I7aeabde61ea48c3be55d5903cfc56258d72f5032
Reviewed-on: https://review.typo3.org/24189
Reviewed-by: Rens Admiraal
Reviewed-by: Aske Ertmann
Reviewed-by: Sebastian Kurfuerst
Tested-by: Sebastian Kurfuerst
History
#1 Updated by Gerrit Code Review about 2 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#2 Updated by Gerrit Code Review about 2 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#3 Updated by Gerrit Code Review about 2 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#4 Updated by Gerrit Code Review about 2 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#5 Updated by Gerrit Code Review about 2 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#6 Updated by Gerrit Code Review about 2 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#7 Updated by Gerrit Code Review about 2 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#8 Updated by Gerrit Code Review about 2 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#9 Updated by Gerrit Code Review about 2 years ago
Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#10 Updated by Aske Ertmann about 2 years ago
- Status changed from Accepted to Under Review
- Target version set to 1.0 beta 1
#11 Updated by Gerrit Code Review almost 2 years ago
Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#12 Updated by Gerrit Code Review almost 2 years ago
Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#13 Updated by Gerrit Code Review almost 2 years ago
Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#14 Updated by Gerrit Code Review almost 2 years ago
Patch set 13 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#15 Updated by Gerrit Code Review almost 2 years ago
Patch set 14 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#16 Updated by Gerrit Code Review almost 2 years ago
Patch set 15 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20782
#17 Updated by Sebastian Kurfuerst almost 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 4bf537ef1aa8b7538051abb4d0aa8210a23eaac3.