[!!!][TASK] Remove alternateBgColors option 31/33831/4
authorWouter Wolters <typo3@wouterwolters.nl>
Wed, 5 Nov 2014 18:51:57 +0000 (19:51 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Thu, 6 Nov 2014 08:33:30 +0000 (09:33 +0100)
Resolves: #53658
Releases: master
Change-Id: If36c17741d8f3a9a07f7cc8ad2f3289164b8b195
Reviewed-on: http://review.typo3.org/33831
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Stefan Froemken <froemken@gmail.com>
Tested-by: Stefan Froemken <froemken@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/core/Documentation/Changelog/master/Breaking-53658-RemoveAlternateBgColorsOption.rst [new file with mode: 0644]
typo3/sysext/fluid/Classes/ViewHelpers/Be/TableListViewHelper.php
typo3/sysext/recordlist/Classes/RecordList.php
typo3/sysext/recordlist/Classes/RecordList/DatabaseRecordList.php
typo3/sysext/t3editor/res/jslib/parse_typoscript/tokenizetyposcript.js
typo3/sysext/t3skin/Resources/Private/Styles/TYPO3/structure/_module_web_list.less
typo3/sysext/t3skin/Resources/Public/Css/visual/t3kin.css

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-53658-RemoveAlternateBgColorsOption.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-53658-RemoveAlternateBgColorsOption.rst
new file mode 100644 (file)
index 0000000..5f59172
--- /dev/null
@@ -0,0 +1,27 @@
+===================================================
+Breaking: #53568 - option alternateBgColors removed
+===================================================
+
+Description
+===========
+
+The PageTSConfig option mod.web_list.alternateBgColors is removed without substitution.
+
+
+Impact
+======
+
+Extensions that extend the DatabaseRecordList and are using the property alternateBgColors
+
+The option in the TableListViewHelper has been deprecated and will be removed in TYPO3 CMS 8.
+
+Affected installations
+======================
+
+A TYPO3 instance is affected if a 3rd party extension uses the removed property.
+
+
+Migration
+=========
+
+Remove the call to the removed property.
\ No newline at end of file
index f9ebdb7..b72452e 100644 (file)
@@ -37,7 +37,7 @@ namespace TYPO3\CMS\Fluid\ViewHelpers\Be;
  * </output>
  *
  * <code title="Full">
- * <f:be.tableList tableName="fe_users" fieldList="{0: 'name', 1: 'email'}" storagePid="1" levels="2" filter='foo' recordsPerPage="10" sortField="name" sortDescending="true" readOnly="true" enableClickMenu="false" clickTitleMode="info" alternateBackgroundColors="true" />
+ * <f:be.tableList tableName="fe_users" fieldList="{0: 'name', 1: 'email'}" storagePid="1" levels="2" filter='foo' recordsPerPage="10" sortField="name" sortDescending="true" readOnly="true" enableClickMenu="false" clickTitleMode="info" />
  * </code>
  * <output>
  * List of "Website user" records with a text property of "foo" stored on PID 1 and two levels down.
@@ -67,11 +67,17 @@ class TableListViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBacken
         * @param bool $readOnly if TRUE, the edit icons won't be shown. Otherwise edit icons will be shown, if the current BE user has edit rights for the specified table!
         * @param bool $enableClickMenu enables context menu
         * @param string $clickTitleMode one of "edit", "show" (only pages, tt_content), "info
-        * @param bool $alternateBackgroundColors if set, rows will have alternate background colors
+        * @param bool $alternateBackgroundColors Deprecated since TYPO3 CMS 7, will be removed in TYPO3 CMS 8
         * @return string the rendered record list
         * @see localRecordList
         */
        public function render($tableName, array $fieldList = array(), $storagePid = NULL, $levels = 0, $filter = '', $recordsPerPage = 0, $sortField = '', $sortDescending = FALSE, $readOnly = FALSE, $enableClickMenu = TRUE, $clickTitleMode = NULL, $alternateBackgroundColors = FALSE) {
+               if ($alternateBackgroundColors) {
+                       \TYPO3\CMS\Core\Utility\GeneralUtility::deprecationLog(
+                               'The option alternateBackgroundColors has no effect anymore and can be removed without problems. The parameter will be removed in TYPO3 CMS 8.'
+                       );
+               }
+
                $pageinfo = \TYPO3\CMS\Backend\Utility\BackendUtility::readPageAccess(\TYPO3\CMS\Core\Utility\GeneralUtility::_GP('id'), $GLOBALS['BE_USER']->getPagePermsClause(1));
                /** @var $dblist \TYPO3\CMS\Recordlist\RecordList\DatabaseRecordList */
                $dblist = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('TYPO3\\CMS\\Recordlist\\RecordList\\DatabaseRecordList');
@@ -83,7 +89,6 @@ class TableListViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Be\AbstractBacken
                $dblist->showClipboard = FALSE;
                $dblist->disableSingleTableView = TRUE;
                $dblist->clickTitleMode = $clickTitleMode;
-               $dblist->alternateBgColors = $alternateBackgroundColors;
                $dblist->clickMenuEnabled = $enableClickMenu;
                if ($storagePid === NULL) {
                        $frameworkConfiguration = $this->configurationManager->getConfiguration(\TYPO3\CMS\Extbase\Configuration\ConfigurationManagerInterface::CONFIGURATION_TYPE_FRAMEWORK);
index 6347ede..f69ed67 100644 (file)
@@ -206,7 +206,6 @@ class RecordList {
                $dblist->hideTables = $this->modTSconfig['properties']['hideTables'];
                $dblist->hideTranslations = $this->modTSconfig['properties']['hideTranslations'];
                $dblist->tableTSconfigOverTCA = $this->modTSconfig['properties']['table.'];
-               $dblist->alternateBgColors = $this->modTSconfig['properties']['alternateBgColors'] ? 1 : 0;
                $dblist->allowedNewTables = GeneralUtility::trimExplode(',', $this->modTSconfig['properties']['allowedNewTables'], TRUE);
                $dblist->deniedNewTables = GeneralUtility::trimExplode(',', $this->modTSconfig['properties']['deniedNewTables'], TRUE);
                $dblist->newWizards = $this->modTSconfig['properties']['newWizards'] ? 1 : 0;
index 694e2fb..6f3a1b7 100644 (file)
@@ -31,14 +31,6 @@ class DatabaseRecordList extends AbstractDatabaseRecordList {
        // *********
 
        /**
-        * If TRUE, table rows in the list will alternate in background colors (and have
-        * background colors at all!)
-        *
-        * @var bool
-        */
-       public $alternateBgColors = FALSE;
-
-       /**
         * Used to indicate which tables (values in the array) that can have a
         * create-new-record link. If the array is empty, all tables are allowed.
         *
@@ -706,12 +698,9 @@ class DatabaseRecordList extends AbstractDatabaseRecordList {
                        if ($cc == $this->totalRowCount || $cc == $this->iLimit) {
                                $rowSpecial .= ' lastcol';
                        }
-                       // Background color, if any:
-                       if ($this->alternateBgColors) {
-                               $row_bgColor = $cc % 2 ? ' class="db_list_normal' . $rowSpecial . '"' : ' class="db_list_alt' . $rowSpecial . '"';
-                       } else {
-                               $row_bgColor = ' class="db_list_normal' . $rowSpecial . '"';
-                       }
+
+                       $row_bgColor = ' class="db_list_normal' . $rowSpecial . '"';
+
                        // Overriding with versions background color if any:
                        $row_bgColor = $row['_CSSCLASS'] ? ' class="' . $row['_CSSCLASS'] . '"' : $row_bgColor;
                        // Incr. counter.
index 91ba2d1..6059dd5 100644 (file)
@@ -127,7 +127,6 @@ var typoscriptWords = {
        'allSaveFunctions': 'reserved',
        'allStdWrap': 'reserved',
        'allWrap': 'reserved',
-       'alternateBgColors': 'reserved',
        'alternativeSortingField': 'reserved',
        'alternativeTempPath': 'reserved',
        'altImgResource': 'reserved',
@@ -1452,7 +1451,6 @@ var typoscriptWords = {
        'allSaveFunctions': 'reserved',
        'allStdWrap': 'reserved',
        'allWrap': 'reserved',
-       'alternateBgColors': 'reserved',
        'alternativeSortingField': 'reserved',
        'alternativeTempPath': 'reserved',
        'altImgResource': 'reserved',
index c9465f9..ef426b4 100644 (file)
@@ -154,8 +154,8 @@ div#typo3-dblist-pagination span.bar {
        padding: 0 2px;
 }
 
-.db_list_normal td.col-control, .db_list_alt td.col-control,
-.db_list_normal td.col-clipboard, .db_list_alt td.col-clipboard {
+.db_list_normal td.col-control,
+.db_list_normal td.col-clipboard {
        padding-left: 3px;
        padding-right: 0px;
 }
index 2ec0db0..bcac925 100644 (file)
@@ -8863,14 +8863,14 @@ table.t3-table > tfoot > tr.success > td,
 table.t3-table > thead > tr.success > th,
 table.t3-table > tbody > tr.success > th,
 table.t3-table > tfoot > tr.success > th {
-  background-color: #dff0d8;
+  background-color: #c7f6d1;
 }
 table.t3-table > tbody > tr > td.success:hover,
 table.t3-table > tbody > tr > th.success:hover,
 table.t3-table > tbody > tr.success:hover > td,
 table.t3-table > tbody > tr:hover > .success,
 table.t3-table > tbody > tr.success:hover > th {
-  background-color: #d0e9c6;
+  background-color: #b1f2c0;
 }
 table.t3-table > thead > tr > td.info,
 table.t3-table > tbody > tr > td.info,
@@ -8884,14 +8884,14 @@ table.t3-table > tfoot > tr.info > td,
 table.t3-table > thead > tr.info > th,
 table.t3-table > tbody > tr.info > th,
 table.t3-table > tfoot > tr.info > th {
-  background-color: #d9edf7;
+  background-color: #d8f0f8;
 }
 table.t3-table > tbody > tr > td.info:hover,
 table.t3-table > tbody > tr > th.info:hover,
 table.t3-table > tbody > tr.info:hover > td,
 table.t3-table > tbody > tr:hover > .info,
 table.t3-table > tbody > tr.info:hover > th {
-  background-color: #c4e3f3;
+  background-color: #c2e7f4;
 }
 table.t3-table > thead > tr > td.warning,
 table.t3-table > tbody > tr > td.warning,
@@ -8905,14 +8905,14 @@ table.t3-table > tfoot > tr.warning > td,
 table.t3-table > thead > tr.warning > th,
 table.t3-table > tbody > tr.warning > th,
 table.t3-table > tfoot > tr.warning > th {
-  background-color: #fcf8e3;
+  background-color: #fbd8bc;
 }
 table.t3-table > tbody > tr > td.warning:hover,
 table.t3-table > tbody > tr > th.warning:hover,
 table.t3-table > tbody > tr.warning:hover > td,
 table.t3-table > tbody > tr:hover > .warning,
 table.t3-table > tbody > tr.warning:hover > th {
-  background-color: #faf2cc;
+  background-color: #f9cba4;
 }
 table.t3-table > thead > tr > td.danger,
 table.t3-table > tbody > tr > td.danger,
@@ -8926,14 +8926,14 @@ table.t3-table > tfoot > tr.danger > td,
 table.t3-table > thead > tr.danger > th,
 table.t3-table > tbody > tr.danger > th,
 table.t3-table > tfoot > tr.danger > th {
-  background-color: #f2dede;
+  background-color: #efc7c7;
 }
 table.t3-table > tbody > tr > td.danger:hover,
 table.t3-table > tbody > tr > th.danger:hover,
 table.t3-table > tbody > tr.danger:hover > td,
 table.t3-table > tbody > tr:hover > .danger,
 table.t3-table > tbody > tr.danger:hover > th {
-  background-color: #ebcccc;
+  background-color: #eab3b3;
 }
 /**
  * This file is part of the TYPO3 CMS project.
@@ -11736,9 +11736,7 @@ div#typo3-dblist-pagination span.bar {
   padding: 0 2px;
 }
 .db_list_normal td.col-control,
-.db_list_alt td.col-control,
-.db_list_normal td.col-clipboard,
-.db_list_alt td.col-clipboard {
+.db_list_normal td.col-clipboard {
   padding-left: 3px;
   padding-right: 0px;
 }