Fix

laravel · Nov 2, 2023 · be2e4fb · be2e4fb
1 parent 2a76ae3
commit be2e4fb
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 17 deletions.
diff --git a/src/Http/Middleware/CheckCredentials.php b/src/Http/Middleware/CheckCredentials.php
@@ -72,12 +72,12 @@ public function handle($request, Closure $next, ...$scopes)
     /**
      * Generate a string representation of the middleware with specified scopes.
      *
-     * @param  array  $scopes 
+     * @param  array  $scopes
      * @return string
      */
     public static function using(...$scopes)
     {
-        return static::class . ':' . implode(',', $scopes);
+        return static::class.':'.implode(',', $scopes);
     }
 
     /**

diff --git a/src/Http/Middleware/CheckForAnyScope.php b/src/Http/Middleware/CheckForAnyScope.php
@@ -35,11 +35,11 @@ public function handle($request, $next, ...$scopes)
     /**
      * Generate a string representation of the middleware with specified scopes.
      *
-     * @param  array  $scopes 
+     * @param  array  $scopes
      * @return string
      */
     public static function using(...$scopes)
     {
-        return static::class . ':' . implode(',', $scopes);
+        return static::class.':'.implode(',', $scopes);
     }
 }
diff --git a/src/Http/Middleware/CheckScopes.php b/src/Http/Middleware/CheckScopes.php
@@ -35,11 +35,11 @@ public function handle($request, $next, ...$scopes)
     /**
      * Generate a string representation of the middleware with specified scopes.
      *
-     * @param  array  $scopes 
+     * @param  array  $scopes
      * @return string
      */
     public static function using(...$scopes)
     {
-        return static::class . ':' . implode(',', $scopes);
+        return static::class.':'.implode(',', $scopes);
     }
 }
diff --git a/src/Http/Middleware/CreateFreshApiToken.php b/src/Http/Middleware/CreateFreshApiToken.php
@@ -61,14 +61,14 @@ public function handle($request, Closure $next, $guard = null)
     /**
      * Specify the guard for the middleware.
      *
-     * @param  string|null  $guard 
+     * @param  string|null  $guard
      * @return string
      */
     public static function using($guard = null)
     {
-        $args = is_null($guard) ? '' : ':' . $guard;
+        $args = is_null($guard) ? '' : ':'.$guard;
 
-        return static::class . $args;
+        return static::class.$args;
     }
 
     /**

diff --git a/tests/Feature/ActingAsTest.php b/tests/Feature/ActingAsTest.php
@@ -50,20 +50,14 @@ public function testActingAsWhenTheRouteIsProtectedByCheckScopesMiddleware()
 
     public function testItCanGenerateDefinitionViaStaticMethod()
     {
-        $signature = (string) CheckScopes::using();
-        $this->assertSame('Laravel\Passport\Http\Middleware\CheckScopes', $signature);
-
         $signature = (string) CheckScopes::using('admin');
         $this->assertSame('Laravel\Passport\Http\Middleware\CheckScopes:admin', $signature);
 
         $signature = (string) CheckScopes::using('admin', 'footest');
         $this->assertSame('Laravel\Passport\Http\Middleware\CheckScopes:admin,footest', $signature);
 
-        $signature = (string) CheckForAnyScope::using('admin', 'footest');
-        $this->assertSame('Laravel\Passport\Http\Middleware\CheckForAnyScope:admin,footest', $signature);
-
-        $signature = (string) CheckForAnyScope::using('admin', 'footest');
-        $this->assertSame('Laravel\Passport\Http\Middleware\CheckForAnyScope:admin,footest', $signature);
+        $signature = (string) CheckForAnyScope::using('admin');
+        $this->assertSame('Laravel\Passport\Http\Middleware\CheckForAnyScope:admin', $signature);
 
         $signature = (string) CheckForAnyScope::using('admin', 'footest');
         $this->assertSame('Laravel\Passport\Http\Middleware\CheckForAnyScope:admin,footest', $signature);