Bug #98601
closed
typolink with parameter "/"
0%
Description
the following viewhelper:
<f:link.typolink parameter="/" ></f:link.typolink>
results in an empty link <a class="">
This affects especially the language menus because the language data handler delivers "link: /" for the homepage in the default language.
In constructions like
<f:link.typolink parameter="{languagemenu.link}" ></f:link.typolink>
this results in empty links for the homepage in the default language.
Reason is a modification in typo3/systext/frontend/Classes/Service/TypoLinkCodecService.php. This line from Typo3 vs. 10:
$value = str_replace(['\\', '"'], ['\\\\', '\\"'], $value);
changes to:
$parts = str_replace(['\\\\', '\\"'], ['\\', '"'], str_getcsv($typoLink, static::$partDelimiter));
str_getcsv removes single slashes.
Updated by Chris Müller over 1 year ago
To circumvent this you can use just a plain "a" tag as the menu link has not to be processed via a view helper.
Updated by Benni Mack about 1 year ago
- Status changed from New to Needs Feedback
Chris Müller wrote in #note-1:
To circumvent this you can use just a plain "a" tag as the menu link has not to be processed via a view helper.
I agree, I would do 100% the same: Using a menu "link" attribute again in a typolink will also double your DB requests for fetching the page again.
Maybe we need to update the docs for the menu processor / language menu processor that people should use <a> tags then.,
Updated by Philipp Kitzberger 10 months ago
Maybe the f:link.typolink should even throw a warning (deprecation log?) when parameter is starting with a slash?
"Don't put rendered pre-rendered stuff where it doesn't belong" ;-)
Updated by Benni Mack 7 months ago
Philipp Kitzberger wrote in #note-3:
Maybe the
f:link.typolinkshould even throw a warning (deprecation log?) whenparameteris starting with a slash?
People actually use it like this as well "/fileadmin/myfile.zip", how should we handle this then?
Updated by Georg Ringer 20 days ago
- Status changed from Needs Feedback to Rejected
hey Peter,
I am closing this issue. using just / in the f:link.typolink viewhelper should really not be something you would do. either use <f:link.page or just use a regular a-tag if you know already the url
Updated by Sascha Egerer 20 days ago
Georg Ringer wrote in #note-5:
hey Peter,
I am closing this issue. using just
/in thef:link.typolinkviewhelper should really not be something you would do. either use<f:link.pageor just use a regular a-tag if you know already the url
I'm not really happy with just closing it, as it's still a bug. It works with /foo but does not work with / - so in my opinion, this is not acceptable behavior.
I think it should either throw an exception because it's not allowed, or the behavior should be consistent. Throwing an exception is, of course, not feasible, so in my opinion, the only way to solve the problem is to fix it.
It is not always obvious that typolink is called twice - at least not if you use third-party code such as fluid_components. Of course, it is unfavorable if typolink is called twice, but as I said, the behavior should still be consistent and not just happen in the event that only a / is passed.