Merge pull request #11737 from nanaya/owner-change-notification-user-ids

Notify all owners on beatmap owner change
ppy · Dec 11, 2024 · 832f2bb · 832f2bb
2 parents 642e973 + 59c0349
commit 832f2bb
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/app/Jobs/Notifications/BeatmapOwnerChange.php b/app/Jobs/Notifications/BeatmapOwnerChange.php
@@ -15,7 +15,6 @@ class BeatmapOwnerChange extends BroadcastNotificationBase
 
     protected $beatmap;
     protected $beatmapset;
-    protected $user;
 
     public static function getMailLink(Notification $notification): string
     {
@@ -32,7 +31,6 @@ public function __construct(Beatmap $beatmap, User $source)
 
         $this->beatmap = $beatmap;
         $this->beatmapset = $beatmap->beatmapset;
-        $this->user = $beatmap->user;
     }
 
     public function getDetails(): array
@@ -48,7 +46,7 @@ public function getDetails(): array
 
     public function getListeningUserIds(): array
     {
-        return [$this->user->getKey()];
+        return $this->beatmap->beatmapOwners()->pluck('user_id')->all();
     }
 
     public function getNotifiable()