From 186e79d41b2b51425784b7e211d32e8e5a8939d7 Mon Sep 17 00:00:00 2001 From: Umherirrender Date: Thu, 2 May 2024 23:25:20 +0200 Subject: [PATCH] Use user_properties.up_value with string in database query The field up_value is a varbinary and should be provided as string, not all rdbms implicit converts int/strings Change-Id: Ideb28ad9cc475dad84d0f7dc45a0f3c286f4487c --- maintenance/NewTopicOptOutActiveUsers.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/maintenance/NewTopicOptOutActiveUsers.php b/maintenance/NewTopicOptOutActiveUsers.php index 48200070a..bd1129328 100644 --- a/maintenance/NewTopicOptOutActiveUsers.php +++ b/maintenance/NewTopicOptOutActiveUsers.php @@ -102,7 +102,7 @@ class NewTopicOptOutActiveUsers extends Maintenance { ->where( [ 'up_user' => $userId, 'up_property' => 'discussiontools-betaenable', - 'up_value' => 1, + 'up_value' => '1', ] ) ->field( '1' ) ->fetchField(); @@ -134,7 +134,7 @@ class NewTopicOptOutActiveUsers extends Maintenance { ->row( [ 'up_user' => $userId, 'up_property' => 'discussiontools-' . HookUtils::NEWTOPICTOOL, - 'up_value' => 0, + 'up_value' => '0', ] ) ->caller( __METHOD__ ) ->execute();