[CLEANUP] Superfluous Code in ElementBrowser and BrowseLinks 56/27856/2
authorMarkus Klein <klein.t3@mfc-linz.at>
Tue, 25 Feb 2014 20:59:05 +0000 (21:59 +0100)
committerMarkus Klein <klein.t3@mfc-linz.at>
Tue, 25 Feb 2014 21:10:27 +0000 (22:10 +0100)
Resolves: #56290
Releases: 6.2
Change-Id: I85348aea9fd514b9774b63846f444fcb9a8310e4
Reviewed-on: https://review.typo3.org/27856
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Klein
Tested-by: Markus Klein
typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php
typo3/sysext/rtehtmlarea/Classes/BrowseLinks.php

index 39e96cc..1943aa1 100644 (file)
@@ -399,17 +399,17 @@ class ElementBrowser {
                // BEGIN accumulation of header JavaScript:
                $JScode = '
                        // This JavaScript is primarily for RTE/Link. jumpToUrl is used in the other cases as well...
-                       var add_href=' . GeneralUtility::quoteJSvalue(($this->curUrlArray['href'] ? '&curUrl[href]=' . rawurlencode($this->curUrlArray['href']) : '')) . ';
-                       var add_target=' . GeneralUtility::quoteJSvalue(($this->setTarget ? '&curUrl[target]=' . rawurlencode($this->setTarget) : '')) . ';
-                       var add_class=' . GeneralUtility::quoteJSvalue(($this->setClass ? '&curUrl[class]=' . rawurlencode($this->setClass) : '')) . ';
-                       var add_title=' . GeneralUtility::quoteJSvalue(($this->setTitle ? '&curUrl[title]=' . rawurlencode($this->setTitle) : '')) . ';
-                       var add_params=' . GeneralUtility::quoteJSvalue(($this->bparams ? '&bparams=' . rawurlencode($this->bparams) : '')) . ';
-
-                       var cur_href=' . GeneralUtility::quoteJSvalue(($this->curUrlArray['href'] ? $this->curUrlArray['href'] : '')) . ';
-                       var cur_target=' . GeneralUtility::quoteJSvalue(($this->setTarget ? $this->setTarget : '')) . ';
-                       var cur_class=' . GeneralUtility::quoteJSvalue(($this->setClass ? $this->setClass : '')) . ';
-                       var cur_title=' . GeneralUtility::quoteJSvalue(($this->setTitle ? $this->setTitle : '')) . ';
-                       var cur_params=' . GeneralUtility::quoteJSvalue(($this->setParams ? $this->setParams : '')) . ';
+                       var add_href=' . GeneralUtility::quoteJSvalue($this->curUrlArray['href'] ? '&curUrl[href]=' . rawurlencode($this->curUrlArray['href']) : '') . ';
+                       var add_target=' . GeneralUtility::quoteJSvalue($this->setTarget ? '&curUrl[target]=' . rawurlencode($this->setTarget) : '') . ';
+                       var add_class=' . GeneralUtility::quoteJSvalue($this->setClass ? '&curUrl[class]=' . rawurlencode($this->setClass) : '') . ';
+                       var add_title=' . GeneralUtility::quoteJSvalue($this->setTitle ? '&curUrl[title]=' . rawurlencode($this->setTitle) : '') . ';
+                       var add_params=' . GeneralUtility::quoteJSvalue($this->bparams ? '&bparams=' . rawurlencode($this->bparams) : '') . ';
+
+                       var cur_href=' . GeneralUtility::quoteJSvalue($this->curUrlArray['href'] ?: '') . ';
+                       var cur_target=' . GeneralUtility::quoteJSvalue($this->setTarget ?: '') . ';
+                       var cur_class=' . GeneralUtility::quoteJSvalue($this->setClass ?: '') . ';
+                       var cur_title=' . GeneralUtility::quoteJSvalue($this->setTitle ?: '') . ';
+                       var cur_params=' . GeneralUtility::quoteJSvalue($this->setParams ?: '') . ';
 
                        function browse_links_setTarget(target) {       //
                                cur_target=target;
index f90e5f8..d4fef8e 100644 (file)
@@ -300,18 +300,18 @@ class BrowseLinks extends \TYPO3\CMS\Recordlist\Browser\ElementBrowser {
                $JScode .= '
                        var plugin = window.parent.RTEarea["' . $this->editorNo . '"].editor.getPlugin("TYPO3Link");
                        var HTMLArea = window.parent.HTMLArea;
-                       var add_href=' . GeneralUtility::quoteJSvalue(($this->curUrlArray['href'] ? '&curUrl[href]=' . rawurlencode($this->curUrlArray['href']) : '')) . ';
-                       var add_target=' . GeneralUtility::quoteJSvalue(($this->setTarget ? '&curUrl[target]=' . rawurlencode($this->setTarget) : '')) . ';
-                       var add_class=' . GeneralUtility::quoteJSvalue(($this->setClass ? '&curUrl[class]=' . rawurlencode($this->setClass) : '')) . ';
-                       var add_title=' . GeneralUtility::quoteJSvalue(($this->setTitle ? '&curUrl[title]=' . rawurlencode($this->setTitle) : '')) . ';
-                       var add_params=' . GeneralUtility::quoteJSvalue(($this->bparams ? '&bparams=' . rawurlencode($this->bparams) : '')) . ';
+                       var add_href=' . GeneralUtility::quoteJSvalue($this->curUrlArray['href'] ? '&curUrl[href]=' . rawurlencode($this->curUrlArray['href']) : '') . ';
+                       var add_target=' . GeneralUtility::quoteJSvalue($this->setTarget ? '&curUrl[target]=' . rawurlencode($this->setTarget) : '') . ';
+                       var add_class=' . GeneralUtility::quoteJSvalue($this->setClass ? '&curUrl[class]=' . rawurlencode($this->setClass) : '') . ';
+                       var add_title=' . GeneralUtility::quoteJSvalue($this->setTitle ? '&curUrl[title]=' . rawurlencode($this->setTitle) : '') . ';
+                       var add_params=' . GeneralUtility::quoteJSvalue($this->bparams ? '&bparams=' . rawurlencode($this->bparams) : '') . ';
                        var additionalValues = ' . (count($this->additionalAttributes) ? json_encode($this->additionalAttributes, JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT) : '{}') . ';';
                // Attributes setting functions
                $JScode .= '
-                       var cur_href="' . GeneralUtility::quoteJSvalue($this->curUrlArray['href'] ? ($this->curUrlInfo['query'] ? substr($this->curUrlArray['href'], 0, -strlen($this->curUrlInfo['query'])) : $this->curUrlArray['href']) : '') . '";
-                       var cur_target="' . GeneralUtility::quoteJSvalue($this->setTarget ?: '') . '";
-                       var cur_class="' . GeneralUtility::quoteJSvalue($this->setClass ?: '') . '";
-                       var cur_title="' . GeneralUtility::quoteJSvalue($this->setTitle ?: '') . '";
+                       var cur_href=' . GeneralUtility::quoteJSvalue($this->curUrlArray['href'] ? ($this->curUrlInfo['query'] ? substr($this->curUrlArray['href'], 0, -strlen($this->curUrlInfo['query'])) : $this->curUrlArray['href']) : '') . ';
+                       var cur_target=' . GeneralUtility::quoteJSvalue($this->setTarget ?: '') . ';
+                       var cur_class=' . GeneralUtility::quoteJSvalue($this->setClass ?: '') . ';
+                       var cur_title=' . GeneralUtility::quoteJSvalue($this->setTitle ?: '') . ';
 
                        function browse_links_setTarget(value) {
                                cur_target=value;