From bff09daf03ad85a1164cb8cd099b389a99876bcd Mon Sep 17 00:00:00 2001 From: Matthias Schmidt Date: Tue, 27 Dec 2016 14:32:50 +0100 Subject: [PATCH] Fix MultiSelectOptionType::checkUser() --- .../files/lib/system/option/MultiSelectOptionType.class.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/wcfsetup/install/files/lib/system/option/MultiSelectOptionType.class.php b/wcfsetup/install/files/lib/system/option/MultiSelectOptionType.class.php index 61a41a1194..6578ad2f84 100644 --- a/wcfsetup/install/files/lib/system/option/MultiSelectOptionType.class.php +++ b/wcfsetup/install/files/lib/system/option/MultiSelectOptionType.class.php @@ -106,9 +106,10 @@ class MultiSelectOptionType extends SelectOptionType { public function checkUser(User $user, Option $option, $value) { if (!is_array($value) || empty($value)) return false; - $optionValues = explode('\n', $user->getUserOption($option->optionName)); + $optionValues = explode("\n", $user->getUserOption($option->optionName)); - return count(array_diff($optionValues, $value)) > 0; + // check if the user has selected all options in $value array + return count(array_intersect($value, $optionValues)) === count($value); } /** -- 2.20.1