loadsman / laravel-plugin

src/Entities/BaseEntity.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use ArrayAccess;
 use Illuminate\Contracts\Support\Arrayable;
 use Illuminate\Contracts\Support\Jsonable;
-use Illuminate\Support\Arr;
 use JsonSerializable;
 
 /**

src/Entities/RouteInfo.php

Doc Comments +4 added lines

@@ -239,6 +239,10 @@
         return $this->route->uri();
     }
 
+    /**
+     * @param string $type
+     * @param string $text
+     */
     protected function setError($type, $text, $params = [])
     {
         $this->errors[$type] = trans($text, $params);

Spacing +6 added lines, -6 removed lines

@@ -109,7 +109,7 @@ 
     {
         $reflection = $this->getActionReflection();
 
-        if (! is_null($reflection)) {
+        if (!is_null($reflection)) {
             return $reflection->getDocComment();
         }
 
@@ -139,7 +139,7 @@ 
             }
 
             // We don't care if argument is anything but FormRequest
-            if (! is_subclass_of($class->name, FormRequest::class)) {
+            if (!is_subclass_of($class->name, FormRequest::class)) {
                 continue;
             }
 
@@ -188,14 +188,14 @@ 
             list($controller, $action) = explode('@', $uses);
 
             // Controller is missing.
-            if (! class_exists($controller)) {
+            if (!class_exists($controller)) {
                 $this->setError('uses', 'controller does not exists');
 
                 return null;
             }
 
             // Action is missing.
-            if (! method_exists($controller, $action)) {
+            if (!method_exists($controller, $action)) {
                 $this->setError('uses', 'controller@method does not exists');
 
                 return null;
@@ -229,8 +229,8 @@ 
      *
      * @return string
      */
-    protected function getUri(){
-        if (method_exists($this->route, 'getPath')){
+    protected function getUri() {
+        if (method_exists($this->route, 'getPath')) {
             // Laravel <5.4
             return $this->route->getPath();
         }

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);
         }
 

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/Providers/RouteServiceProvider.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 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;

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';
     }
 
     /**

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/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);

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();