[TASK] Remove DocumentTemplate->spacer() and mark as deprecated 82/44382/3
authorRuud Silvrants <ruud@beech.it>
Fri, 30 Oct 2015 09:08:31 +0000 (10:08 +0100)
committerMorton Jonuschat <m.jonuschat@mojocode.de>
Fri, 30 Oct 2015 09:33:58 +0000 (10:33 +0100)
Resolves: #71153
Releases: master
Change-Id: I1ee8ab5c905b6e8324755e28785b85cde8c76e5b
Reviewed-on: https://review.typo3.org/44382
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
typo3/sysext/backend/Classes/Template/DocumentTemplate.php
typo3/sysext/compatibility6/Classes/Controller/WebFunctionWizardsBaseController.php
typo3/sysext/core/Documentation/Changelog/master/Deprecation-71153-DocumentTemplateSpacer.rst [new file with mode: 0644]
typo3/sysext/recordlist/Classes/Browser/ElementBrowser.php
typo3/sysext/recordlist/Classes/Browser/FileBrowser.php
typo3/sysext/tstemplate/Classes/Controller/TemplateAnalyzerModuleFunctionController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateConstantEditorModuleFunctionController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateInformationModuleFunctionController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateObjectBrowserModuleFunctionController.php

index d87497f..6b657fd 100644 (file)
@@ -998,9 +998,11 @@ function jumpToUrl(URL) {
      *
      * @param int $dist Padding-top for the div-section (should be margin-top but konqueror (3.1) doesn't like it :-(
      * @return string HTML content
+     * @deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
      */
     public function spacer($dist)
     {
+        GeneralUtility::logDeprecatedFunction();
         if ($dist > 0) {
             return '
 
index 33f5d32..6a6145a 100644 (file)
@@ -80,7 +80,7 @@ class WebFunctionWizardsBaseController extends \TYPO3\CMS\Backend\Module\Abstrac
         if (!empty($menu)) {
             $menu = $GLOBALS['LANG']->getLL('wiz_lWizards', true) . ': ' . $menu;
             $content = $this->pObj->doc->section('', '<span class="text-nowrap">' . $menu . '</span>');
-            $content .= $this->pObj->doc->spacer(20);
+            $content .= '<div style="padding-top: 20px;"></div>';
         }
 
         $content .= $this->extObjContent();
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Deprecation-71153-DocumentTemplateSpacer.rst b/typo3/sysext/core/Documentation/Changelog/master/Deprecation-71153-DocumentTemplateSpacer.rst
new file mode 100644 (file)
index 0000000..7315ee1
--- /dev/null
@@ -0,0 +1,20 @@
+================================================
+Deprecation: #71153 - DocumentTemplate->spacer()
+================================================
+
+Description
+===========
+
+Method ``TYPO3\CMS\Backend\Template\DocumentTemplate::spacer()`` has been deprecated.
+
+
+Affected Installations
+======================
+
+Instances with custom backend modules that use this method.
+
+
+Migration
+=========
+
+Add the needed margin as HTML / CSS.
index ac82d42..3aacf84 100755 (executable)
@@ -2575,7 +2575,7 @@ class ElementBrowser
         $labelImportSelection = $lang->sL('LLL:EXT:lang/locallang_browse_links.xlf:importSelection', true);
         // Getting flag for showing/not showing thumbnails:
         $noThumbsInEB = $this->getBackendUser()->getTSConfigVal('options.noThumbsInEB');
-        $out = $this->doc->spacer(10) . '<div>' . '<a href="#" onclick="BrowseLinks.Selector.handle()"'
+        $out = '<div style="padding-top:10px;">' . '<a href="#" onclick="BrowseLinks.Selector.handle()"'
             . 'title="' . $labelImportSelection . '">'
             . $this->iconFactory->getIcon('actions-document-import-t3d', Icon::SIZE_SMALL)->render()
             . $labelImportSelection . '</a>&nbsp;&nbsp;&nbsp;'
@@ -2590,12 +2590,12 @@ class ElementBrowser
             $addParams = '&act=' . $this->act . '&mode=' . $this->mode
                 . '&expandFolder=' . rawurlencode($this->selectedFolder->getCombinedIdentifier())
                 . '&bparams=' . rawurlencode($this->bparams);
-            $thumbNailCheck = '<div class="checkbox"><label for="checkDisplayThumbs">' . BackendUtility::getFuncCheck('', 'SET[displayThumbs]', $_MOD_SETTINGS['displayThumbs'],
+            $thumbNailCheck = '<div class="checkbox" style="padding:5px 0 15px 0"><label for="checkDisplayThumbs">' . BackendUtility::getFuncCheck('', 'SET[displayThumbs]', $_MOD_SETTINGS['displayThumbs'],
                     GeneralUtility::_GP('M') ? '' : $this->thisScript, $addParams, 'id="checkDisplayThumbs"')
                 . $lang->sL('LLL:EXT:lang/locallang_mod_file_list.xlf:displayThumbs', true) . '</label></div>';
-            $out .= $this->doc->spacer(5) . $thumbNailCheck . $this->doc->spacer(15);
+            $out .= $thumbNailCheck;
         } else {
-            $out .= $this->doc->spacer(15);
+            $out .= '<div style="padding-top: 15px;"></div>';
         }
         return $out;
     }
@@ -2611,7 +2611,7 @@ class ElementBrowser
             . '&bparams=' . rawurlencode($this->bparams)
             . (is_array($this->P) ? GeneralUtility::implodeArrayForUrl('P', $this->P) : '');
         $out = '
-                       <form method="post" action="' . htmlspecialchars($action) . '">
+                       <form method="post" action="' . htmlspecialchars($action) . '" style="padding-bottom: 15px;">
                                <div class="input-group">
                                        <input class="form-control" type="text" name="searchWord" value="' . htmlspecialchars($this->searchWord) . '">
                                        <span class="input-group-btn">
@@ -2619,7 +2619,6 @@ class ElementBrowser
                                        </span>
                                </div>
                        </form>';
-        $out .= $this->doc->spacer(15);
         return $out;
     }
 
index 0cbf68c..d8c98d6 100644 (file)
@@ -385,7 +385,7 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
         $labelToggleSelection = $lang->sL('LLL:EXT:lang/locallang_browse_links.xlf:toggleSelection', true);
         $labelImportSelection = $lang->sL('LLL:EXT:lang/locallang_browse_links.xlf:importSelection', true);
 
-        $out = $this->doc->spacer(10) . '<div>' . '<a href="#" id="t3-js-importSelection" title="' . $labelImportSelection . '">'
+        $out = '<div style="padding-top:10px;">' . '<a href="#" id="t3-js-importSelection" title="' . $labelImportSelection . '">'
             . $this->iconFactory->getIcon('actions-document-import-t3d', Icon::SIZE_SMALL)
             . $labelImportSelection . '</a>&nbsp;&nbsp;&nbsp;'
             . '<a href="#" id="t3-js-toggleSelection" title="' . $labelToggleSelection . '">'
@@ -400,7 +400,7 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
             $_MCONF['name'] = 'file_list';
             $_MOD_SETTINGS = BackendUtility::getModuleData($_MOD_MENU, GeneralUtility::_GP('SET'), $_MCONF['name']);
             $addParams = GeneralUtility::implodeArrayForUrl('', $this->getUrlParameters(['identifier' => $this->selectedFolder->getCombinedIdentifier()]));
-            $thumbNailCheck = '<div class="checkbox"><label for="checkDisplayThumbs">'
+            $thumbNailCheck = '<div class="checkbox" style="padding:5px 0 15px 0"><label for="checkDisplayThumbs">'
                 . BackendUtility::getFuncCheck(
                     '',
                     'SET[displayThumbs]',
@@ -410,9 +410,9 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
                     'id="checkDisplayThumbs"'
                 )
                 . $lang->sL('LLL:EXT:lang/locallang_mod_file_list.xlf:displayThumbs', true) . '</label></div>';
-            $out .= $this->doc->spacer(5) . $thumbNailCheck . $this->doc->spacer(15);
+            $out .= $thumbNailCheck;
         } else {
-            $out .= $this->doc->spacer(15);
+            $out .= '<div style="padding-top: 15px;"></div>';
         }
         return $out;
     }
@@ -426,7 +426,7 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
     {
         $action = $this->getScriptUrl() . GeneralUtility::implodeArrayForUrl('', $this->getUrlParameters([]));
         $out = '
-                       <form method="post" action="' . htmlspecialchars($action) . '">
+                       <form method="post" action="' . htmlspecialchars($action) . '" style="padding-bottom: 15px;">
                                <div class="input-group">
                                        <input class="form-control" type="text" name="searchWord" value="' . htmlspecialchars($this->searchWord) . '">
                                        <span class="input-group-btn">
@@ -434,7 +434,6 @@ class FileBrowser extends AbstractElementBrowser implements ElementBrowserInterf
                                        </span>
                                </div>
                        </form>';
-        $out .= $this->doc->spacer(15);
         return $out;
     }
 
index e4ccaf3..f5337ea 100644 (file)
@@ -142,7 +142,7 @@ class TemplateAnalyzerModuleFunctionController extends AbstractFunctionModule
         $head .= '</tr></thead>';
         $hierar = implode(array_reverse($templateService->ext_getTemplateHierarchyArr($hierarchyInfo, '', array(), 1)), '');
         $hierar = '<div class="table-fit"><table class="table table-striped table-hover" id="ts-analyzer">' . $head . $hierar . '</table></div>';
-        $theOutput .= $this->pObj->doc->spacer(5);
+        $theOutput .= '<div style="padding-top: 5px;"></div>';
         $theOutput .= $this->pObj->doc->section($lang->getLL('templateHierarchy', true), $hierar, 0, 1);
         $urlParameters = array(
             'id' => $GLOBALS['SOBE']->id,
@@ -151,9 +151,9 @@ class TemplateAnalyzerModuleFunctionController extends AbstractFunctionModule
         $aHref = BackendUtility::getModuleUrl('web_ts', $urlParameters);
 
         $completeLink = '<p><a href="' . htmlspecialchars($aHref) . '" class="btn btn-default">' . $lang->getLL('viewCompleteTS', true) . '</a></p>';
-        $theOutput .= $this->pObj->doc->spacer(5);
+        $theOutput .= '<div style="padding-top: 5px;"></div>';
         $theOutput .= $this->pObj->doc->section($lang->getLL('completeTS', true), $completeLink, 0, 1);
-        $theOutput .= $this->pObj->doc->spacer(15);
+        $theOutput .= '<div style="padding-top: 15px;"></div>';
         // Output options
         $theOutput .= $this->pObj->doc->section($lang->getLL('displayOptions', true), '', false, true);
 
@@ -180,7 +180,7 @@ class TemplateAnalyzerModuleFunctionController extends AbstractFunctionModule
                 '</label></div>';
         }
         $theOutput .=  '</div>';
-        $theOutput .= $this->pObj->doc->spacer(25);
+        $theOutput .= '<div style="padding-top: 25px;"></div>';
 
         if ($template) {
             // Output Constants
@@ -206,7 +206,7 @@ class TemplateAnalyzerModuleFunctionController extends AbstractFunctionModule
             }
 
             // Output Setup
-            $theOutput .= $this->pObj->doc->spacer(15);
+            $theOutput .= '<div style="padding-top: 15px;"></div>';
             $theOutput .= $this->pObj->doc->section($lang->getLL('setup', true), '', 0, 1);
             $theOutput .= $this->pObj->doc->sectionEnd();
             $templateService->ext_lineNumberOffset = 0;
index 1e54dec..307ffc4 100644 (file)
@@ -92,7 +92,7 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends AbstractF
     {
         $templateService = $this->getExtendedTemplateService();
         if ($templateService->helpConfig['imagetag'] || $templateService->helpConfig['description'] || $templateService->helpConfig['header']) {
-            $theOutput .= $this->pObj->doc->spacer(30);
+            $theOutput .= '<div style="padding-top: 30px;"></div>';
             $theOutput .= $this->pObj->doc->section($templateService->helpConfig['header'], '<div align="center">' . $templateService->helpConfig['imagetag'] . '</div><BR>' . ($templateService->helpConfig['description'] ? implode(explode('//', $templateService->helpConfig['description']), '<BR>') . '<BR>' : '') . ($templateService->helpConfig['bulletlist'] ? '<ul><li>' . implode(explode('//', $templateService->helpConfig['bulletlist']), '<li>') . '</ul>' : '<BR>'));
         }
         return $theOutput;
@@ -160,7 +160,7 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends AbstractF
             if ($manyTemplatesMenu) {
                 $theOutput .= $this->pObj->doc->section('', $manyTemplatesMenu);
             }
-            $theOutput .= $this->pObj->doc->spacer(10);
+            $theOutput .= '<div style="padding-top: 10px;"></div>';
             if (!empty($this->pObj->MOD_MENU['constant_editor_cat'])) {
                 $menu = '<div class="form-inline form-inline-spaced">';
                 $menu .= BackendUtility::getDropdownMenu($this->pObj->id, 'SET[constant_editor_cat]', $this->pObj->MOD_SETTINGS['constant_editor_cat'], $this->pObj->MOD_MENU['constant_editor_cat']);
@@ -169,7 +169,7 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends AbstractF
             } else {
                 $theOutput .= $this->pObj->doc->section($lang->getLL('noConstants', true), $lang->getLL('noConstantsDescription', true), false, false, 1);
             }
-            $theOutput .= $this->pObj->doc->spacer(15);
+            $theOutput .= '<div style="padding-top: 15px;"></div>';
             // Category and constant editor config:
             $category = $this->pObj->MOD_SETTINGS['constant_editor_cat'];
             $templateService->ext_getTSCE_config($category);
index bc6db69..53df4a0 100644 (file)
@@ -262,7 +262,7 @@ class TypoScriptTemplateInformationModuleFunctionController extends AbstractFunc
             if ($manyTemplatesMenu) {
                 $theOutput .= $this->pObj->doc->section('', $manyTemplatesMenu);
             }
-            $theOutput .= $this->pObj->doc->spacer(10);
+            $theOutput .= '<div style="padding-top: 10px;"></div>';
             $numberOfRows = 35;
             // If abort pressed, nothing should be edited:
             if (isset($POST['_saveandclosedok'])) {
@@ -274,7 +274,7 @@ class TypoScriptTemplateInformationModuleFunctionController extends AbstractFunc
                 // Display "Include TypoScript file content?" checkbox
                 $outCode .= '<div class="checkbox"><label for="checkIncludeTypoScriptFileContent">' . BackendUtility::getFuncCheck($this->pObj->id, 'SET[includeTypoScriptFileContent]', $this->pObj->MOD_SETTINGS['includeTypoScriptFileContent'], '', '&e[constants]=1', 'id="checkIncludeTypoScriptFileContent"');
                 $outCode .= $lang->getLL('includeTypoScriptFileContent') . '</label></div><br />';
-                $theOutput .= $this->pObj->doc->spacer(15);
+                $theOutput .= '<div style="padding-top: 15px;"></div>';
                 $theOutput .= $this->pObj->doc->section($lang->getLL('constants'), '', true);
                 $theOutput .= $this->pObj->doc->sectionEnd() . $outCode;
             }
@@ -284,7 +284,7 @@ class TypoScriptTemplateInformationModuleFunctionController extends AbstractFunc
                 // Display "Include TypoScript file content?" checkbox
                 $outCode .= '<div class="checkbox"><label for="checkIncludeTypoScriptFileContent">' . BackendUtility::getFuncCheck($this->pObj->id, 'SET[includeTypoScriptFileContent]', $this->pObj->MOD_SETTINGS['includeTypoScriptFileContent'], '', '&e[config]=1', 'id="checkIncludeTypoScriptFileContent"');
                 $outCode .= $lang->getLL('includeTypoScriptFileContent') . '</label></div><br />';
-                $theOutput .= $this->pObj->doc->spacer(15);
+                $theOutput .= '<div style="padding-top: 15px;"></div>';
                 $theOutput .= $this->pObj->doc->section($lang->getLL('setup'), '', true);
                 $theOutput .= $this->pObj->doc->sectionEnd() . $outCode;
             }
index 044f971..8dd7f53 100644 (file)
@@ -234,7 +234,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
             if ($manyTemplatesMenu) {
                 $theOutput .= $this->pObj->doc->section('', $manyTemplatesMenu);
             }
-            $theOutput .= $this->pObj->doc->spacer(10);
+            $theOutput .= '<div style="padding-top: 10px;"></div>';
             if ($POST['add_property'] || $POST['update_value'] || $POST['clear_object']) {
                 // add property
                 $line = '';
@@ -352,7 +352,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                 $out .= '      <input class="form-control" type="text" name="data[' . htmlspecialchars($this->pObj->sObj) . '][propertyValue]"' . $documentTemplate->formWidth(40) . ' />';
                 $out .= '      <input class="btn btn-default" type="submit" name="add_property" value="' . $lang->getLL('addButton') . '" />';
                 $out .= '</div>';
-                $theOutput .= $this->pObj->doc->spacer(20);
+                $theOutput .= '<div style="padding-top: 20px;"></div>';
                 $theOutput .= $this->pObj->doc->section($lang->getLL('addProperty'), $out, 0, 0);
                 // clear
                 $out = '<div class="form-group">';
@@ -364,16 +364,16 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                 $out .= '              <input class="btn btn-default" type="submit" name="clear_object" value="' . $lang->getLL('clearButton') . '" />';
                 $out .= '      </div>';
                 $out .= '</div>';
-                $theOutput .= $this->pObj->doc->spacer(20);
+                $theOutput .='<div style="padding-top: 20px;"></div>';
                 $theOutput .= $this->pObj->doc->section($lang->getLL('clearObject'), $out, 0, 0);
-                $theOutput .= $this->pObj->doc->spacer(10);
+                $theOutput .= '<div style="padding-top: 10px;"></div>';
                 // Inline Form Area End
                 $theOutput .= '</div>';
             } else {
                 $noTemplateMessage = GeneralUtility::makeInstance(FlashMessage::class, $lang->getLL('noCurrentTemplate'), $lang->getLL('edit'), FlashMessage::ERROR);
                 $this->addFlashMessage($noTemplateMessage);
                 $theOutput .= htmlspecialchars($this->pObj->sObj) . ' = <strong>' . htmlspecialchars($theSetupValue) . '</strong>';
-                $theOutput .= $this->pObj->doc->spacer(10);
+                $theOutput .= '<div style="padding-top: 10px;"></div>';
             }
             // Links:
             $out = '';
@@ -457,7 +457,7 @@ class TypoScriptTemplateObjectBrowserModuleFunctionController extends AbstractFu
                     $errorLink = ' <a href="' . htmlspecialchars(($aHref . '&SET[function]=TYPO3\\CMS\\Tstemplate\\Controller\\TemplateAnalyzerModuleFunctionController&template=all&SET[ts_analyzer_checkLinenum]=1#line-' . $inf[2])) . '">' . $lang->getLL('errorShowDetails') . '</a>';
                     $errMsg[] = $inf[1] . ': &nbsp; &nbsp;' . $inf[0] . $errorLink;
                 }
-                $theOutput .= $this->pObj->doc->spacer(10);
+                $theOutput .= '<div style="padding-top: 10px;"></div>';
 
                 $title = $lang->getLL('errorsWarnings');
                 $message = '<p>' . implode($errMsg, '<br />') . '</p>';