#8374 Missing bool return tasks updated

classes/task/DepositDois.php

@@ -42,5 +42,7 @@ protected function executeActions()
         foreach ($contextIds as $contextId) {
             dispatch(new DepositContext($contextId));
         }
+
+        return true;
     }
 }

classes/task/EditorialReminders.php

@@ -64,5 +64,7 @@ protected function executeActions()
                 ScheduledTaskHelper::SCHEDULED_TASK_MESSAGE_TYPE_NOTICE
             );
         }
+
+        return true;
     }
 }

classes/task/RemoveUnvalidatedExpiredUsers.php

@@ -36,10 +36,15 @@ public function getName()
      */
     public function executeActions()
     {
+        // No need to remove invalidated users if validation requirement is turned off
+        if ( !Config::getVar('email', 'require_validation', false) ) {
+            return true;
+        }
+
         $validationMaxDeadlineInDays = (int) Config::getVar('general', 'user_validation_period');
 
         if ($validationMaxDeadlineInDays <= 0) {
-            return;
+            return true;
         }
 
         $dateTillValid = Carbon::now()->startOfDay()->subDays($validationMaxDeadlineInDays);