Incorrect usage of `NOT IN()` for columns that accept `NULL`

`NULL` is considered to fail the condition `NOT IN (1, 2, …)`.

diff --git a/wcfsetup/install/files/lib/system/box/AbstractCommentListBoxController.class.php b/wcfsetup/install/files/lib/system/box/AbstractCommentListBoxController.class.php
index ce1eb865e79fb21d6f2e7569d51ddbdfccb9f7f0..cd4a0f60c62cbb894c4141ce9cd1413ad328a5a5 100644 (file)
--- a/wcfsetup/install/files/lib/system/box/AbstractCommentListBoxController.class.php
+++ b/wcfsetup/install/files/lib/system/box/AbstractCommentListBoxController.class.php
@@ -86,7 +86,7 @@ abstract class AbstractCommentListBoxController extends AbstractDatabaseObjectLi
                $this->applyObjectTypeFilters($commentList);
                
                if (!empty(UserProfileHandler::getInstance()->getIgnoredUsers())) {
-                       $commentList->getConditionBuilder()->add("comment.userID NOT IN (?)", [UserProfileHandler::getInstance()->getIgnoredUsers()]);
+                       $commentList->getConditionBuilder()->add("(comment.userID IS NULL OR comment.userID NOT IN (?))", [UserProfileHandler::getInstance()->getIgnoredUsers()]);
                }
                
                return $commentList;