Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 8, 2024

Backport of PR #44095


foreach ($this->getUsersToNotify() as $uid) {
$notification->setUser($uid);
$notification->setUser((string) $uid);

Check notice

Code scanning / Psalm

RedundantCastGivenDocblockType

Redundant cast to string given docblock-provided type

foreach ($this->getUsersToNotify() as $uid) {
$notification->setUser($uid);
$notification->setUser((string) $uid);

Check notice

Code scanning / Psalm

RedundantCastGivenDocblockType

Redundant cast to string given docblock-provided type
@SystemKeeper SystemKeeper changed the title [stable27] [stable28] fix(UpdateNotifications): Handle numeric user ids [stable27] fix(UpdateNotifications): Handle numeric user ids Mar 8, 2024
@solracsf solracsf added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 9, 2024
@joshtrichards joshtrichards force-pushed the backport/44095/stable27 branch from 953d90c to a57c528 Compare March 10, 2024 18:59
manual backport of #44093 to fix #44051 for <=v28

Signed-off-by: Josh <[email protected]>
@joshtrichards joshtrichards force-pushed the backport/44095/stable27 branch from a57c528 to 1f8dc63 Compare March 12, 2024 01:43
@skjnldsv skjnldsv merged commit d34d6a6 into stable27 Mar 16, 2024
@skjnldsv skjnldsv deleted the backport/44095/stable27 branch March 16, 2024 12:22
@Altahrim Altahrim mentioned this pull request Mar 21, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: activity and notification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants