[BUGFIX] Fix invalid button group markup in scheduler 65/51365/2
authorAndreas Fernandez <a.fernandez@scripting-base.de>
Thu, 19 Jan 2017 20:56:29 +0000 (21:56 +0100)
committerWouter Wolters <typo3@wouterwolters.nl>
Mon, 23 Jan 2017 09:19:19 +0000 (10:19 +0100)
An element with the class `btn-group` must not also have the class `btn`.
Such occurences are fixed in this patch.

Resolves: #79395
Related: #78199
Releases: master
Change-Id: I4793afc984936c2fc7226feac24f03b6ec94794f
Reviewed-on: https://review.typo3.org/51365
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Riccardo De Contardi <erredeco@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/scheduler/Classes/Controller/SchedulerModuleController.php

index d9a7140..60bb28e 100644 (file)
@@ -1045,7 +1045,7 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
                     // Define checkbox
                     $startExecutionElement = '<label class="btn btn-default btn-checkbox"><input type="checkbox" name="tx_scheduler[execute][]" value="' . $schedulerRecord['uid'] . '" id="task_' . $schedulerRecord['uid'] . '"><span class="t3-icon fa"></span></label>';
 
-                    $actions = '<div class="btn btn-group" role="group">' . $editAction . $toggleHiddenAction . $deleteAction . '</div>';
+                    $actions = '<div class="btn-group" role="group">' . $editAction . $toggleHiddenAction . $deleteAction . '</div>';
 
                     // Check the disable status
                     // Row is shown dimmed if task is disabled, unless it is still running
@@ -1059,9 +1059,9 @@ class SchedulerModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClas
 
                     // Show no action links (edit, delete) if task is running
                     if ($isRunning) {
-                        $actions = '<div class="btn btn-group" role="group">' . $stopAction . '</div>';
+                        $actions = '<div class="btn-group" role="group">' . $stopAction . '</div>';
                     } else {
-                        $actions .= '<div class="btn btn-group" role="group">' . $runAction . '</div>';
+                        $actions .= '&nbsp;<div class="btn-group" role="group">' . $runAction . '</div>';
                     }
 
                     // Check if the last run failed