[MGNLDEMO-139] Dialog broken for some travel-demo page templates since MTK Created: 20/Jan/16  Updated: 15/Feb/16  Resolved: 29/Jan/16

Status: Closed
Project: Magnolia Demo Projects
Component/s: None
Affects Version/s: 0.9
Fix Version/s: 0.10

Type: Bug Priority: Major
Reporter: Edwin Guilbert Assignee: AntonĂ­n Juran
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 3h 5m
Original Estimate: Not Specified

Attachments: Text File exception.log    
Issue Links:
Cloners
is cloned by MGNLDEMO-140 Wrong dialog property for tourSportst... Closed
supersession
is superseded by MTE-79 Clean up MTE Closed
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: Kromeriz 28
Story Points: 2

 Description   

NoSuchDefinitionException thrown when trying to open the page properties dialog for the following page templates:

  • searchResultPage
  • pur
  • aboutDemo

All of these templates are defining the dialog like:

dialog: mtk:pages/pageProperties

If the definition is reverted to the old mte value, it works again:

dialog: mte:pages/pageProperties


 Comments   
Comment by Edwin Guilbert [ 20/Jan/16 ]

Exception thrown

Comment by AntonĂ­n Juran [ 27/Jan/16 ]

czimmermann Is the correct solution for setting dialog using rewriting mtk to mte in template definition of the page as described above? Wouldn't be a better solution to move the dialog configuration from mte to mtk?

Comment by Christopher Zimmermann [ 28/Jan/16 ]

I agree with you Antonin that we should eventually move it. But for now the priority is to fix the public demo as soon as possible - and I think this is the cleanest way to do that, only having to update the demo modules.
Also since the MTK now has only component templates - i'd rather wait till we move all the page templates into the mtk as well.

Generated at Mon Feb 12 05:16:42 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.