[TYPO3-core] RFC Bug #10452: Evaluation PATH_INFO in tslib_fe::checkAlternativeIdMethods() sets false positives and should be moved to tx_simulatestatic

[Oliver Hader]

FYI: Committed follow-up to Trunk (rev. 5044)

olly


Oliver Hader schrieb:
> Hi,
> 
> I'm going to commit the attached patch in some hours as a follow-up
> since the previous patch introduced another bug or was not complete.
> It's not possible to check config.simulateStaticDocuments at that point,
> thus, evaluating the URL scheme "index.php/.../.../" must be enough as
> indication that PATH_INFO was used.
> 
> Sorry for the confusion!
> 
> olly
> 
> 
> Oliver Hader schrieb:
>> This is an SVN patch request.
>>
>> Type: Bugfix
>>
>> Bugtracker references:
>> http://bugs.typo3.org/view.php?id=10452
>>
>> Branch: Trunk
>>
>> Problem:
>> The method tslib_fe::checkAlternativeIdMethods() in TSFE tries to fetch
>> the server environment PATH_INFO and if there's something it expects an
>> URL like "/index.php/<page-id>/<type-number>/" and sets the current id
>> and type accordant.
>>
>> However, if config.simulateStaticDocuments is not set to "PATH_INFO"
>> this is a false positive.
>>
>> Since the functionality of simulateStaticDocuments was extracted to the
>> system extension tx_simulatestatic in TYPO3 4.3, the mentioned part
>> should also be moved to that place.
>>
>> Solution:
>> Check whether config.simulateStaticDocuments is set to "PATH_INFO" and
>> only then try to evaluate the id and type by the PATH_INFO information.
>> Additionally the functionality gets moved to tx_simulatestatic since
>> that's the correct place.
>>
>> Notes:
>> This RFC is nearly the same as RFC #10451 - but since it moves things
>> around and changes the structure a bit, it's a separate RFC.
>>
>>
>> olly
-- 
Oliver Hader
TYPO3 Release Manager 4.3