Merge branch 'master' into legacy-score-id-zero

app/Exceptions/Handler.php

@@ -48,7 +48,7 @@ class Handler extends ExceptionHandler
     public static function exceptionMessage($e)
     {
         if ($e instanceof ModelNotFoundException) {
-            return;
+            return static::modelNotFoundMessage($e);
         }
 
         if (static::statusCode($e) >= 500) {
@@ -90,6 +90,20 @@ private static function isOAuthSessionException(Throwable $e): bool
             && $e->getMessage() === 'Authorization request was not present in the session.';
     }
 
+    private static function modelNotFoundMessage(ModelNotFoundException $e): string
+    {
+        $model = $e->getModel();
+        $modelTransKey = "models.name.{$model}";
+
+        $params = [
+            'model' => trans_exists($modelTransKey, $GLOBALS['cfg']['app']['fallback_locale'])
+                ? osu_trans($modelTransKey)
+                : trim(strtr($model, ['App\Models\\' => '']), '\\'),
+        ];
+
+        return osu_trans('models.not_found', $params);
+    }
+
     private static function reportWithSentry(Throwable $e): void
     {
         $ref = log_error_sentry($e, ['http_code' => (string) static::statusCode($e)]);

app/Http/Controllers/BeatmapsController.php

@@ -75,7 +75,7 @@ private static function beatmapScores(string $id, ?string $scoreTransformerType,
             'type' => $type,
             'user' => $currentUser,
         ]);
-        $scores = $esFetch->all()->loadMissing(['beatmap', 'user.country']);
+        $scores = $esFetch->all()->loadMissing(['beatmap', 'user.country', 'processHistory']);
         $userScore = $esFetch->userBest();
         $scoreTransformer = new ScoreTransformer($scoreTransformerType);
 
@@ -494,7 +494,7 @@ public function userScoreAll($beatmapId, $userId)
             'sort' => 'score_desc',
             'user_id' => (int) $userId,
         ]);
-        $scores = (new ScoreSearch($params))->records();
+        $scores = (new ScoreSearch($params))->records()->loadMissing('processHistory');
 
         return [
             'scores' => json_collection($scores, new ScoreTransformer()),

app/Libraries/Search/ArtistTrackSearchParams.php

@@ -27,10 +27,12 @@ class ArtistTrackSearchParams extends SearchParams
     public ?string $album;
     public ?string $artist;
     public ?array $bpm;
+    public ?array $bpmInput;
     public bool $exclusiveOnly = false;
     public ?string $genre;
     public bool $isDefaultSort = false;
     public ?array $length;
+    public ?array $lengthInput;
     public string $sortField;
     public string $sortOrder;
 

app/Models/Solo/Score.php

@@ -149,6 +149,11 @@ public function user()
         return $this->belongsTo(User::class, 'user_id');
     }
 
+    public function processHistory()
+    {
+        return $this->hasOne(ScoreProcessHistory::class, 'score_id');
+    }
+
     public function scopeDefault(Builder $query): Builder
     {
         return $query->whereHas('beatmap.beatmapset');
@@ -243,6 +248,7 @@ public function getAttribute($key)
 
             'beatmap',
             'performance',
+            'processHistory',
             'reportedIn',
             'user' => $this->getRelationValue($key),
         };

app/Models/Solo/ScoreProcessHistory.php

@@ -0,0 +1,25 @@
+<?php
+
+// Copyright (c) ppy Pty Ltd <[email protected]>. Licensed under the GNU Affero General Public License v3.0.
+// See the LICENCE file in the repository root for full licence text.
+
+declare(strict_types=1);
+
+namespace App\Models\Solo;
+
+use App\Models\Model;
+
+/**
+ * @property int $score_id
+ * @property int $processed_version
+ * @property \Carbon\Carbon $processed_at
+ */
+class ScoreProcessHistory extends Model
+{
+    protected $table = 'score_process_history';
+
+    public function score()
+    {
+        return $this->belongsTo(Score::class, 'score_id');
+    }
+}

app/Transformers/Multiplayer/RoomTransformer.php

@@ -21,7 +21,6 @@ class RoomTransformer extends TransformerAbstract
         'playlist',
         'playlist_item_stats',
         'recent_participants',
-        'scores',
     ];
 
     public function transform(Room $room)
@@ -94,12 +93,4 @@ public function includePlaylistItemStats(Room $room)
     {
         return $this->primitive($room->playlistItemStats());
     }
-
-    public function includeScores(Room $room)
-    {
-        return $this->collection(
-            $room->scores()->completed()->get(),
-            new ScoreTransformer()
-        );
-    }
 }

app/Transformers/ScoreTransformer.php

@@ -24,6 +24,7 @@ class ScoreTransformer extends TransformerAbstract
     // warning: the preload is actually for PlaylistItemUserHighScore, not for Score
     const MULTIPLAYER_BASE_PRELOAD = [
         'scoreLink.score',
+        'scoreLink.score.processHistory',
         'scoreLink.user.country',
     ];
 
@@ -35,6 +36,7 @@ class ScoreTransformer extends TransformerAbstract
     const USER_PROFILE_INCLUDES_PRELOAD = [
         'beatmap',
         'beatmap.beatmapset',
+        'processHistory',
         // it's for user profile so the user is already available
         // 'user',
     ];
@@ -102,6 +104,7 @@ public function transformSolo(MultiplayerScoreLink|ScoreModel|SoloScore $score)
         if ($score instanceof SoloScore) {
             $extraAttributes['ranked'] = $score->ranked;
             $extraAttributes['preserve'] = $score->preserve;
+            $extraAttributes['processed'] = $score->legacy_score_id !== null || $score->processHistory !== null;
         }
 
         $hasReplay = $score->has_replay;

resources/js/components/block-button.tsx

@@ -120,7 +120,6 @@ export default class BlockButton extends React.Component<Props> {
     if (core.currentUser != null) {
       core.currentUser.blocks = data.filter((d) => d.relation_type === 'block');
       core.currentUser.friends = data.filter((d) => d.relation_type === 'friend');
-      $.publish('user:update', core.currentUser);
     }
 
     this.props.onClick?.();

resources/js/components/follow-toggle.tsx

@@ -20,6 +20,7 @@ interface State {
   toggling: boolean;
 }
 
+// TODO: mobx and turn to observer
 export default class FollowToggle extends React.PureComponent<Props, State> {
   static defaultProps = {
     following: true,
@@ -81,10 +82,7 @@ export default class FollowToggle extends React.PureComponent<Props, State> {
       this.toggleXhr = $.ajax(route('follows.store'), { data: params, method })
         .done(() => {
           if (this.props.follow.subtype === 'mapping') {
-            $.publish('user:followUserMapping:update', {
-              following: !this.state.following,
-              userId: this.props.follow.notifiable_id,
-            });
+            core.currentUserModel.updateFollowUserMapping(!this.state.following, this.props.follow.notifiable_id);
           } else {
             this.setState({ following: !this.state.following });
           }
@@ -97,7 +95,7 @@ export default class FollowToggle extends React.PureComponent<Props, State> {
   private readonly refresh = () => {
     if (this.props.follow.subtype === 'mapping') {
       this.setState({
-        following: core.currentUser != null && core.currentUser.follow_user_mapping.includes(this.props.follow.notifiable_id),
+        following: core.currentUserModel.following.has(this.props.follow.notifiable_id),
       });
     }
   };

resources/js/components/follow-user-mapping-button.tsx

@@ -27,6 +27,7 @@ interface State {
 
 const bn = 'user-action-button';
 
+// TODO: mobx and turn to observer
 export default class FollowUserMappingButton extends React.Component<Props, State> {
   private readonly buttonRef = React.createRef<HTMLButtonElement>();
   private readonly eventId = `follow-user-mapping-button-${nextVal()}`;
@@ -35,7 +36,7 @@ export default class FollowUserMappingButton extends React.Component<Props, Stat
   constructor(props: Props) {
     super(props);
 
-    const following = core.currentUser?.follow_user_mapping.includes(this.props.userId) ?? false;
+    const following = core.currentUserModel.following.has(this.props.userId);
     let followersWithoutSelf = this.props.followers ?? 0;
 
     if (following !== false) followersWithoutSelf -= 1;
@@ -123,7 +124,7 @@ export default class FollowUserMappingButton extends React.Component<Props, Stat
 
   private readonly refresh = () => {
     this.setState({
-      following: core.currentUser?.follow_user_mapping.includes(this.props.userId) ?? false,
+      following: core.currentUserModel.following.has(this.props.userId),
     });
   };
 
@@ -152,6 +153,6 @@ export default class FollowUserMappingButton extends React.Component<Props, Stat
   }
 
   private readonly updateData = () => {
-    $.publish('user:followUserMapping:update', { following: !this.state.following, userId: this.props.userId });
+    core.currentUserModel.updateFollowUserMapping(!this.state.following, this.props.userId);
   };
 }

resources/js/components/friend-button.tsx

@@ -188,7 +188,6 @@ export default class FriendButton extends React.Component<Props> {
 
     // TODO: move logic to a user object?
     core.currentUser.friends = data;
-    $.publish('user:update', core.currentUser);
     dispatch(new FriendUpdated(this.props.userId));
   };
 }