Fix order of cronjob start* fields in PIP GUI
authorMatthias Schmidt <gravatronics@live.com>
Sat, 17 Nov 2018 08:35:33 +0000 (09:35 +0100)
committerMatthias Schmidt <gravatronics@live.com>
Sat, 17 Nov 2018 08:35:33 +0000 (09:35 +0100)
See #2545

wcfsetup/install/files/lib/system/package/plugin/CronjobPackageInstallationPlugin.class.php

index 4425dd778f96b232670b0d800055bfd2885392d3..db73e7ea4d4f5ab6c33d2218c6ee257bb021d097 100644 (file)
@@ -220,7 +220,7 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
                                ->value(true)
                ]);
                
-               foreach (['startDom', 'startDow', 'startHour', 'startMinute', 'startMonth'] as $timeProperty) {
+               foreach (['startMinute', 'startHour', 'startDom', 'startMonth', 'startDow'] as $timeProperty) {
                        $dataContainer->insertBefore(
                                TextFormField::create($timeProperty)
                                        ->objectProperty(strtolower($timeProperty))
@@ -408,11 +408,11 @@ class CronjobPackageInstallationPlugin extends AbstractXMLPackageInstallationPlu
                $this->appendElementChildren(
                        $cronjob,
                        [
-                               'startmonth',
+                               'startminute',
+                               'starthour',
                                'startdom',
+                               'startmonth',
                                'startdow',
-                               'starthour',
-                               'startminute',
                                'options' => '',
                                'canbeedited' => 1,
                                'canbedisabled' => 1,