[BUGFIX] ElementBrowser: correct link attributes 83/30483/3
authorMarkus Klein <klein.t3@mfc-linz.at>
Thu, 29 May 2014 10:40:58 +0000 (12:40 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Tue, 3 Jun 2014 20:06:57 +0000 (22:06 +0200)
This is a regression fix for #55983, which introduced
additional urlencoding for title, class, ...

Remove the encoding again.

Resolves: #59199
Releases: 6.2
Change-Id: I28650676f4ae0b74256a770af12054750027cbfa
Reviewed-on: https://review.typo3.org/30483
Reviewed-by: Markus Bucher
Tested-by: Markus Bucher
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php

index 6df16b4..919a109 100644 (file)
@@ -297,16 +297,16 @@ class ElementBrowser {
                }
 
                // Initializing the target value (RTE)
-               $this->setTarget = $this->curUrlArray['target'] != '-' ? rawurlencode($this->curUrlArray['target']) : '';
+               $this->setTarget = $this->curUrlArray['target'] != '-' ? $this->curUrlArray['target'] : '';
                if ($this->thisConfig['defaultLinkTarget'] && !isset($this->curUrlArray['target'])) {
                        $this->setTarget = $this->thisConfig['defaultLinkTarget'];
                }
                // Initializing the class value (RTE)
-               $this->setClass = $this->curUrlArray['class'] != '-' ? rawurlencode($this->curUrlArray['class']) : '';
+               $this->setClass = $this->curUrlArray['class'] != '-' ? $this->curUrlArray['class'] : '';
                // Initializing the title value (RTE)
-               $this->setTitle = $this->curUrlArray['title'] != '-' ? rawurlencode($this->curUrlArray['title']) : '';
+               $this->setTitle = $this->curUrlArray['title'] != '-' ? $this->curUrlArray['title'] : '';
                // Initializing the params value
-               $this->setParams = $this->curUrlArray['params'] != '-' ? rawurlencode($this->curUrlArray['params']) : '';
+               $this->setParams = $this->curUrlArray['params'] != '-' ? $this->curUrlArray['params'] : '';
 
                // Finally, add the accumulated JavaScript to the template object:
                // also unset the default jumpToUrl() function before