Merge pull request #12018 from nanaya/beatmap-scores-count

Include score count in beatmap scores response

Author: notbakaneko

app/Http/Controllers/BeatmapsController.php

@@ -79,6 +79,7 @@ private static function beatmapScores(string $id, ?string $scoreTransformerType,
         $scoreTransformer = new ScoreTransformer($scoreTransformerType);
 
         $results = [
+            'score_count' => UserRank::getCount($esFetch->baseParams),
             'scores' => json_collection(
                 $scores,
                 $scoreTransformer,

app/Libraries/Score/BeatmapScores.php

@@ -14,8 +14,8 @@
 
 class BeatmapScores
 {
+    public ScoreSearchParams $baseParams;
     public array $result;
-    private ScoreSearchParams $baseParams;
 
     public function __construct(private array $rawParams)
     {
@@ -48,7 +48,6 @@ public function rank(SoloScore $score): int
 
         $params = clone $this->baseParams;
         $params->beforeScore = $score;
-        $params->setSort(null);
 
         return UserRank::getRank($params);
     }

app/Libraries/Score/UserRank.php

@@ -11,12 +11,9 @@
 
 class UserRank
 {
-    public static function getRank(ScoreSearchParams $params): int
+    public static function getCount(ScoreSearchParams $params): int
     {
-        if ($params->beforeTotalScore === null && $params->beforeScore === null) {
-            throw new InvariantException('beforeScore or beforeTotalScore must be specified');
-        }
-
+        $params->setSort(null);
         $search = new ScoreSearch($params);
 
         $search->size(0);
@@ -27,6 +24,15 @@ public static function getRank(ScoreSearchParams $params): int
         $response = $search->response();
         $search->assertNoError();
 
-        return 1 + $response->aggregations($aggName)['value'];
+        return $response->aggregations($aggName)['value'];
+    }
+
+    public static function getRank(ScoreSearchParams $params): int
+    {
+        if ($params->beforeTotalScore === null && $params->beforeScore === null) {
+            throw new InvariantException('beforeScore or beforeTotalScore must be specified');
+        }
+
+        return 1 + static::getCount($params);
     }
 }