[BUGFIX] Fix last line in table wizard 20/20920/2
authorAlexander Opitz <opitz.alexander@googlemail.com>
Fri, 3 May 2013 08:36:03 +0000 (10:36 +0200)
committerPhilipp Gampe <philipp.gampe@typo3.org>
Sun, 19 May 2013 14:27:05 +0000 (16:27 +0200)
In the table wizard (table ce) you have the possibility to move first line
to the last line and vice versa. But on last line the false icon and
input field is set. We also have a FIXME comment there.

With the patch we fix the problem and you can again move the last line on
top of the lines.

Resolves: #16531
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: I7fe69397ba595a6b2aec8cdfba05b286f3b8ebe4
Reviewed-on: https://review.typo3.org/20918
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-on: https://review.typo3.org/20920

typo3/sysext/backend/Classes/Controller/Wizard/TableController.php

index d713a30..c0f59ca 100644 (file)
@@ -278,6 +278,7 @@ class TableController {
                // Traverse the rows:
                $tRows = array();
                $k = 0;
+               $countLines = count($cfgArr);
                foreach ($cfgArr as $cellArr) {
                        if (is_array($cellArr)) {
                                // Initialize:
@@ -305,8 +306,7 @@ class TableController {
                                        $ctrl .= '<input type="image" name="TABLE[row_bottom][' . ($k + 1) * 2 . ']"' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->doc->backPath, 'gfx/turn_up.gif', '') . $onClick . ' title="' . $GLOBALS['LANG']->getLL('table_bottom', 1) . '" />' . $brTag;
                                }
                                $ctrl .= '<input type="image" name="TABLE[row_remove][' . ($k + 1) * 2 . ']"' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->doc->backPath, 'gfx/garbage.gif', '') . $onClick . ' title="' . $GLOBALS['LANG']->getLL('table_removeRow', 1) . '" />' . $brTag;
-                               // FIXME what is $tLines? See wizard_forms.php for the same.
-                               if ($k + 1 != count($tLines)) {
+                               if ($k + 1 != $countLines) {
                                        $ctrl .= '<input type="image" name="TABLE[row_down][' . ($k + 1) * 2 . ']"' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->doc->backPath, 'gfx/pil2down.gif', '') . $onClick . ' title="' . $GLOBALS['LANG']->getLL('table_down', 1) . '" />' . $brTag;
                                } else {
                                        $ctrl .= '<input type="image" name="TABLE[row_top][' . ($k + 1) * 2 . ']"' . \TYPO3\CMS\Backend\Utility\IconUtility::skinImg($this->doc->backPath, 'gfx/turn_down.gif', '') . $onClick . ' title="' . $GLOBALS['LANG']->getLL('table_top', 1) . '" />' . $brTag;