laravel · hafezdivandari · May 30, 2024 · Jun 15, 2024 · Jun 15, 2024 · Jun 16, 2024
diff --git a/UPGRADE.md b/UPGRADE.md
@@ -36,6 +36,16 @@ Passport's `oauth_personal_access_clients` table has been redundant and unnecess
 
 In addition, the `Laravel\Passport\PersonalAccessClient` model, `Passport::$personalAccessClientModel` property, `Passport::usePersonalAccessClientModel()`, `Passport::personalAccessClientModel()`, and `Passport::personalAccessClient()` methods have been removed.
 
+### Redundant Columns Removal
+
+PR: https://github.com/laravel/passport/pull/1752
+
+The `user_id`, `client_id` and `scopes` columns of the Passport's `oauth_auth_codes` table have been removed. Therefore, you may create a migration that drops these columns:
+
+    Schema::table('oauth_auth_codes', function (Blueprint $table) {
+        $table->dropColumn(['user_id', 'client_id', 'scopes']);
+    });
+
 ## Upgrading To 12.0 From 11.x
 
 ### Migration Changes

diff --git a/database/migrations/2016_06_01_000001_create_oauth_auth_codes_table.php b/database/migrations/2016_06_01_000001_create_oauth_auth_codes_table.php
@@ -13,9 +13,6 @@ public function up(): void
     {
         Schema::create('oauth_auth_codes', function (Blueprint $table) {
             $table->string('id', 100)->primary();
-            $table->unsignedBigInteger('user_id')->index();
-            $table->unsignedBigInteger('client_id');
-            $table->text('scopes')->nullable();
             $table->boolean('revoked');
             $table->dateTime('expires_at')->nullable();
         });

diff --git a/src/AuthCode.php b/src/AuthCode.php
@@ -51,16 +51,6 @@ class AuthCode extends Model
      */
     protected $keyType = 'string';
 
-    /**
-     * Get the client that owns the authentication code.
-     *
-     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
-     */
-    public function client()
-    {
-        return $this->belongsTo(Passport::clientModel());
-    }
-
     /**
      * Get the current connection name for the model.
      *

diff --git a/src/Bridge/AuthCodeRepository.php b/src/Bridge/AuthCodeRepository.php
@@ -8,8 +8,6 @@
 
 class AuthCodeRepository implements AuthCodeRepositoryInterface
 {
-    use FormatsScopesForStorage;
-
     /**
      * {@inheritdoc}
      */
@@ -25,9 +23,6 @@ public function persistNewAuthCode(AuthCodeEntityInterface $authCodeEntity): voi
     {
         $attributes = [
             'id' => $authCodeEntity->getIdentifier(),
-            'user_id' => $authCodeEntity->getUserIdentifier(),
-            'client_id' => $authCodeEntity->getClient()->getIdentifier(),
-            'scopes' => $this->formatScopesForStorage($authCodeEntity->getScopes()),
             'revoked' => false,
             'expires_at' => $authCodeEntity->getExpiryDateTime(),
         ];

diff --git a/src/Bridge/FormatsScopesForStorage.php b/src/Bridge/FormatsScopesForStorage.php
@@ -4,16 +4,6 @@
 
 trait FormatsScopesForStorage
 {
-    /**
-     * Format the given scopes for storage.
-     *
-     * @param  \League\OAuth2\Server\Entities\ScopeEntityInterface[]  $scopes
-     */
-    public function formatScopesForStorage(array $scopes): string
-    {
-        return json_encode($this->scopesToArray($scopes));
-    }
-
     /**
      * Get an array of scope identifiers for storage.
      *

diff --git a/src/Client.php b/src/Client.php
@@ -88,16 +88,6 @@ public function user()
         );
     }
 
-    /**
-     * Get all of the authentication codes for the client.
-     *
-     * @return \Illuminate\Database\Eloquent\Relations\HasMany
-     */
-    public function authCodes()
-    {
-        return $this->hasMany(Passport::authCodeModel(), 'client_id');
-    }
-
     /**
      * Get all of the tokens that belong to the client.
      *

diff --git a/src/Console/InstallCommand.php b/src/Console/InstallCommand.php
@@ -66,7 +66,6 @@ protected function configureUuids()
         Passport::setClientUuids(true);
 
         $this->replaceInFile(config_path('passport.php'), '\'client_uuids\' => false', '\'client_uuids\' => true');
-        $this->replaceInFile(database_path('migrations/****_**_**_******_create_oauth_auth_codes_table.php'), '$table->unsignedBigInteger(\'client_id\');', '$table->uuid(\'client_id\');');
         $this->replaceInFile(database_path('migrations/****_**_**_******_create_oauth_access_tokens_table.php'), '$table->unsignedBigInteger(\'client_id\');', '$table->uuid(\'client_id\');');
         $this->replaceInFile(database_path('migrations/****_**_**_******_create_oauth_clients_table.php'), '$table->bigIncrements(\'id\');', '$table->uuid(\'id\')->primary();');
     }