fix(webpush): Don't reuse query object for second query

nickvergessen · backportbot[bot] · commit bb975a72b5c2 · 2026-06-12T13:26:16.000Z
Signed-off-by: Joas Schilling &lt;coding@schilljs.com&gt;
diff --git a/lib/Controller/WebPushController.php b/lib/Controller/WebPushController.php
@@ -298,6 +298,8 @@ protected function activateSubscription(IUser $user, IToken|ISession $token, str
 		if ($row['activated']) {
 			return ActivationSubStatus::OK;
 		}
+
+		$query = $this->db->getQueryBuilder();
 		$query->update('notifications_webpush')
 			->set('activated', $query->createNamedParameter(true))
 			->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))

Original file line number	Diff line number	Diff line change
`@@ -298,6 +298,8 @@ protected function activateSubscription(IUser $user, IToken\|ISession $token, str`
`298`	`298`	`if ($row['activated']) {`
`299`	`299`	`return ActivationSubStatus::OK;`
`300`	`300`	`}`
	`301`	`+`
	`302`	`+ $query = $this->db->getQueryBuilder();`
`301`	`303`	`$query->update('notifications_webpush')`
`302`	`304`	`->set('activated', $query->createNamedParameter(true))`
`303`	`305`	`->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))`