parsnick / eloquentjs

src/Console/GenerateJavascript.php

Unused Use Statements -2 removed lines

@@ -2,9 +2,7 @@
 
 namespace EloquentJs\Console;
 
-use EloquentJs\Generator\EndpointLocator;
 use EloquentJs\Generator\Generator;
-use EloquentJs\Generator\ModelFinder;
 use EloquentJs\Generator\ModelInputParser;
 use Illuminate\Console\Command;
 

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
      */
     protected function printMapping($models)
     {
-        $rows = array_map(function ($model, $endpoint) {
+        $rows = array_map(function($model, $endpoint) {
             return [$model, $endpoint];
         }, array_keys($models), $models);
 
@@ -94,7 +94,7 @@ 
      */
     protected function populateMissingEndpoints(&$models)
     {
-        array_walk($models, function (&$endpoint, $model) {
+        array_walk($models, function(&$endpoint, $model) {
             if (empty($endpoint)) {
                 $endpoint = $this->ask("Enter the endpoint to use for the '{$model}' model");
             }

src/Controllers/GenericResourceController.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      * Store a newly created resource in storage.
      *
      * @param  \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
+     * @return Model
      */
     public function store(Request $request)
     {

src/EloquentJsServiceProvider.php

Indentation +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
     public function register()
     {
         $this->app->bind(QueryTranslator::class, function ($app) {
-           return new JsonQueryTranslator($app['request']->input('query', '[]'));
+            return new JsonQueryTranslator($app['request']->input('query', '[]'));
         });
 
         $this->commands([GenerateJavascript::class]);

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function register()
     {
-        $this->app->bind(QueryTranslator::class, function ($app) {
+        $this->app->bind(QueryTranslator::class, function($app) {
            return new JsonQueryTranslator($app['request']->input('query', '[]'));
         });
 
@@ -49,7 +49,7 @@ 
      */
     protected function addGenericResourceRouting(Router $router)
     {
-        $router->macro('eloquent', function ($uri, $resource, $options = []) use ($router) {
+        $router->macro('eloquent', function($uri, $resource, $options = []) use ($router) {
 
             // The $router->resource() method doesn't allow custom route attributes
             // in the $options array. So, while the group() may look redundant here,
@@ -58,7 +58,7 @@ 
             //
             // This is so when we come to resolve the controller (see below), we
             // can easily tell what type of resource we need, i.e. which model.
-            $router->group(compact('resource'), function ($router) use ($uri, $options) {
+            $router->group(compact('resource'), function($router) use ($uri, $options) {
 
                 if (empty($options['only'])) { // Exclude the routes for displaying forms
                     $options['except'] = (array) array_get($options, 'except', []) + ['create', 'edit'];
@@ -72,7 +72,7 @@ 
         // Typically you'd have dedicated controllers for each resource.
         // Since that's not the case here, we need some way of telling
         // our generic controller which resource we're working with.
-        $this->app->resolving(function (GenericResourceController $controller) {
+        $this->app->resolving(function(GenericResourceController $controller) {
 
             $currentRoute = $this->app['router']->current();
 

src/Generator/Generator.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     /**
      * @type string location of the base EloquentJs build
      */
-    const BASE_BUILD = __DIR__ . '/../../eloquent.js';
+    const BASE_BUILD = __DIR__.'/../../eloquent.js';
 
     /**
      * @param Filesystem $files

src/Generator/ModelFinder.php

Spacing +2 added lines, -2 removed lines

@@ -41,8 +41,8 @@
     public function inList(array $classes, $namespace = null)
     {
         if ($namespace) {
-            array_walk($classes, function (&$className) use ($namespace) {
-                $className = str_replace('\\\\', '\\', $namespace . '\\' . $className);
+            array_walk($classes, function(&$className) use ($namespace) {
+                $className = str_replace('\\\\', '\\', $namespace.'\\'.$className);
             });
         }
 

src/Generator/ModelInspector.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
      */
     public function getScopes()
     {
-        return array_map(function ($method) {
+        return array_map(function($method) {
             return lcfirst(substr($method, 5));
         }, $this->getScopeMethods());
     }
@@ -63,7 +63,7 @@ 
     protected function getScopeMethods()
     {
         return array_values(
-            array_filter(get_class_methods($this->model), function ($method) {
+            array_filter(get_class_methods($this->model), function($method) {
                 return substr($method, 0, 5) === 'scope' && ! in_array($method, ['scopeScope', 'scopeUseEloquentJs']);
             })
         );