nikhil-pandey / guitar-shop

app/Http/Controllers/PurchaseController.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
     {
         $this->validate($request, $this->rules($request->input('supplier_id')));
         $purchase = $this->getSupplier($request)->makePurchase(
-            $request->only(['id', 'make_id', 'quantity', 'price', 'date_purchased', 'delivery_date'])
+            $request->only([ 'id', 'make_id', 'quantity', 'price', 'date_purchased', 'delivery_date' ])
         );
         event(new PurchaseWasCreated($purchase));
     }
@@ -75,7 +75,7 @@ 
             'supplier.location' => 'required_if:supplier_id,new|string|max:255'
         ];
         if ($supplierId != 'new') {
-            $rules += ['supplier_id' => 'required|integer|exists:suppliers,id'];
+            $rules += [ 'supplier_id' => 'required|integer|exists:suppliers,id' ];
         }
 
         return $rules;

app/Http/Controllers/MakeController.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             'name' => 'required|string|max:255'
         ]);
 
-        $make = Make::create($request->only(['name']));
+        $make = Make::create($request->only([ 'name' ]));
         event(new MakeWasCreated($make));
     }
 

app/Http/Controllers/ModelController.php

Spacing +2 added lines, -2 removed lines

@@ -41,12 +41,12 @@
             'photo'   => 'required|image',
         ]);
         $photo    = $request->file('photo');
-        $fileName = sha1(time() . $photo->getClientOriginalName()) . '.' . $photo->getClientOriginalExtension();
+        $fileName = sha1(time().$photo->getClientOriginalName()).'.'.$photo->getClientOriginalExtension();
         $photo->move(config('shop.images.path'), $fileName);
         $model = Model::create($request->only([
                 'name',
                 'make_id'
-            ]) + ['photo' => config('shop.images.dir') . $fileName]);
+            ]) + [ 'photo' => config('shop.images.dir').$fileName ]);
         event(new ModelWasCreated($model));
     }
 

app/Http/Controllers/SupplierController.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
             'location' => 'required|string|max:255',
         ]);
 
-        $supplier = Supplier::create($request->only(['name', 'location']));
+        $supplier = Supplier::create($request->only([ 'name', 'location' ]));
         event(new SupplierWasCreated($supplier));
     }
 

app/Notification.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
      *
      * @var array
      */
-    protected $guarded = [];
+    protected $guarded = [ ];
 
     /**
      * @return \Illuminate\Database\Eloquent\Relations\MorphTo
@@ -47,6 +47,6 @@ 
      */
     public function dismiss()
     {
-        return $this->update(['dismissed' => true]);
+        return $this->update([ 'dismissed' => true ]);
     }
 }

app/Transformers/GuitarTransformer.php

Doc Comments +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
      *
      * @param Guitar $guitar
      *
-     * @return \League\Fractal\Resource\Collection
+     * @return \League\Fractal\Resource\Item
      */
     public function includePurchase(Guitar $guitar)
     {
@@ -68,7 +68,7 @@ 
      *
      * @param Guitar $guitar
      *
-     * @return \League\Fractal\Resource\Collection
+     * @return \League\Fractal\Resource\Item
      */
     public function includeSale(Guitar $guitar)
     {

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @var array
      */
-    protected $availableIncludes = ['purchase', 'sale'];
+    protected $availableIncludes = [ 'purchase', 'sale' ];
 
     /**
      * Turn this item object into a generic array.
@@ -44,10 +44,10 @@ 
                 'capacity' => $guitar->rack->capacity,
             ],
             'colour'    => $guitar->colour,
-            'price'     => (float)$guitar->price,
-            'damaged'   => (bool)$guitar->damaged,
+            'price'     => (float) $guitar->price,
+            'damaged'   => (bool) $guitar->damaged,
             'condition' => $guitar->condition,
-            'sold'      => (bool)$guitar->isSold(),
+            'sold'      => (bool) $guitar->isSold(),
         ];
     }
 

app/Http/routes.php

Spacing +7 added lines, -7 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
-Route::group(['middleware' => 'web'], function ($router) {
-    $router->get('/', function () {
+Route::group([ 'middleware' => 'web' ], function($router) {
+    $router->get('/', function() {
         return redirect('/dashboard');
     });
 
@@ -34,10 +34,10 @@ 
     $router->put('api/notifications/{id}/dismiss', 'API\NotificationController@dismiss');
     $router->get('api/users/me', 'API\UserController@getCurrentUser');
 
-    $router->resource('makes', 'MakeController', ['except' => ['create', 'show', 'edit', 'update']]);
-    $router->resource('models', 'ModelController', ['except' => ['create', 'show', 'edit', 'update']]);
-    $router->resource('purchases', 'PurchaseController', ['except' => ['create', 'edit', 'update']]);
+    $router->resource('makes', 'MakeController', [ 'except' => [ 'create', 'show', 'edit', 'update' ] ]);
+    $router->resource('models', 'ModelController', [ 'except' => [ 'create', 'show', 'edit', 'update' ] ]);
+    $router->resource('purchases', 'PurchaseController', [ 'except' => [ 'create', 'edit', 'update' ] ]);
     $router->post('purchases/{id}/guitars', 'GuitarController@store');
-    $router->resource('suppliers', 'SupplierController', ['except' => ['create', 'show', 'edit', 'update']]);
-    $router->resource('warehouses', 'WarehouseController', ['except' => ['create', 'show', 'edit']]);
+    $router->resource('suppliers', 'SupplierController', [ 'except' => [ 'create', 'show', 'edit', 'update' ] ]);
+    $router->resource('warehouses', 'WarehouseController', [ 'except' => [ 'create', 'show', 'edit' ] ]);
 });

app/Http/Controllers/API/PurchaseController.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         $purchase = Purchase::findOrFail($id);
 
         return fractal()->item($purchase, new PurchaseTransformer)
-                        ->parseIncludes(['guitars', 'supplier'])
+                        ->parseIncludes([ 'guitars', 'supplier' ])
                         ->toArray();
     }
 }

app/Http/Controllers/GuitarController.php

Spacing +6 added lines, -6 removed lines

@@ -19,14 +19,14 @@ 
     public function store($id, Request $request)
     {
         $purchase = Purchase::findOrFail($id);
-        if ( ! $purchase->hasArrived() || ! $purchase->isPendingStorage()) {
-            return response(['purchase' => ['The purchase has not yet arrived or doesn\'t require addition of guitars.']],
+        if (!$purchase->hasArrived() || !$purchase->isPendingStorage()) {
+            return response([ 'purchase' => [ 'The purchase has not yet arrived or doesn\'t require addition of guitars.' ] ],
                 422);
         }
 
         $this->validate($request, [
-            'rack_id'   => 'required|exists:racks,id,make_id,' . $purchase->make_id,
-            'model_id'  => 'required|exists:models,id,make_id,' . $purchase->make_id,
+            'rack_id'   => 'required|exists:racks,id,make_id,'.$purchase->make_id,
+            'model_id'  => 'required|exists:models,id,make_id,'.$purchase->make_id,
             'colour'    => 'required|string|max:255',
             'damaged'   => 'boolean',
             'condition' => 'required_if:damaged,true',
@@ -35,10 +35,10 @@ 
 
         $rack = Rack::find($request->input('rack_id'));
         if ($rack->used >= $rack->capacity) {
-            return response(['rack_id' => ['The selected rack is full.']], 422);
+            return response([ 'rack_id' => [ 'The selected rack is full.' ] ], 422);
         }
 
-        $guitar = Guitar::create($request->only(['rack_id', 'model_id', 'colour', 'price']) + [
+        $guitar = Guitar::create($request->only([ 'rack_id', 'model_id', 'colour', 'price' ]) + [
                 'make_id'     => $purchase->make_id,
                 'purchase_id' => $purchase->id,
                 'damaged'     => $request->input('damaged'),