Do not display an error message if the cronjob is already disabled/enabled and this status is to be changed to that status.

Author: Cyperghost

wcfsetup/install/files/lib/system/endpoint/controller/core/cronjobs/DisableCronjob.class.php

@@ -31,7 +31,9 @@ public function __invoke(ServerRequestInterface $request, array $variables): Res
 
         $this->assertCronjobCanBeDisabled($cronjob);
 
-        (new CronjobAction([$cronjob], 'toggle'))->executeAction();
+        if (!$cronjob->isDisabled) {
+            (new CronjobAction([$cronjob], 'toggle'))->executeAction();
+        }
 
         return new JsonResponse([]);
     }
@@ -40,11 +42,7 @@ private function assertCronjobCanBeDisabled(Cronjob $cronjob): void
     {
         WCF::getSession()->checkPermissions(['admin.management.canManageCronjob']);
 
-        if ($cronjob->canBeDisabled()) {
-            throw new PermissionDeniedException();
-        }
-
-        if ($cronjob->isDisabled) {
+        if (!$cronjob->canBeDisabled()) {
             throw new PermissionDeniedException();
         }
     }

wcfsetup/install/files/lib/system/endpoint/controller/core/cronjobs/EnableCronjob.class.php

@@ -31,7 +31,9 @@ public function __invoke(ServerRequestInterface $request, array $variables): Res
 
         $this->assertCronjobCanBeEnabled($cronjob);
 
-        (new CronjobAction([$cronjob], 'toggle'))->executeAction();
+        if ($cronjob->isDisabled) {
+            (new CronjobAction([$cronjob], 'toggle'))->executeAction();
+        }
 
         return new JsonResponse([]);
     }
@@ -40,11 +42,7 @@ private function assertCronjobCanBeEnabled(Cronjob $cronjob): void
     {
         WCF::getSession()->checkPermissions(['admin.management.canManageCronjob']);
 
-        if ($cronjob->canBeDisabled()) {
-            throw new PermissionDeniedException();
-        }
-
-        if (!$cronjob->isDisabled) {
+        if (!$cronjob->canBeDisabled()) {
             throw new PermissionDeniedException();
         }
     }