[BUGFIX] Support target in fluid_styled_content menus

Add condition and only render target attribute if set on page.

Resolves: #82464
Releases: master, 8.7
Change-Id: I14c561188f1e8a7389b145adf85c64918e61d461
Reviewed-on: https://review.typo3.org/54160


Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
Reviewed-by: Sascha Wilking <sascha.wilking@hmmh.de>
Tested-by: Sascha Wilking <sascha.wilking@hmmh.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuAbstract.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 					<f:if condition="{page.data.abstract}">

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuCategorizedPages.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 				</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuPages.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 				</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuRecentlyUpdated.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 				</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuRelatedPages.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 				</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuSection.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 					<f:if condition="{page.content}">
@@ -14,7 +14,7 @@
 							<f:for each="{page.content}" as="element">
 								<f:if condition="{element.data.header}">
 								<li>
-									<a href="{page.link}#c{element.data.uid}" target="{page.target}" title="{element.data.header}">
+									<a href="{page.link}#c{element.data.uid}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{element.data.header}">
 										<span>{element.data.header}</span>
 									</a>
 								</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuSectionPages.html

@@ -6,14 +6,14 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 					<f:if condition="{page.content}">
 						<ul>
 							<f:for each="{page.content}" as="element">
 								<li>
-									<a href="{page.link}#c{element.data.uid}" target="{page.target}" title="{element.data.header}">
+									<a href="{page.link}#c{element.data.uid}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{element.data.header}">
 										<span>{element.data.header}</span>
 									</a>
 								</li>

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuSitemap.html

@@ -11,7 +11,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 					<f:render section="Menu" arguments="{menu: page.children}" />

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuSitemapPages.html

@@ -11,7 +11,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 					<f:render section="Menu" arguments="{menu: page.children}" />

typo3/sysext/fluid_styled_content/Resources/Private/Templates/MenuSubpages.html

@@ -6,7 +6,7 @@
 		<ul>
 			<f:for each="{menu}" as="page">
 				<li>
-					<a href="{page.link}" target="{page.target}" title="{page.title}">
+					<a href="{page.link}"{f:if(condition: page.target, then: ' target="{page.target}"')} title="{page.title}">
 						<span>{page.title}</span>
 					</a>
 				</li>