Rendering improvements
(MAGNOLIA-5895)
|
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | rendering |
| Affects Version/s: | None |
| Fix Version/s: | 5.4 |
| Type: | Sub-task | Priority: | Neutral |
| 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: |
|
||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||
| Description |
|
Current rendering listeners are quite limited as they are not able influence rendering engine in any way. We need to be able return some sort of status from listeners so rendering engine could react on that.
|
| Comments |
| Comment by Roman Kovařík [ 21/Aug/14 ] |
|
Resolved on branch feature/snippet-caching. |
| Comment by Roman Kovařík [ 10/Nov/14 ] |
|
Squashed on the parent ticket. |
| Comment by Magnolia International [ 29/Dec/14 ] |
|
While looking at info.magnolia.rendering.renderer.registry.RendererRegistry, I see the List<Class<RenderingListener>> listeners is still there but unused. You probably just forgot about it ? Similar, in info.magnolia.rendering.renderer.registry.ConfiguredRendererManager, I see a bunch of constants that now seem unused. Since they were public, they should probably be deprecated instead of just removed (? your call, I'm not sure what they were used for) There are maybe other such details to cleanup ? |
| Comment by Roman Kovařík [ 05/Jan/15 ] |
|
Changes related to this ticket: https://git.magnolia-cms.com/gitweb/?p=magnolia_main.git;a=commitdiff;h=1946233c18ece59846403dc9ba3f0a65dc300479 |