nikhil-pandey / guitar-shop

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/Transformers/PurchaseTransformer.php

Spacing +14 added lines, -14 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @var array
      */
-    protected $availableIncludes = ['supplier', 'guitars'];
+    protected $availableIncludes = [ 'supplier', 'guitars' ];
 
     /**
      * Turn this item object into a generic array.
@@ -24,27 +24,27 @@ 
     public function transform(Purchase $purchase)
     {
         return [
-            'id'          => (int)$purchase->id,
-            'supplier_id' => (int)$purchase->supplier_id,
-            'quantity'    => (int)$purchase->quantity,
-            'price'       => (float)$purchase->price,
-            'stored'      => (int)$purchase->stored,
-            'sold'        => (int)$purchase->sold,
+            'id'          => (int) $purchase->id,
+            'supplier_id' => (int) $purchase->supplier_id,
+            'quantity'    => (int) $purchase->quantity,
+            'price'       => (float) $purchase->price,
+            'stored'      => (int) $purchase->stored,
+            'sold'        => (int) $purchase->sold,
             'make'        => [
-                'id'   => (int)$purchase->make->id,
-                'name' => (string)$purchase->make->name,
+                'id'   => (int) $purchase->make->id,
+                'name' => (string) $purchase->make->name,
             ],
             'purchased'   => [
                 'timestamp' => $purchase->date_purchased->getTimestamp(),
-                'date'      => (string)$purchase->date_purchased->toFormattedDateString(),
-                'readable'  => (string)$purchase->date_purchased->diffForHumans(),
+                'date'      => (string) $purchase->date_purchased->toFormattedDateString(),
+                'readable'  => (string) $purchase->date_purchased->diffForHumans(),
             ],
             'delivery'    => [
                 'timestamp' => $purchase->delivery_date->getTimestamp(),
-                'date'      => (string)$purchase->delivery_date->toFormattedDateString(),
-                'readable'  => (string)$purchase->delivery_date->diffForHumans(),
+                'date'      => (string) $purchase->delivery_date->toFormattedDateString(),
+                'readable'  => (string) $purchase->delivery_date->diffForHumans(),
             ],
-            'status'      => (string)$purchase->status,
+            'status'      => (string) $purchase->status,
         ];
     }
 

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'),