[TASK] Remove ModuleTemplate->spacer() method 83/44383/5
authorRuud Silvrants <ruudsilvrants@gmail.com>
Fri, 30 Oct 2015 09:25:43 +0000 (10:25 +0100)
committerJan Helke <typo3@helke.de>
Fri, 30 Oct 2015 09:37:55 +0000 (10:37 +0100)
The spacer helper method is desired to be removed. It is not mentioned
as deprecated as this is an internal method that was never available in
a released TYPO3 CMS version.

Change-Id: I57e22815a6d6e4a114f85645a24f9aed1b5b2550
Resolves: #71153
Releases: master
Reviewed-on: https://review.typo3.org/44383
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
typo3/sysext/backend/Classes/Template/ModuleTemplate.php
typo3/sysext/core/Classes/Database/QueryView.php
typo3/sysext/impexp/Classes/Controller/ImportExportController.php
typo3/sysext/scheduler/Classes/Controller/SchedulerModuleController.php

index 8b27080..bcb4d12 100644 (file)
@@ -626,26 +626,6 @@ class ModuleTemplate
     }
 
     /**
-     * Returns a blank <div>-section with a height
-     *
-     * @param int $dist Padding-top for the div-section
-     *
-     * @return string HTML content
-     * @internal
-     */
-    public function spacer($dist)
-    {
-        if ($dist > 0) {
-            return '
-
-       <!-- Spacer element -->
-       <div style="padding-top: ' . (int)$dist . 'px;"></div>
-';
-        }
-        return '';
-    }
-
-    /**
      * Make a section header.
      * Begins a section if not already open.
      *
index e6d0f37..ce602a1 100644 (file)
@@ -334,7 +334,7 @@ class QueryView
             if ($msg) {
                 $output .= '<br />' . $msg;
             }
-            $output .= $GLOBALS['SOBE']->getModuleTemplate()->spacer(20);
+            $output .= '<div style="padding-top: 20px;"></div>';
         }
         // Query Maker:
         $qGen = GeneralUtility::makeInstance(\TYPO3\CMS\Core\Database\QueryGenerator::class);
index 3163373..c01414e 100644 (file)
@@ -153,7 +153,7 @@ class ImportExportController extends BaseScriptClass
         // Input data grabbed:
         $inData = GeneralUtility::_GP('tx_impexp');
         $this->content .= $this->moduleTemplate->sectionHeader($this->lang->getLL('title_' . (string)$inData['action']));
-        $this->content .= $this->moduleTemplate->spacer(5);
+        $this->content .= '<div style="padding-top: 5px;"></div>';
         $this->checkUpload();
         switch ((string)$inData['action']) {
             case 'export':
index 49da2ae..b31f7da 100644 (file)
@@ -162,7 +162,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
         } else {
             // If no access, only display the module's title
             $this->content = '<h1>' . $this->getLanguageService()->getLL('title.') . '</h1>';
-            $this->content .= $this->moduleTemplate->spacer(5);
+            $this->content .='<div style="padding-top: 5px;"></div>';
         }
         $this->getButtons();
         $this->getModuleMenu();