[MGNLCDEP-74] Warn on deletion of page with modified children Created: 04/Nov/14 Updated: 18/Dec/14 Resolved: 18/Dec/14 |
|
| Status: | Closed |
| Project: | Content Dependencies |
| Component/s: | None |
| Affects Version/s: | 1.1.3 |
| Fix Version/s: | 1.1.4 |
| Type: | Bug | Priority: | Major |
| Reporter: | Roman Kovařík | Assignee: | Christopher Zimmermann |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | support | ||
| 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
|
||||||||
| Release notes required: |
Yes
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
Respect changes done by |
| Comments |
| Comment by Roman Kovařík [ 05/Nov/14 ] |
|
[09:54] Federico Grilli: |
| Comment by Roman Kovařík [ 05/Nov/14 ] |
|
| Comment by Christopher Zimmermann [ 18/Dec/14 ] |
|
Reopen. When you attempt to delete a node in another tree besides the Pages app, such as a node in the Templating / Sites "app", you get a big mess of html instead of the node name. |
| Comment by Christopher Zimmermann [ 18/Dec/14 ] |
|
Added check to deleteNode in contentDependencies.js script so that it only calls checkIfChildrenModified() if the repository is website - since this ticket is about Pages & since checkIfChildrenModified is only implemented in the WebsiteTreeHandler. (The problem is that trees with other handlers did not have the checkIfChildrenModified command available, and so AdminTreeMVCHandler SHOW command was run by default which was just returning fragments of the tree!) |