Add check if any users are affected in UserGroupAssignmentCronjob
authorMatthias Schmidt <gravatronics@live.com>
Sun, 21 Aug 2016 13:40:18 +0000 (15:40 +0200)
committerMatthias Schmidt <gravatronics@live.com>
Sun, 21 Aug 2016 13:40:18 +0000 (15:40 +0200)
wcfsetup/install/files/lib/system/cronjob/UserGroupAssignmentCronjob.class.php

index 89b899590096aa891e66cf7ce46a86f607c226e9..d8bc124561c56133b39d52424c917402748ce907 100644 (file)
@@ -9,7 +9,7 @@ use wcf\system\user\group\assignment\UserGroupAssignmentHandler;
  * Executes automatic user group assignments.
  * 
  * @author     Matthias Schmidt
- * @copyright  2001-2015 WoltLab GmbH
+ * @copyright  2001-2016 WoltLab GmbH
  * @license    GNU Lesser General Public License <http://opensource.org/licenses/lgpl-license.php>
  * @package    com.woltlab.wcf
  * @subpackage system.cronjob
@@ -33,12 +33,14 @@ class UserGroupAssignmentCronjob extends AbstractCronjob {
                }
                
                foreach ($usersToGroup as $groupID => $users) {
-                       $userAction = new UserAction(array_unique($users), 'addToGroups', array(
-                               'addDefaultGroups' => false,
-                               'deleteOldGroups' => false,
-                               'groups' => array($groupID)
-                       ));
-                       $userAction->executeAction();
+                       if (!empty($users)) {
+                               $userAction = new UserAction(array_unique($users), 'addToGroups', array(
+                                       'addDefaultGroups' => false,
+                                       'deleteOldGroups' => false,
+                                       'groups' => array($groupID)
+                               ));
+                               $userAction->executeAction();
+                       }
                }
        }
 }