[TYPO3-core] JumpUrl restructuring

[Alexander Stehlik]

Hi Guys,

that is great news :)

I uploaded a new patchset to Gerrit:

https://review.typo3.org/#/c/23940/

There is one question still open:

Should the getPublicUrl() method of a file / folder not URL encode 
special characters in the file path and filename? As far as I can see 
this should be done in the LocalDriver class. I mean, this method is 
supposed to return a valid URL, right?

Maybe the snippet from the jump URL links can be reused to do that since 
it seems to work quite good:

str_replace('%2F', '/', rawurlencode($fileUrl));

Then I could get rid of the URL encoding in the typolink() method which 
I do not really like the way it is currently implemented (see also 
inline comments in the patch request).

Cheers,
Alex

Am 11.10.2013 16:21, schrieb Helmut Hummel:
> Hi Alex,
>
> On 10.10.13 11:25, Oliver Hader wrote:
>
>> this topic is on our agenda for further discussion at the current TYPO3
>> CMS code sprint in Hannover [1]..
>
> We agreed that such a change is a good enough refactoring step which can
> be built upon.
>
> So if the change itself works and does not introduce further issues, it
> can be accepted.
>
> Please note that it should be ready until feature freeze next Tuesday if
> it is targeted for 6.2
>
> Kind regards,
> Helmut
>