[TYPO3-dev] WARNING!?? tslib_patcher, pp_chashchecker
Macmade
macmade at gadlab.net
Thu Aug 23 12:05:36 CEST 2007
Hello list,
I didn't expected so many reactions on that tslib_patcher extension.
Please be assured that it wasn't in my intentions to override the core team.
I'll try to explain why I published this extension.
I got some bug reports on my others extensions about that cHash problem.
I've tried to check that, but all the links were created as said in the
documentation. But still no cHash in the URLs.
By speaking with Popy, I discovered that it wasn't computed right by
the typoLink function and the TS menus.
Some issues were already published on the bugtracker, and a patch file
was existing for the typoLink function (provided by Popy).
I really needed to have those problems fixed. And I didn't wanted to
patch the core classes by hand. That's why I created the extension.
I personnaly think that the best way of contributing is to provide a
patch, and get in touch with the core developers. But that was already
done. And I really needed the fix for some of my clients. So I tried to
do this as clean as possible, by providing a way to decide which core
class will be overriden. Actually, when you install the extension, no
core class in overriden. You need to explicitely activate the features
for that.
I've published it in the hope that it may be useful for other people,
that encountered the same problems. It wasn't for overriding the bugfix
process. As I did that work, I just wanted to share it. That's also how
I see the «inspiring people to share» thing...
About the cHash, just try to enable the «pageNotFoundOnCHashError» in
the installTool... I think you'll be surprised by the result.
About the L variable, it's actually not a core parameter (maybe it
should). So there's definitively a need for the cHash parameter when
this variable is present (and with realURL it's always there). The only
«core» parameters, which doesn't need a cHash are actually
id,type,no_cache,cHash,MP,ftu. They are defined in
t3lib_div::cHashParams().
Somebody quoted my documentation, about the cHash. But as it's said,
those quotes were taken on an official article about the cHash, which
says clearly that this parameter is needed.
Here's that article, for those who are interested:
http://typo3.org/development/articles/the-mysteries-of-chash/
So I'm really sorry if you still disagree with the publishing of the
extension. As I said, it wasn't in my intentions to create bad noise
about this.
I'm looking forward to do what needs to be done to solve the problem,
if any, and to correctly inform the users. If there's something to
change, in the extension and in the documentation, just tell me, I will
be happy to do those changes. : )
Peace,
Best regards,
P.S.: A little suggestion for that kind of discussion. If you need to
talk on that mailing list about a particular extension, please inform
it's author about the discussion. In my humble opinion, this would be a
much more nicer way to discuss about that kind of problems... : )
More information about the TYPO3-dev
mailing list