[MAGNOLIA-4867] Throw IAE in DelegateNodeWrapper.setWrappedNode() when node is already wrapped with this class - 4.5 Created: 25/Feb/13  Updated: 22/Apr/13  Resolved: 27/Feb/13

Status: Closed
Project: Magnolia
Component/s: None
Affects Version/s: None
Fix Version/s: 4.5.8

Type: Improvement Priority: Major
Reporter: Roman Kovařík Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Cloners
is cloned by MAGNOLIA-4873 Throw IAE in DelegateNodeWrapper.setW... Closed
causality
is causing MSHOP-67 Don't wrap nodes multiple times Closed
is causing MAGNOLIA-4916 Add possibility to allow multiple wra... Closed
dependency
is depended upon by MGNLSTK-1095 Escape values for rendering, don't es... 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)

 Description   

We don't want double wrapped nodes. For example double escaping with HtmlEscapingNodeWrapper can cause problems in FTL templates.
Throw IllegalArgumentException in DelegateNodeWrapper.setWrappedNode() when node is already wrapped with this class.



 Comments   
Comment by Roman Kovařík [ 27/Feb/13 ]

Port to master is registered under MAGNOLIA-4873.

Generated at Mon Feb 12 03:59:47 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.