[Typo3-dev] typolink.addQueryString is broken

[Michael Stucki]

Hi Christopher & Martin,

>> > 3.8.1. I tested it by building a simple typolink to return the current
>> > page's url including all GET params.  On e.g. a  tx_calendar page, the
>> > typolink returns something like http://www.example.com/events/event/
>> > when it should return
>> > http://www.example.com/events/event/details/eventname...
>>
>> I see.
>>
>> addQueryString uses the real QUERY_STRING, but Real URL manipulates _GET
>>
>> To make it work for your kind of setup, the Core function must be
>> changed.
>>
>> Michael, does it make sense to loop over _GET (for Real URL) instead of
>> simply using QUERY_STRING? When I implement removeParams I'll have to do
>> a loop anyway.

Yes I think it does (although I never used RealURL and cannot test the
changes).

Just make sure that the existing behaviour does not change.

> Oh, that makes sense then; it's even what it says in the TSref for
> that matter... Even if it's not practical to change the behaviour, it
> might be worth slightly expanding the note in the TSref entry to
> clarify the difference.

Until the problem has been fixed, I have added a notice to TSref.

- michael
-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/