[MAGNOLIA-1034] Patches to cms:newBar and cms:contentNodeIterator to support nested paragraphs Created: 08/Sep/06 Updated: 23/Jan/13 Resolved: 09/Sep/06 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | taglibs |
| Affects Version/s: | 2.1.6 |
| Fix Version/s: | 2.1.6, 3.0 RC3 |
| Type: | Improvement | Priority: | Minor |
| Reporter: | David Smith | Assignee: | Philipp Bärfuss |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 20m | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | 20m | ||
| Environment: |
N/A |
||
| 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)
|
| Date of First Response: |
| Description |
|
The tags cms:contentNodeIterator and cms:newBar do not support the notion of nested paragraphs. Both are centered on ActivePage, completely ignoring LocalContentNode. Additionally the iterator tag corrupts the state of any previous content node iterator if they are nested and/or applied recursively. The provided patch corrects both short-comings in the tags so users can nest iterators recursively if needed. |
| Comments |
| Comment by David Smith [ 08/Sep/06 ] |
|
Here's the patch as a diff file created with Tortoise SVN client. |
| Comment by Philipp Bracher [ 09/Sep/06 ] |
|
applied to 2.1 branch. will try to port to 3.0 Thanks a lot! |
| Comment by Philipp Bracher [ 09/Sep/06 ] |
|
I ported the patch to 3.0. Can someone test it (perhaps deliver a sample page which demostrates this?) |