From 109e70e761a6bb4a22992896f6f2e8ef729cf99d Mon Sep 17 00:00:00 2001 From: Matthias Schmidt Date: Sun, 21 Aug 2016 15:40:18 +0200 Subject: [PATCH] Add check if any users are affected in UserGroupAssignmentCronjob --- .../cronjob/UserGroupAssignmentCronjob.class.php | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/wcfsetup/install/files/lib/system/cronjob/UserGroupAssignmentCronjob.class.php b/wcfsetup/install/files/lib/system/cronjob/UserGroupAssignmentCronjob.class.php index 89b8995900..d8bc124561 100644 --- a/wcfsetup/install/files/lib/system/cronjob/UserGroupAssignmentCronjob.class.php +++ b/wcfsetup/install/files/lib/system/cronjob/UserGroupAssignmentCronjob.class.php @@ -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 * @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(); + } } } } -- 2.20.1