Changes between Initial Version and Version 1 of Ticket #24555, comment 3


Ignore:
Timestamp:
Mar 31, 2015, 9:45:43 AM (10 years ago)
Author:
Preston Timmons

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #24555, comment 3

    initial v1  
    15153) Cache the template object rather than the nodelist and use that.
    1616
    17 The easiest solution is to remove internal caching and leave it to the loaders. That also fixes #23441 that's caused by `InclusionNode` setting self.nodelist at render time. The downside is a performance regression if ``InclusionNode`` is used in a for loop.
     17The easiest solution is to remove internal caching and leave it to the loaders. That also fixes #23441 that's caused by `InclusionNode` setting self.nodelist at render time. The downside is a performance regression if `InclusionNode` is used in a for loop.
    1818
    1919If internal caching is kept, it should be done in the render_context rather than on the node.
Back to Top