also expire key after update

notbakaneko · notbakaneko · commit 7803c73bff24 · 2025-03-13T21:07:03.000+09:00
diff --git a/app/Http/Controllers/BeatmapTagsController.php b/app/Http/Controllers/BeatmapTagsController.php
@@ -46,6 +46,8 @@ public function update($beatmapId, $tagId)
             ->beatmapTags()
             ->firstOrCreate(['beatmap_id' => $beatmapId, 'user_id' => \Auth::user()->getKey()]);
 
+        $beatmap->expireTopTagIds();
+
         return response()->noContent();
     }
 }
diff --git a/app/Models/Beatmap.php b/app/Models/Beatmap.php
@@ -251,6 +251,11 @@ public function canBeConvertedTo(int $rulesetId)
         return $this->playmode === static::MODES['osu'] || $this->playmode === $rulesetId;
     }
 
+    public function expireTopTagIds()
+    {
+        cache_expire_with_fallback("beatmap_top_tag_ids:{$this->getKey()}", $GLOBALS['cfg']['osu']['tags']['beatmap_tags_cache_duration']);
+    }
+
     public function getAttribute($key)
     {
         return match ($key) {

Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,8 @@ public function update($beatmapId, $tagId)`
`46`	`46`	`->beatmapTags()`
`47`	`47`	`->firstOrCreate(['beatmap_id' => $beatmapId, 'user_id' => \Auth::user()->getKey()]);`
`48`	`48`
	`49`	`+ $beatmap->expireTopTagIds();`
	`50`	`+`
`49`	`51`	`return response()->noContent();`
`50`	`52`	`}`
`51`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -251,6 +251,11 @@ public function canBeConvertedTo(int $rulesetId)`
`251`	`251`	`return $this->playmode === static::MODES['osu'] \|\| $this->playmode === $rulesetId;`
`252`	`252`	`}`
`253`	`253`
	`254`	`+ public function expireTopTagIds()`
	`255`	`+ {`
	`256`	`+ cache_expire_with_fallback("beatmap_top_tag_ids:{$this->getKey()}", $GLOBALS['cfg']['osu']['tags']['beatmap_tags_cache_duration']);`
	`257`	`+ }`
	`258`	`+`
`254`	`259`	`public function getAttribute($key)`
`255`	`260`	`{`
`256`	`261`	`return match ($key) {`