Bug #16112
closedHTML-Source in Error-Pages not valid (FE)
0%
Description
e. g. "Error. Reason: Page is not available in the requested language."
- missing <!DOCTYPE> declaration
- various errors in source (e. g. <td style="text-align:center";>)
(issue imported from #M3391)
Updated by Ernesto Baschny over 18 years ago
The referred page is being generated by:
t3lib/class.t3lib_timetrack.php::debug_typo3PrintError
Do you have a cleaner suggestion on how this could look? If so, please attach an example to this request.
I think best would be to have a very good default but have this error output configurable (through TYPO3CONF) from an external template with markers (###BASEURL###, ###HEADER###, ###TEXT###).
Updated by Toralf Lenz over 18 years ago
A external template would be fine! Primary I would correct the errors and insert the DOCTYPE. DOCTYPE should be PageTS "config.doctype".
Updated by Martin Kutschker over 18 years ago
"Nice" HTML markup is fine, but is validity really an issue for debugging code?
Updated by Ernesto Baschny over 18 years ago
@Martin,
this is not "debugging code", this is the error page that TYPO3 generates in the FE (e.g. the "404" or the RealURL error message if something is wrong in the path).
As this might be shown to anonymous users of your site, I would find it cool if the site admin could design his own error page or if this would at least be consistently styled.
Updated by Toralf Lenz over 18 years ago
@Ernesto,
this is exactly what I mean! A page in the FE should be valid and editable by the admin (f. e.: styleguide of the page).
Updated by Martin Kutschker over 18 years ago
Ah, now I see.
Toralf is right, the markup should be fixed. But IMHO it's enough to produce valid stuff (no real need to make it configurable, but if it's easy fine).
Ernesto is also right when he asks for a customization. Please file a bug (type "feature").
Updated by Toralf Lenz over 18 years ago
@Ernesto,
write you the new feature-wish to the bugtracker?
Updated by Ernesto Baschny over 18 years ago
Patch submitted to the core-list.
@Toralf, if you have some time, please post a new feature request to the bug-tracker. Thanks for your help!