[CLEANUP] Fix usage of GeneralUtility::quoteJSvalue 27/51727/2
authorSascha Egerer <sascha@sascha-egerer.de>
Thu, 16 Feb 2017 22:22:54 +0000 (23:22 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Thu, 16 Feb 2017 23:44:54 +0000 (00:44 +0100)
The function GeneralUtility::quoteJSvalue is sometimes called
with a second parameter but it does only have on parameter.

Resolves: #79867
Releases: master
Change-Id: I994501574b798026f43a0e463a633e05ffc1cff3
Reviewed-on: https://review.typo3.org/51727
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/feedit/Classes/FrontendEditPanel.php
typo3/sysext/frontend/Classes/ContentObject/Menu/JavaScriptMenuContentObject.php

index f41b232..187d1e1 100644 (file)
@@ -275,7 +275,7 @@ class FrontendEditPanel
         } else {
             if ($confirm && $this->backendUser->jsConfirmation(JsConfirmation::FE_EDIT)) {
                 // Gets htmlspecialchared later
-                $cf1 = 'if (confirm(' . GeneralUtility::quoteJSvalue($confirm, true) . ')) {';
+                $cf1 = 'if (confirm(' . GeneralUtility::quoteJSvalue($confirm) . ')) {';
                 $cf2 = '}';
             } else {
                 $cf1 = ($cf2 = '');
index bbb48e2..3912586 100644 (file)
@@ -154,7 +154,7 @@ var ' . $this->JSVarName . $a . '=0;';
                         $url = $this->getTypoScriptFrontendController()->baseUrlWrap($LD['totalURL']);
                         $target = $LD['target'];
                     }
-                    $codeLines .= LF . $var . $count . '=' . $menuName . '.add(' . $parent . ',' . $prev . ',0,' . GeneralUtility::quoteJSvalue($title, true) . ',' . GeneralUtility::quoteJSvalue($url, true) . ',' . GeneralUtility::quoteJSvalue($target, true) . ');';
+                    $codeLines .= LF . $var . $count . '=' . $menuName . '.add(' . $parent . ',' . $prev . ',0,' . GeneralUtility::quoteJSvalue($title) . ',' . GeneralUtility::quoteJSvalue($url) . ',' . GeneralUtility::quoteJSvalue($target) . ');';
                     // If the active one should be chosen...
                     $active = $levelConf['showActive'] && $this->isActive($data['uid'], $MP_var);
                     // If the first item should be shown
@@ -178,7 +178,7 @@ var ' . $this->JSVarName . $a . '=0;';
             $levelConf['firstLabel'] = $this->mconf['firstLabelGeneral'];
         }
         if ($levelConf['firstLabel'] && $codeLines) {
-            $codeLines .= LF . $menuName . '.defTopTitle[' . $count . '] = ' . GeneralUtility::quoteJSvalue($levelConf['firstLabel'], true) . ';';
+            $codeLines .= LF . $menuName . '.defTopTitle[' . $count . '] = ' . GeneralUtility::quoteJSvalue($levelConf['firstLabel']) . ';';
         }
         return $codeLines;
     }