[TYPO3-core] JumpUrl restructuring

[Anja Leichsenring]

Hello Alexander,

thats sounds like lots of work and effort. Did I miss something, or is 
your new class still hidden? You ask for feedback, but forgot most 
probably to provide a link?

Kind regards
Anja

On 09/26/2013 05:09 PM, Alexander Stehlik wrote:
> Hi Guys,
>
> in the last couple of days I spent some time looking into the jump URL
> feature. Originally I intended to supply a patch for
> http://forge.typo3.org/issues/52156 but I e ended up creating a new
> JumpUrlUtility class for bringing all the JumpUrl functionality in one
> place.
>
> The good thing about this rewrite is, that all places where jump URLs
> can be configured the same options are available. This was not always
> the case in the old integration.
>
> Another issue is tackled with this patch. When creating file links with
> the typoLink method the parameter that was expected needed to be created
> in a very special way so that everything works properly:
>
> str_replace('%2F', '/', $fileUrl);
>
> This is not very convenient. The new version simply expects a
> rawurlencoded version of the file URL which can be called by stdWrap. It
> will also make sure that files with special characters that are linked
> using their FAL UID will work (which is currently not always the case).
>
> I created a bunch of Unit Tests to make sure the system behaves the same
> before and after the patch. You can use the ContentObjectRendererTest
> class to test an unpatched master. The tests will be cleaned so that
> they only work in the patchted version when you are finished testing :)
>
> Would be great to get some feedback and hints for improvements or
> further tests.
>
> Cheers,
> Alex
>