loadsman / laravel-plugin

src/Http/Controllers/RequestController.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     }
 
     /**
-     * @return mixed
+     * @return \Symfony\Component\HttpFoundation\Response
      */
     public function index()
     {

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
     {
         $id = $request->id;
 
-        if (! $this->repository->exists($id)) {
+        if (!$this->repository->exists($id)) {
             return response(null, 404);
         }
 

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,12 +2,12 @@
 
 namespace Loadsman\LaravelPlugin\Http\Controllers;
 
+use Illuminate\Http\Request;
+use Illuminate\Routing\Controller;
 use Loadsman\LaravelPlugin\Contracts\RequestRepositoryInterface;
 use Loadsman\LaravelPlugin\Entities\RequestEntity;
 use Loadsman\LaravelPlugin\Http\Requests\StoreRequest;
 use Loadsman\LaravelPlugin\Http\Requests\UpdateRequest;
-use Illuminate\Http\Request;
-use Illuminate\Routing\Controller;
 
 /**
  * Class RequestController

src/LoadsmanServiceProvider.php

Unused Use Statements -3 removed lines

@@ -7,9 +7,6 @@
 use Loadsman\LaravelPlugin\Modules\Rule\RuleRepository;
 use Loadsman\LaravelPlugin\Modules\Rule\RuleServiceProvider;
 use Loadsman\LaravelPlugin\Providers\MacroServiceProvider;
-use Loadsman\LaravelPlugin\Providers\RepositoryServiceProvider;
-use Loadsman\LaravelPlugin\Providers\RouteServiceProvider;
-use Illuminate\Routing\Router;
 
 class LoadsmanServiceProvider extends \Illuminate\Support\ServiceProvider
 {

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
     {
         $this->mergeConfigFrom($this->getConfigPath(), 'loadsman');
 
-        if (! $this->app['config']['loadsman.enabled']) {
+        if (!$this->app['config']['loadsman.enabled']) {
             return;
         }
 
@@ -25,10 +25,10 @@ 
         $this->app->register(FrameworkServiceProvider::class);
         $this->app->register(RuleServiceProvider::class);
 
-        $this->app->bind(RuleRepository::class, function (Container $app) {
+        $this->app->bind(RuleRepository::class, function(Container $app) {
 
             $routers = [];
-            foreach ($app['config']['loadsman.routers'] as $routerName){
+            foreach ($app['config']['loadsman.routers'] as $routerName) {
                 $routers[] = $app->make($routerName);
             }
 

src/Http/Middleware/DetectRoute.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 
             // Laravel 5.2 event
             $this->events->listen(RouteMatched::class,
-                function (RouteMatched $event) {
+                function(RouteMatched $event) {
                     $this->handleMatchedRoute($event->route);
                 });
         }
@@ -46,7 +46,7 @@ 
         return $next($request);
     }
 
-    public function handleMatchedRoute($route){
+    public function handleMatchedRoute($route) {
         $routeInfo = new RouteInfo($route);
 
         response()->json([

src/Http/Requests/UpdateRequest.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         ];
     }
 
-    public function authorize(){
+    public function authorize() {
         return true;
     }
 }

src/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
             'prefix' => config('api-tester.route'),
             'namespace' => $this->getNamespace(),
             'middleware' => $this->getMiddleware(),
-        ], function () {
+        ], function() {
             $this->requireRoutes();
         });
     }
@@ -73,7 +73,7 @@ 
      */
     protected function requireRoutes()
     {
-        require __DIR__ . '/../Http/routes.php';
+        require __DIR__.'/../Http/routes.php';
     }
 
     /**

Unused Use Statements +2 added lines, -3 removed lines

@@ -2,12 +2,11 @@
 
 namespace Loadsman\LaravelPlugin\Providers;
 
-use Loadsman\LaravelPlugin\Http\Middleware\DebugState;
-use Loadsman\LaravelPlugin\Http\Middleware\DetectRoute;
-use Loadsman\LaravelPlugin\Http\Middleware\PreventRedirect;
 use Illuminate\Contracts\Http\Kernel;
 use Illuminate\Routing\Router;
 use Illuminate\Support\ServiceProvider;
+use Loadsman\LaravelPlugin\Http\Middleware\DetectRoute;
+use Loadsman\LaravelPlugin\Http\Middleware\PreventRedirect;
 
 class RouteServiceProvider extends ServiceProvider
 {

src/Providers/RepositoryServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function register()
     {
-        $this->app->singleton(RouteRepositoryInterface::class, function (Container $app) {
+        $this->app->singleton(RouteRepositoryInterface::class, function(Container $app) {
             $repositories = [];
             foreach (config('api-tester.route_repositories') as $repository) {
                 $repositories[] = $app->make($repository);

Unused Use Statements +2 added lines, -2 removed lines

@@ -3,12 +3,12 @@
 namespace Loadsman\LaravelPlugin\Providers;
 
 
+use Illuminate\Contracts\Container\Container;
+use Illuminate\Support\ServiceProvider;
 use Loadsman\LaravelPlugin\Collections\RouteCollection;
 use Loadsman\LaravelPlugin\Contracts\RequestRepositoryInterface;
 use Loadsman\LaravelPlugin\Contracts\RouteRepositoryInterface;
 use Loadsman\LaravelPlugin\Repositories\RouteRepository;
-use Illuminate\Contracts\Container\Container;
-use Illuminate\Support\ServiceProvider;
 
 class RepositoryServiceProvider extends ServiceProvider
 {

src/Providers/MacroServiceProvider.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
          * Present controller method as `post` route.
          */
         app('router')->macro('expose',
-            function ($controller = '', Array $actions) {
+            function($controller = '', Array $actions) {
                 foreach ($actions as $action) {
                     $this->post(snake_case($action, '-'),
                         $controller.'@'.$action);
@@ -26,13 +26,13 @@ 
          * Expose all public methods of controller.
          * @see `expose` macro /\
          */
-        app('router')->macro('exposeAll', function ($controllerClassName) {
+        app('router')->macro('exposeAll', function($controllerClassName) {
             $controllerClassReflection = new \ReflectionClass($controllerClassName);
             $publicActions = $controllerClassReflection->getMethods(\ReflectionMethod::IS_PUBLIC);
             foreach ($publicActions as $publicAction) {
                 $declaringClass = $publicAction->getDeclaringClass()->getName();
                 $isOwn = $declaringClass === $controllerClassName;
-                if (! $isOwn) {
+                if (!$isOwn) {
                     continue;
                 }
                 $action = $publicAction->getName();

src/Collections/RouteCollection.php

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
             }
         }
 
-        return $this->filter(function ($route) use ($patterns) {
+        return $this->filter(function($route) use ($patterns) {
             // If any of patterns matches - route passes.
             foreach ($patterns as $pattern) {
                 if ($this->isRouteMatchesPattern($route, $pattern)) {
@@ -69,17 +69,17 @@ 
     private function isRouteMatchesPattern(array $route, array $pattern)
     {
         foreach ($route as $key => $value) {
-            if (! array_key_exists($key, $pattern)) {
+            if (!array_key_exists($key, $pattern)) {
                 continue;
             }
 
-            if(is_array($value)){
+            if (is_array($value)) {
                 $value = implode(',', $value);
             }
 
             $regex = '#'.$pattern[$key].'#';
 
-            return ! ! preg_match($regex, $value);
+            return !!preg_match($regex, $value);
         }
 
         return true;

src/Modules/Routers/Laravel/LaravelRouter.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     {
         $routes = $this->router->getRoutes()->getRoutes();
 
-        return array_map(function (Route $route) {
+        return array_map(function(Route $route) {
             return $this->transformer->cast($route);
         }, $routes);
     }