Bug #14246
closedcharset and generator meta tags are written in XHTML by default
0%
Description
On lines 597 and 599 of class.tslib_pagegen.php the generator and content-type meta-tags are written with a slash before the ending > of the tag.
This will make it impossible to generate valid HTML pages with TYPO3.
The slashes should either be made dependant on the selected doctype or just be removed as they get inserted anyway if you set config.xhtmlcleaning .
(issue imported from #M236)
Files
Updated by Ingmar Schlecht over 20 years ago
Hmm, the long time goal still is XHTML, so I would not like to remove the slashes.
Perhaps we should create something like this:
config.htmlcleaning
However, isn't it better to just use "tidy"?
Does it still have these problems of destroying your layout?
cheers,
Ingmar
Updated by Sacha Vorbeck over 20 years ago
Hi Ingmar,
Hmm, the long time goal still is XHTML, so I would not like to remove the
slashes.
but then it is not possible to generate valid HTML-code with TYPO3. And that can`t be a desired behaviour, can it?
At the moment the default output of TYPO3 is HTML. If you want to have XHTML, you set the XHTML cleaning option and choose an appropriate doctype. The ending slashes will then be added automatically by the function processTag in class.tslib_pagegen.php.
Perhaps we should create something like this:
config.htmlcleaning
I don`t see the necessity of it as the standard output would be nice clean HTML without those two trailing slashes. They are a break of concept IMHO.
However, isn't it better to just use "tidy"?
I`m not sure about the effects tidy has on the general performance. Will the contents be piped through tidy and then be dumped into the TYPO3 cache?
Does it still have these problems of destroying your layout?
I guess tidy could solve this problem. But see above. I don`t understand why I have to use tidy if it`s only needed to remove those two slashes.
Ciao,
Sacha
Updated by Michael Scharkow almost 20 years ago
I don't think tidy is a proper solution because a) it's costly in performance, b) it might not be available, c) page rendering should be correct with TYPO3 alone.
BTW, it's still hardcoded in 3.7
Updated by Lars Houmark about 19 years ago
Is this to be fixed in the next version? It's very annoying. It's the only hardcoding done in all our installations :/
I agree that there should be added a variable, that would include the tag if xhtml_cleaning is enabled, but the default way should be without the trailing /.
Isn't it more than just 2 places. I think it's three or four, also the stylesheet inclusion is causing trouble though there is other ways of including the stylesheet ;)
Updated by Christian Buelter over 14 years ago
attached patch is for current TYPO3 4.4 Trunk