remove unnecessary SQL queries

app/Http/Controllers/UserController.php

@@ -75,6 +75,7 @@ class UserController extends Controller
 
     public function scoreInfo()
     {
+        /** @var User */
         $user = Auth::user();
 
         return response()->json([
@@ -87,7 +88,7 @@ class UserController extends Controller
                 'players' => (int) option('score_per_player'),
             ],
             'usage' => [
-                'players' => $user->players->count(),
+                'players' => $user->players()->count(),
                 'storage' => (int) Texture::where('uploader', $user->uid)->sum('size'),
             ],
             'signAfterZero' => (bool) option('sign_after_zero'),

app/Listeners/ResetInvalidTextureForPlayer.php

@@ -1,19 +0,0 @@
-<?php
-
-namespace App\Listeners;
-
-class ResetInvalidTextureForPlayer
-{
-    public function handle($event)
-    {
-        $player = $event->player;
-
-        foreach (['skin', 'cape'] as $type) {
-            $field = "tid_$type";
-            if (!\App\Models\Texture::find($player->$field)) {
-                $player->$field = 0;
-            }
-        }
-        $player->save();
-    }
-}

app/Providers/EventServiceProvider.php

@@ -9,9 +9,6 @@ class EventServiceProvider extends ServiceProvider
 {
     // The event listener mappings for the application.
     protected $listen = [
-        'App\Events\PlayerRetrieved' => [
-            Listeners\ResetInvalidTextureForPlayer::class,
-        ],
         'App\Events\PluginWasEnabled' => [
             Listeners\CopyPluginAssets::class,
             Listeners\GeneratePluginTranslations::class,

resources/misc/changelogs/en/5.0.0.md

@@ -51,6 +51,7 @@
 - Changed icon of "Walking/Running" button of skin viewer.
 - Changed API of retrieving all players.
 - Changed format of avatar and 2D preview to WebP.
+- Reduced some unnecessary SQL queries.
 
 ## Fixed
 

resources/misc/changelogs/zh_CN/5.0.0.md

@@ -51,6 +51,7 @@
 - 更换皮肤预览器的「行走/奔跑」按钮图标
 - 更改获取角色的 API
 - 头像和预览图的格式改为 WebP
+- 减少不必要的 SQL 查询语句
 
 ## 修复
 

tests/ListenersTest/ResetInvalidTextureForPlayerTest.php

@@ -1,23 +0,0 @@
-<?php
-
-namespace Tests;
-
-use App\Models\Player;
-use Illuminate\Foundation\Testing\DatabaseTransactions;
-
-class ResetInvalidTextureForPlayerTest extends TestCase
-{
-    use DatabaseTransactions;
-
-    public function testHandle()
-    {
-        $pid = factory(Player::class)->create([
-            'tid_skin' => 1,
-            'tid_cape' => 2,
-        ])->pid;
-
-        $player = Player::find($pid);
-        $this->assertEquals(0, $player->tid_skin);
-        $this->assertEquals(0, $player->tid_cape);
-    }
-}