[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: |
|
||||||||
| 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
|
||||||||
| Date of First Response: | |||||||||
| Team: | |||||||||
| Description |
|
Step to reproduce on demo-author: |
| 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. |