[TYPO3-core] RFC #9095: xhtml in the backend by default
Christian Kuhn
lolli at schwarzbu.ch
Thu Aug 14 16:50:08 CEST 2008
Hi.
Benjamin Mack wrote:
> Hey all,
>
> this is a SVN patch request.
>
> Branches: trunk only
>
> Type: pure cleanup
>
> BT reference: http://bugs.typo3.org/view.php?id=9095
>
> Problem: Well, if you look at the code of the BE classes, you see that
> there are many places where the template.php class is instantiated and
> then the xhtml doktype is set. as we all agreed to have xhtml as
> default, let's set this as default in template.html and get rid of all
> the lines that clutter our code to make it easier to read and to
> "not-have-to-think-about-adding-this-line" when coding new stuff.
>
> Solution:
> Set the default in the switch/case to xhtml_trans and remove all the
> occurences in the core where this is set manually.
>
> Notes:
> This does not change any functionality for anything in the core and if
> extensions still use this flag, nothing changes for them as well, it
> just makes the core code more readable and more compact in my eyes.
+1 on reading and testing.
Attached a new patch to fix issues mentioned by Ernesto, Oliver and Dmitry:
- Align break; with code in typo3/template.php
- Fix comment of $docType in typo3/template.php
- Add fallthrough comment to switch in typo3/template.php
- Fixed a failing Hunk in typo3/alt_file_navframe.php
Thanks
Christian
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 20080814_xhtml_by_default.patch
Url: http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20080814/6a69877a/attachment-0001.txt
More information about the TYPO3-team-core
mailing list