[TYPO3-core] Getting an overview

Michael Stucki michael at typo3.org
Mon Mar 5 22:55:34 CET 2007


REPOST. See my reply to Ingmar, this is about the cHash issue...

- michael

Michael Stucki wrote:

> There is still one remaining problem:
> 
> In case of a cached plugin (eg. a shop, or a forum) which contains
> non-cached parts, extension developers usually call $TSFE->set_no_cache()
> to work prevent caching of this page.
> 
> This will cause $TSFE->no_cache to be true while the plugin still
> indicates that it uses cHashes ($this->pi_checkCHash = TRUE).
> 
> So actually the plugin is misconfigured, but luckily we can easily work
> around that and prevent other extensions from having the same problem:
> Just check if $TSFE->no_cache is set and do not check the cHash in this
> case.
> 
> Attached is a patch that should fix the problem.
> 
> Thanks a lot Oliver for investigating this problem of Franz' extension...
> 
> I would especially like to know from Franz if this fixes all problems in
> tt_products. Any other extension that makes trouble so far?
> 
> - michael
> 
> Michael Stucki wrote:
> 
>> Hi Oliver,
>> 
>>> >    * Request parameters could not be validated
>>> >      http://bugs.typo3.org/view.php?id=4940
>>>
>>> I picked this one and added a small patch for it (see in bugtracker).
>>> "pageNotFoundOnCHashError" was set to true by default between beta3 and
>>> RC1. So if an extension (perhaps an old one) doesn't use the cHash (eg
>>> "typolink.useCHash=1") the error page will be shown. So my solution is
>>> to let "pageNotFoundOnCHashError" as it is and to define a new config
>>> variable "pageNotFoundOnCHashEmpty", which is set to false by default.
>>> If the admin wants the error page if cHash is empty he could enable this
>>> in his local TYPO3 installation. What do you think?
>>> Additionally I could also add a RFC on that and post it to a new thread.
>> 
>> No, I think you shouldn't do that. It doesn't make much sense to allow
>> empty cHashes if they should be checked. See my detailed explanation
>> inside the bugtracker...
>> 
>> - michael
> 

-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug_4940.diff
Type: text/x-diff
Size: 643 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20070305/8fb1b906/attachment.bin 


More information about the TYPO3-team-core mailing list