[MAGNOLIA-1470] Paragraphs can't reuse dialogs Created: 19/Apr/07 Updated: 28/Nov/13 Resolved: 28/Nov/13 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | admininterface, core |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major |
| Reporter: | Magnolia International | Assignee: | Philipp Bärfuss |
| Resolution: | Outdated | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| 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: |
| Description |
|
If one creates a paragraph with a dialog property refering to an existing dialog, it just does not work. Upon dialog selection, Magnolia will complain that a dialog with name a <paragraph-name> does not exist. It just ignores the dialog property of the paragraph. ParagraphSelectDialog can easily be fixed for this, but then the edit functionality as the exact same issue. (See attached patch) This is also partly related to the fact that upon dialog registering, Magnolia checks if the dialog is used by a paragraph, and if so re-registers the dialog with the paragraph name (DialogHandlerManager.registerAsParagraphDialog()) |
| Comments |
| Comment by Philipp Bracher [ 20/Apr/07 ] |
|
This dialog property is not completely ignored, but used to register the referenced dialog under the name of the paragraph. This is ugly and needs changing but if you reference an other dialog (not having the same name as the paragraph) it works fine. |