Show custom user tags on beatmapset

app/Http/Controllers/BeatmapTagsController.php

@@ -23,22 +23,6 @@ public function __construct()
                 'destroy',
             ],
         ]);
-
-        $this->middleware('require-scopes:public', ['only' => 'index']);
-    }
-
-    public function index($beatmapId)
-    {
-        $topBeatmapTags = cache_remember_mutexed(
-            "beatmap_tags:{$beatmapId}",
-            $GLOBALS['cfg']['osu']['tags']['beatmap_tags_cache_duration'],
-            [],
-            fn () => Tag::topTags($beatmapId),
-        );
-
-        return [
-            'beatmap_tags' => $topBeatmapTags,
-        ];
     }
 
     public function destroy($beatmapId, $tagId)

app/Http/Controllers/BeatmapsetsController.php

@@ -404,6 +404,7 @@ private function showJson($beatmapset)
             'beatmaps.failtimes',
             'beatmaps.max_combo',
             'beatmaps.owners',
+            'beatmaps.top_tag_ids',
             'converts',
             'converts.failtimes',
             'converts.owners',
@@ -415,6 +416,7 @@ private function showJson($beatmapset)
             'pack_tags',
             'ratings',
             'recent_favourites',
+            'related_tags',
             'related_users',
             'user',
         ]);

app/Http/Controllers/TagsController.php

@@ -7,9 +7,6 @@
 
 namespace App\Http\Controllers;
 
-use App\Models\Tag;
-use App\Transformers\TagTransformer;
-
 class TagsController extends Controller
 {
     public function __construct()
@@ -21,15 +18,8 @@ public function __construct()
 
     public function index()
     {
-        $tags = cache_remember_mutexed(
-            'tags',
-            $GLOBALS['cfg']['osu']['tags']['tags_cache_duration'],
-            [],
-            fn () => Tag::all(),
-        );
-
         return [
-            'tags' => json_collection($tags, new TagTransformer()),
+            'tags' => app('tags')->json(),
         ];
     }
 }

app/Models/Beatmap.php

@@ -348,6 +348,16 @@ public function status()
         return array_search($this->approved, Beatmapset::STATES, true);
     }
 
+    public function topTagIds()
+    {
+        return cache_remember_mutexed(
+            "beatmap_top_tag_ids:{$this->getKey()}",
+            $GLOBALS['cfg']['osu']['tags']['beatmap_tags_cache_duration'],
+            [],
+            fn () => Tag::topTagIds($this->getKey())->toArray(),
+        );
+    }
+
     private function getDifficultyrating()
     {
         if ($this->convert) {

app/Models/Tag.php

@@ -23,19 +23,19 @@ public function beatmapTags(): HasMany
         return $this->hasMany(BeatmapTag::class);
     }
 
-    public static function topTags($beatmapId)
+    public static function topTagIds(int $beatmapId, int $limit = 50)
     {
         return static
             ::joinRelation(
                 'beatmapTags',
                 fn ($q) => $q->where('beatmap_id', $beatmapId)->whereHas('user', fn ($userQuery) => $userQuery->default())
             )
             ->groupBy('id')
-            ->select('id', 'name')
+            ->select('id')
             ->selectRaw('COUNT(*) as count')
             ->orderBy('count', 'desc')
             ->orderBy('id', 'desc')
-            ->limit(50)
+            ->limit($limit)
             ->get();
     }
 }

app/Providers/AppServiceProvider.php

@@ -31,6 +31,7 @@ class AppServiceProvider extends ServiceProvider
         'layout-cache' => Singletons\LayoutCache::class,
         'medals' => Singletons\Medals::class,
         'smilies' => Singletons\Smilies::class,
+        'tags' => Singletons\Tags::class,
         'user-cover-presets' => Singletons\UserCoverPresets::class,
     ];
 

app/Singletons/Tags.php

@@ -0,0 +1,35 @@
+<?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\Singletons;
+
+use App\Models\Tag;
+use App\Traits\Memoizes;
+use App\Transformers\TagTransformer;
+
+class Tags
+{
+    use Memoizes;
+
+    public function get(int $id): ?Tag
+    {
+        $allById = $this->memoize(
+            'allById',
+            fn () => Tag::all()->keyBy('id'),
+        );
+
+        return $allById[$id] ?? null;
+    }
+
+    public function json(): array
+    {
+        return $this->memoize(
+            __FUNCTION__,
+            fn () => json_collection(Tag::all(), new TagTransformer()),
+        );
+    }
+}

app/Transformers/BeatmapCompactTransformer.php

@@ -18,6 +18,7 @@ class BeatmapCompactTransformer extends TransformerAbstract
         'failtimes',
         'max_combo',
         'owners',
+        'top_tag_ids',
         'user',
     ];
 
@@ -83,6 +84,11 @@ public function includeOwners(Beatmap $beatmap)
         ]);
     }
 
+    public function includeTopTagIds(Beatmap $beatmap)
+    {
+        return $this->primitive($beatmap->topTagIds());
+    }
+
     public function includeUser(Beatmap $beatmap)
     {
         return $this->item(

app/Transformers/BeatmapsetCompactTransformer.php

@@ -40,6 +40,7 @@ class BeatmapsetCompactTransformer extends TransformerAbstract
         'ratings',
         'recent_favourites',
         'related_users',
+        'related_tags',
         'user',
     ];
 
@@ -299,6 +300,24 @@ public function includeRelatedUsers(Beatmapset $beatmapset)
         return $this->collection($users, new UserCompactTransformer());
     }
 
+    public function includeRelatedTags(Beatmapset $beatmapset)
+    {
+        $beatmaps = $this->beatmaps($beatmapset);
+        $tagIdSet = new Set($beatmaps->flatMap->topTagIds()->pluck('id'));
+
+        $cachedTags = app('tags');
+        $json = [];
+
+        foreach ($tagIdSet as $tagId) {
+            $tag = $cachedTags->get($tagId);
+            if ($tag !== null) {
+                $json[] = $tag;
+            }
+        }
+
+        return $this->primitive($json);
+    }
+
     private function beatmaps(Beatmapset $beatmapset, ?Fractal\ParamBag $params = null): EloquentCollection
     {
         $rel = $beatmapset->trashed() || ($params !== null && $params->get('with_trashed')) ? 'allBeatmaps' : 'beatmaps';

resources/js/beatmapsets-show/controller.ts

@@ -2,6 +2,7 @@
 // See the LICENCE file in the repository root for full licence text.
 
 import { BeatmapsetJsonForShow } from 'interfaces/beatmapset-extended-json';
+import { TagJsonWithCount } from 'interfaces/tag-json';
 import UserJson from 'interfaces/user-json';
 import { keyBy } from 'lodash';
 import { action, computed, makeObservable, observable, runInAction } from 'mobx';
@@ -10,6 +11,7 @@ import core from 'osu-core-singleton';
 import { find, findDefault, group } from 'utils/beatmap-helper';
 import { parse } from 'utils/beatmapset-page-hash';
 import { parseJson } from 'utils/json';
+import { present } from 'utils/string';
 import { currentUrl } from 'utils/turbolinks';
 
 export type ScoreLoadingState = null | 'error' | 'loading' | 'supporter_only' | 'unranked';
@@ -70,6 +72,52 @@ export default class Controller {
     return this.beatmaps.get(this.currentBeatmap.mode) ?? [];
   }
 
+  get relatedTags() {
+    const map = new Map();
+
+    for (const tag of this.beatmapset.related_tags) {
+      map.set(tag.id, tag);
+    }
+
+    return map;
+  }
+
+  @computed
+  get tags() {
+    const mapperTagSet = new Set(this.beatmapset.tags.split(' ').filter(present));
+    const tagMap = new Map<number, TagJsonWithCount>();
+
+    for (const tag of this.beatmapset.related_tags) {
+      tag.count = 0; // assign 0 and cast
+      tagMap.set(tag.id, tag as TagJsonWithCount);
+    }
+
+    for (const beatmap of this.beatmapset.beatmaps) {
+      if (beatmap.top_tag_ids == null) continue;
+
+      for (const tagId of beatmap.top_tag_ids) {
+        const tag = tagMap.get(tagId.id);
+        if (tag == null) continue;
+
+        tag.count += tagId.count;
+
+        // TODO: case insensitivity
+        if (mapperTagSet.has(tag.name)) {
+          tag.count++;
+          mapperTagSet.delete(tag.name);
+        }
+      }
+    }
+
+    return {
+      mapperTags: [...mapperTagSet.values()],
+      userTags: [...tagMap.values()].sort((a, b) => {
+        const diff = b.count - a.count;
+        return diff !== 0 ? diff : a.id - b.id;
+      }),
+    };
+  }
+
   @computed
   get usersById() {
     return keyBy(this.beatmapset.related_users, 'id') as Partial<Record<number, UserJson>>;

resources/js/beatmapsets-show/info.tsx

@@ -65,6 +65,15 @@ export default class Info extends React.Component<Props> {
     return ret;
   }
 
+  private get tags() {
+    const tags = this.controller.tags;
+
+    return [
+      ...tags.userTags.map((tag) => tag.name),
+      ...tags.mapperTags,
+    ];
+  }
+
   private get withEditDescription() {
     return this.controller.beatmapset.description.bbcode != null;
   }
@@ -84,10 +93,6 @@ export default class Info extends React.Component<Props> {
   }
 
   render() {
-    const tags = this.controller.beatmapset.tags
-      .split(' ')
-      .filter(present);
-
     return (
       <div className='beatmapset-info u-fancy-scrollbar'>
         {this.isEditingDescription &&
@@ -191,14 +196,14 @@ export default class Info extends React.Component<Props> {
             </div>
           </div>
 
-          {tags.length > 0 &&
+          {this.tags.length > 0 &&
             <div className='beatmapset-info__row beatmapset-info__row--value-overflow'>
               <h3 className='beatmapset-info__header'>
                 {trans('beatmapsets.show.info.tags')}
               </h3>
               <div className='beatmapset-info__value-overflow'>
-                {tags.map((tag, i) => (
-                  <React.Fragment key={`${tag}-${i}`}>
+                {this.tags.map((tag) => (
+                  <React.Fragment key={`${tag}`}>
                     <a
                       className='beatmapset-info__link'
                       href={route('beatmapsets.index', { q: tag })}

resources/js/interfaces/beatmap-json.ts

@@ -17,6 +17,7 @@ interface BeatmapJsonAvailableIncludes {
   failtimes: BeatmapFailTimesArray;
   max_combo: number;
   owners: BeatmapOwnerJson[];
+  top_tag_ids: { count: number; id: number }[];
   user: UserJson;
 }
 

resources/js/interfaces/beatmapset-extended-json.ts

@@ -55,6 +55,7 @@ type BeatmapsetJsonForShowIncludes = Required<Pick<BeatmapsetExtendedJson,
 | 'language'
 | 'ratings'
 | 'recent_favourites'
+| 'related_tags'
 | 'related_users'
 | 'user'
 >>;

resources/js/interfaces/beatmapset-json.ts

@@ -9,6 +9,7 @@ import BeatmapsetNominationJson from './beatmapset-nomination-json';
 import GenreJson from './genre-json';
 import LanguageJson from './language-json';
 import Ruleset from './ruleset';
+import TagJson from './tag-json';
 import UserJson, { UserJsonDeleted } from './user-json';
 
 export interface Availability {
@@ -92,8 +93,10 @@ interface BeatmapsetJsonAvailableIncludes {
   nominations: BeatmapsetNominationsInterface;
   ratings: number[];
   recent_favourites: UserJson[];
+  related_tags: TagJson[];
   related_users: UserJson[];
   user: UserJson | UserJsonDeleted;
+  user_tags: TagJson[];
 }
 
 interface HypeData {

resources/js/interfaces/tag-json.ts

@@ -0,0 +1,11 @@
+// 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.
+
+export default interface TagJson {
+  count?: number;
+  description: string;
+  id: number;
+  name: string;
+}
+
+export type TagJsonWithCount = TagJson & Required<Pick<TagJson, 'count'>>;

routes/web.php

@@ -430,7 +430,7 @@
                     });
                 });
 
-                Route::apiResource('tags', 'BeatmapTagsController', ['only' => ['index', 'store', 'destroy']]);
+                Route::apiResource('tags', 'BeatmapTagsController', ['only' => ['store', 'destroy']]);
             });
         });