[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