[MGNLRES-50] Aggregation of "Processed CSS/JS" leads to encoded content Created: 20/Sep/12  Updated: 29/Mar/22  Resolved: 26/Sep/12

Status: Closed
Project: Magnolia Resources Module
Component/s: None
Affects Version/s: 1.5
Fix Version/s: 1.5.4

Type: Bug Priority: Major
Reporter: Milan Divilek Assignee: Jaroslav Simak
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: XML File sample.xml    
Issue Links:
relation
is related to MAGNOLIA-4541 Aggregation of "Processed CSS/JS" lea... 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:
Team: Nucleus

 Description   

Step to reproduce on demo-author:
1. import attached sample.xml file to Templating kit-resources-templating-kit
2. then go to http://demoauthor.magnolia-cms.com/resources/templating-kit/all.js
3. see that <br/> tags in code



 Comments   
Comment by Jan Haderka [ 26/Sep/12 ]

NodeUtil.isWrappedWith() and NodeUtil.deepUnwrap() methods perform the same - traverse the wrapper hierarchy recursively and in first case return true on hit and in second perform the actual unwrapping. For that reason it's not necessary to traverse the hierarchy twice and the isWrapppedWith check should be skipped to make code perform better.

Generated at Mon Feb 12 06:46:53 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.