[
http://jira.magnolia.info/browse/MAGNOLIA-2074?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Grégory Joseph updated MAGNOLIA-2074:
-------------------------------------
Assignee: Grégory Joseph (was: Fabrizio Giustina)
Priority: Major (was: Critical)
> IncludeTemplate Tag renders contentNodes twice
> ----------------------------------------------
>
> Key: MAGNOLIA-2074
> URL: http://jira.magnolia.info/browse/MAGNOLIA-2074
> Project: Magnolia
> Issue Type: Bug
> Components: taglibs
> Affects Versions: 3.5.4
> Reporter: Jürgen Hirt
> Assignee: Grégory Joseph
> Fix For: 3.5.5
>
>
> If the Tag "includeTemplate" has the attribute "contentNodeName" as well as
> the attribute "path",
> the content is rendered twice:
> 1) the standard template for the contentNodeName is used
> 2) then the template of the attribute "path" is used
> example:
> <cms:includeTemplate contentNodeName="headlineWithRichText"
> path="../../../paragraphs/teasers/headline_with_rich_text.jsp"/>
> In my opinion, only one template should be used:
> if there's a path attribute, take the template of the path attribute,
> otherwise take the standard template of the contentNode
> When I look into the code of the File Include.java, there's following lines
> of code:
> if (this.path!=null)
> {
> // TODO
> log.warn("You are using the deprecated path attribute of the include tag.
> Your jsp will be included for now, but you might want to update your code to
> avoid bad surprises in the future.");
> pageContext.include(this.path);
> }
> ParagraphRenderingFacade.getInstance().render(content, pageContext.getOut(),
> pageContext);
> Wouldn't it be more correct to put the last line into the else branch of that
> if clause ?
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/docs/en/editor/stayupdated.html
----------------------------------------------------------------
