[TASK] Remove leftover backPath and thumbs.php variables 38/46738/2
authorBenni Mack <benni@typo3.org>
Tue, 16 Feb 2016 22:33:32 +0000 (23:33 +0100)
committerBenni Mack <benni@typo3.org>
Wed, 17 Feb 2016 09:30:00 +0000 (10:30 +0100)
Resolves: #73512
Releases: master
Change-Id: Ib0add5b4a07bd43a68a46d29fe0f13d448240966
Reviewed-on: https://review.typo3.org/46738
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Thomas Schlumberger <thomas@b13.de>
Tested-by: Thomas Schlumberger <thomas@b13.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
typo3/sysext/backend/Classes/Form/Container/AbstractContainer.php
typo3/sysext/backend/Classes/Form/FormResultCompiler.php
typo3/sysext/backend/Classes/Utility/BackendUtility.php
typo3/sysext/backend/Resources/Public/JavaScript/FormEngine.js
typo3/sysext/backend/Resources/Public/JavaScript/jsfunc.tbe_editor.js
typo3/sysext/core/Classes/Resource/ResourceCompressor.php
typo3/sysext/core/Classes/TypoScript/ConfigurationForm.php
typo3/sysext/core/Tests/Unit/Resource/ResourceCompressorTest.php
typo3/sysext/workspaces/Classes/ExtDirect/ExtDirectServer.php

index be4c49d..51766f3 100644 (file)
@@ -134,7 +134,7 @@ abstract class AbstractContainer extends AbstractNode
                         $table,
                         $field,
                         '',
-                        'thumbs.php',
+                        '',
                         $config['config']['uploadfolder'], 0, ' align="middle"'
                     ) .
                     ($absFilePath ? BackendUtility::wrapClickMenuOnIcon($fileIcon, $absFilePath, 0, 1, '', '+copy,info,edit,view') : $fileIcon) .
index 57f7311..a6a004a 100644 (file)
@@ -192,7 +192,6 @@ class FormResultCompiler
         // set variables to be accessible for JS
         $pageRenderer = $this->getPageRenderer();
         $pageRenderer->addInlineSetting('FormEngine', 'formName', 'editform');
-        $pageRenderer->addInlineSetting('FormEngine', 'backPath', '');
 
         return $this->JSbottom('editform');
     }
@@ -279,7 +278,6 @@ class FormResultCompiler
         $out .= '
                TBE_EDITOR.formname = "' . $formname . '";
                TBE_EDITOR.formnameUENC = "' . rawurlencode($formname) . '";
-               TBE_EDITOR.backPath = "";
                TBE_EDITOR.isPalettedoc = null;
                TBE_EDITOR.doSaveFieldName = "' . ($this->doSaveFieldName ? addslashes($this->doSaveFieldName) : '') . '";
                TBE_EDITOR.labels.fieldsChanged = ' . GeneralUtility::quoteJSvalue($languageService->sL('LLL:EXT:lang/locallang_core.xlf:labels.fieldsChanged')) . ';
index fb5c1d7..1817fc8 100755 (executable)
@@ -1468,7 +1468,7 @@ class BackendUtility
                     $imgTag = '<span title="' . htmlspecialchars($fileObject->getName()) . '">' . $iconFactory->getIconForResource($fileObject, Icon::SIZE_SMALL)->render() . '</span>';
                 }
                 if ($linkInfoPopup) {
-                    $onClick = 'top.launchView(\'_FILE\',\'' . (int)$fileObject->getUid() . '\',' . GeneralUtility::quoteJSvalue($backPath) . '); return false;';
+                    $onClick = 'top.launchView(\'_FILE\',\'' . (int)$fileObject->getUid() . '\'); return false;';
                     $thumbData .= '<a href="#" onclick="' . htmlspecialchars($onClick) . '">' . $imgTag . '</a> ';
                 } else {
                     $thumbData .= $imgTag;
@@ -1519,7 +1519,7 @@ class BackendUtility
                         ))->getPublicUrl(true);
                         $image = '<img src="' . htmlspecialchars($imageUrl) . '" hspace="2" border="0" title="' . htmlspecialchars($fileObject->getName()) . '"' . $tparams . ' alt="" />';
                         if ($linkInfoPopup) {
-                            $onClick = 'top.launchView(\'_FILE\', ' . GeneralUtility::quoteJSvalue($fileName) . ',\'\',' . GeneralUtility::quoteJSvalue($backPath) . ');return false;';
+                            $onClick = 'top.launchView(\'_FILE\', ' . GeneralUtility::quoteJSvalue($fileName) . ',\'\');return false;';
                             $thumbData .= '<a href="#" onclick="' . htmlspecialchars($onClick) . '">' . $image . '</a> ';
                         } else {
                             $thumbData .= $image;
@@ -1528,7 +1528,7 @@ class BackendUtility
                         // Gets the icon
                         $fileIcon = '<span title="' . htmlspecialchars($fileObject->getName()) . '">' . $iconFactory->getIconForResource($fileObject, Icon::SIZE_SMALL)->render() . '</span>';
                         if ($linkInfoPopup) {
-                            $onClick = 'top.launchView(\'_FILE\', ' . GeneralUtility::quoteJSvalue($fileName) . ',\'\',' . GeneralUtility::quoteJSvalue($backPath) . '); return false;';
+                            $onClick = 'top.launchView(\'_FILE\', ' . GeneralUtility::quoteJSvalue($fileName) . ',\'\'); return false;';
                             $thumbData .= '<a href="#" onclick="' . htmlspecialchars($onClick) . '">' . $fileIcon . '</a> ';
                         } else {
                             $thumbData .= $fileIcon;
@@ -3187,19 +3187,17 @@ class BackendUtility
      *
      * @param string $moduleName Name of the module
      * @param array $urlParameters URL parameters that should be added as key value pairs
-     * @param bool|string $backPathOverride (unused)
-     * @param bool $returnAbsoluteUrl If set to TRUE, the URL returned will be absolute, $backPathOverride will be ignored in this case
      * @return string Calculated URL
      */
-    public static function getModuleUrl($moduleName, $urlParameters = array(), $backPathOverride = false, $returnAbsoluteUrl = false)
+    public static function getModuleUrl($moduleName, $urlParameters = array())
     {
         /** @var UriBuilder $uriBuilder */
         $uriBuilder = GeneralUtility::makeInstance(UriBuilder::class);
         try {
-            $uri = $uriBuilder->buildUriFromRoute($moduleName, $urlParameters, $returnAbsoluteUrl ? UriBuilder::ABSOLUTE_URL : UriBuilder::ABSOLUTE_PATH);
+            $uri = $uriBuilder->buildUriFromRoute($moduleName, $urlParameters);
         } catch (\TYPO3\CMS\Backend\Routing\Exception\RouteNotFoundException $e) {
             // no route registered, use the fallback logic to check for a module
-            $uri = $uriBuilder->buildUriFromModule($moduleName, $urlParameters, $returnAbsoluteUrl ? UriBuilder::ABSOLUTE_URL : UriBuilder::ABSOLUTE_PATH);
+            $uri = $uriBuilder->buildUriFromModule($moduleName, $urlParameters);
         }
         return (string)$uri;
     }
@@ -3213,20 +3211,18 @@ class BackendUtility
      *
      * @param string $ajaxIdentifier Identifier of the AJAX callback
      * @param array $urlParameters URL parameters that should be added as key value pairs
-     * @param bool $backPathOverride (unused)
-     * @param bool $returnAbsoluteUrl If set to TRUE, the URL returned will be absolute, $backPathOverride will be ignored in this case
      * @return string Calculated URL
      */
-    public static function getAjaxUrl($ajaxIdentifier, array $urlParameters = array(), $backPathOverride = false, $returnAbsoluteUrl = false)
+    public static function getAjaxUrl($ajaxIdentifier, array $urlParameters = array())
     {
         /** @var UriBuilder $uriBuilder */
         $uriBuilder = GeneralUtility::makeInstance(UriBuilder::class);
         try {
             $routeIdentifier = 'ajax_' . $ajaxIdentifier;
-            $uri = $uriBuilder->buildUriFromRoute($routeIdentifier, $urlParameters, $returnAbsoluteUrl ? UriBuilder::ABSOLUTE_URL : UriBuilder::ABSOLUTE_PATH);
+            $uri = $uriBuilder->buildUriFromRoute($routeIdentifier, $urlParameters);
         } catch (\TYPO3\CMS\Backend\Routing\Exception\RouteNotFoundException $e) {
             // no route registered, use the fallback logic to check for a module
-            $uri = $uriBuilder->buildUriFromAjaxId($ajaxIdentifier, $urlParameters, $returnAbsoluteUrl ? UriBuilder::ABSOLUTE_URL : UriBuilder::ABSOLUTE_PATH);
+            $uri = $uriBuilder->buildUriFromAjaxId($ajaxIdentifier, $urlParameters);
         }
         return (string)$uri;
     }
index 578c12a..6f2ad5e 100644 (file)
@@ -37,12 +37,11 @@ define(['jquery',
 
        /**
         *
-        * @type {{formName: *, backPath: *, openedPopupWindow: null, legacyFieldChangedCb: Function, browserUrl: string}}
+        * @type {{formName: *, openedPopupWindow: null, legacyFieldChangedCb: Function, browserUrl: string}}
         * @exports TYPO3/CMS/Backend/FormEngine
         */
        var FormEngine = {
                formName: TYPO3.settings.FormEngine.formName
-               ,backPath: TYPO3.settings.FormEngine.backPath
                ,openedPopupWindow: null
                ,legacyFieldChangedCb: function() { !$.isFunction(TYPO3.settings.FormEngine.legacyFieldChangedCb) || TYPO3.settings.FormEngine.legacyFieldChangedCb(); }
                ,browserUrl: ''
@@ -68,7 +67,7 @@ define(['jquery',
         * @param {Number} height height of the window
         */
        FormEngine.openPopupWindow = setFormValueOpenBrowser = function(mode, params, width, height) {
-               var url = FormEngine.backPath + FormEngine.browserUrl + '&mode=' + mode + '&bparams=' + params;
+               var url = FormEngine.browserUrl + '&mode=' + mode + '&bparams=' + params;
                width = width ? width : top.TYPO3.configuration.PopupWindow.width;
                height = height ? height : top.TYPO3.configuration.PopupWindow.height;
                FormEngine.openedPopupWindow = window.open(url, 'Typo3WinBrowser', 'height=' + height + ',width=' + width + ',status=0,menubar=0,resizable=1,scrollbars=1');
index 9107512..2eaa3ae 100644 (file)
@@ -38,8 +38,6 @@ var TBE_EDITOR = {
        formnameUENC: '',
        isChanged: 0,
 
-       backPath: '',
-
        isPalettedoc: null,
        doSaveFieldName: 0,
 
index c649109..84df70a 100644 (file)
@@ -167,7 +167,6 @@ class ResourceCompressor
             if (!empty($fileOptions['excludeFromConcatenation'])) {
                 continue;
             }
-            // we remove BACK_PATH from $filename, so make it relative to root path
             $filenameFromMainDir = $this->getFilenameFromMainDir($fileOptions['file']);
             // if $options['baseDirectories'] set, we only include files below these directories
             if (
@@ -229,7 +228,6 @@ class ResourceCompressor
             if (!isset($filesToInclude[$fileOptions['section']])) {
                 $filesToInclude[$fileOptions['section']] = array();
             }
-            // we remove BACK_PATH from $filename, so make it relative to root path
             $filenameFromMainDir = $this->getFilenameFromMainDir($fileOptions['file']);
             if ($fileOptions['forceOnTop']) {
                 array_unshift($filesToInclude[$fileOptions['section']], $filenameFromMainDir);
index 55bde12..bf8c1a1 100644 (file)
@@ -53,18 +53,12 @@ class ConfigurationForm extends ExtendedTemplateService
     protected $ext_realValues = array();
 
     /**
-     * @var string
-     */
-    protected $ext_backPath = '';
-
-    /**
      * @param string $configTemplate
      * @param string $pathRel PathRel is the path relative to the typo3/ directory
      * @param string $pathAbs PathAbs is the absolute path from root
-     * @param string $backPath BackPath is the backReference from current position to typo3/ dir
      * @return array
      */
-    public function ext_initTSstyleConfig($configTemplate, $pathRel, $pathAbs, $backPath = '')
+    public function ext_initTSstyleConfig($configTemplate, $pathRel, $pathAbs)
     {
         // Do not log time-performance information
         $this->tt_track = 0;
@@ -72,8 +66,7 @@ class ConfigurationForm extends ExtendedTemplateService
         // The editable constants are returned in an array.
         $theConstants = $this->generateConfig_constants();
         $this->ext_localGfxPrefix = $pathAbs;
-        $this->ext_localWebGfxPrefix = $backPath . $pathRel;
-        $this->ext_backPath = $backPath;
+        $this->ext_localWebGfxPrefix = $pathRel;
         return $theConstants;
     }
 
index 792e1db..8cfddc5 100644 (file)
@@ -391,8 +391,7 @@ class ResourceCompressorTest extends BaseTestCase
         // we have to fix relative paths, if we aren't working on a file in our target directory
         $relativeFilename = str_replace(PATH_site, '', $cssFile);
         if (strpos($relativeFilename, $this->subject->_get('targetDirectory')) === false) {
-            $filenameRelativeToMainDir = substr($relativeFilename, strlen($this->subject->_get('backPath')));
-            $compressedCss = $this->subject->_call('cssFixRelativeUrlPaths', $compressedCss, dirname($filenameRelativeToMainDir) . '/');
+            $compressedCss = $this->subject->_call('cssFixRelativeUrlPaths', $compressedCss, dirname($relativeFilename) . '/');
         }
         $this->assertEquals(file_get_contents($expected), $compressedCss, 'Group of file CSS assets optimized correctly.');
     }
index 8a851ca..edcb127 100644 (file)
@@ -216,8 +216,8 @@ class ExtDirectServer extends AbstractHandler
                     );
 
                     if ($configuration['type'] == 'group' && $configuration['internal_type'] == 'file') {
-                        $versionThumb = BackendUtility::thumbCode($versionRecord, $parameter->table, $fieldName, '');
-                        $liveThumb = BackendUtility::thumbCode($liveRecord, $parameter->table, $fieldName, '');
+                        $versionThumb = BackendUtility::thumbCode($versionRecord, $parameter->table, $fieldName);
+                        $liveThumb = BackendUtility::thumbCode($liveRecord, $parameter->table, $fieldName);
                         $diffReturnArray[] = array(
                             'field' => $fieldName,
                             'label' => $fieldTitle,