[TYPO3-core] RFC #15108: Bug/Code cleanup: .htaccess cleanups

[Markus Klein]

Hi Ernesto,

I read the discussion about your RFC.
This one is not only about cleaning the Installer but rather about cleaning up all this .htaccess code spread throughout a couple of files and the Installer.
In my opinion it's not a good idea to a code sample here and a sample there. Especially when thinking about maintainability this is horrible.
For instance, the _.htaccess file shipped with 4.4.2 dummy package is not the same as the misc/advanced.htaccess file.
Furthermore there's this tiny '/' bug in misc/advanced.htaccess which is not present in _.htaccess.

So there's only one way to go: Merge all .htaccess stuff to only one single file!
(Actually I would also remove this misc/simple.htaccess file)

BTW: It's a great idea to improve the Installer->All Configuration section.

cheers
Markus

> 
> Hi,
> 
> also refer to #15504 (pending on the core list), which adds possibility
> for HTML code in config_default descriptions. Thus allowing a <pre>
> around the RewriteRule sample. I already included that change in the
> config_default.diff attached to that issue.
> 
> Cheers,
> Ernesto
> 
> Markus Klein schrieb am 19.08.2010 12:06:
> > Ok, this is v3 now.
> > Just added a tiny note to NEWS.txt and another FAQ question to
> INSTALL.txt.
> >
> > kind regards
> >
> > Markus Klein
> > m.klein at mfc-linz.at
> >
> >> Hi!
> >>
> >> Markus Klein wrote:
> >>> Branches:
> >>> trunk
> >>>
> >>> Problem:
> >>> config_default.php contains Rewrite* statements within the
> >> description of some options.
> >>> As there's no possibility to format these descriptions they're
> quite
> >> hard to read. The author actually used dashes to indicate line
> breaks.
> >> (But a dash is a valid operator for RewriteCond)
> >>> Solution:
> >>> Remove all .htaccess code samples from config_default.php and put
> >> them into misc/advanced.htaccess.
> >>> I modified the descriptions which now refer to this file.
> >>> There's also been a small bug in advanced.htaccess with favicon.ico
> >> files.
> >>
> >> +1 by reading. I would be happy if Benni allows this for 4.4 too.
> >>
> >>> Notes:
> >>> Please also remove the _.htaccess file from the dummy package as
> this
> >> file is not consistent with advanced.htaccess anymore.
> >>> Anyway I think we shouldn't have two of this files. Please also add
> a
> >> note to INSTALL.txt accordingly.
> >>
> >> INSTALL.txt is supplied with TYPO3 core, you can make a patch for
> it.
> >>
> >> Btw, thanks for this clean up :)
> >>
> >> --
> >> Dmitry Dulepov
> >> TYPO3 core&security team member
> >> Twitter: http://twitter.com/dmitryd
> >> Read more @ http://dmitry-dulepov.com/
> >