nikhil-pandey / guitar-shop

app/Providers/EventServiceProvider.php

Spacing +16 added lines, -16 removed lines

@@ -13,35 +13,35 @@
      * @var array
      */
     protected $listen = [
-        'App\Events\MakeWasCreated' => [],
-        'App\Events\MakeWasDeleted' => [],
+        'App\Events\MakeWasCreated' => [ ],
+        'App\Events\MakeWasDeleted' => [ ],
 
-        'App\Events\ModelWasCreated' => [],
-        'App\Events\ModelWasDeleted' => [],
+        'App\Events\ModelWasCreated' => [ ],
+        'App\Events\ModelWasDeleted' => [ ],
 
-        'App\Events\WarehouseWasCreated' => [],
-        'App\Events\WarehouseWasUpdated' => [],
-        'App\Events\WarehouseWasDeleted' => [],
+        'App\Events\WarehouseWasCreated' => [ ],
+        'App\Events\WarehouseWasUpdated' => [ ],
+        'App\Events\WarehouseWasDeleted' => [ ],
 
-        'App\Events\SupplierWasCreated' => [],
-        'App\Events\SupplierWasDeleted' => [],
+        'App\Events\SupplierWasCreated' => [ ],
+        'App\Events\SupplierWasDeleted' => [ ],
 
         'App\Events\PurchaseWasCreated' => [
             'App\Listeners\GeneratePurchaseNotification',
         ],
-        'App\Events\PurchaseWasDeleted' => [],
+        'App\Events\PurchaseWasDeleted' => [ ],
 
         'App\Events\GuitarWasCreated' => [
             'App\Listeners\GenerateRackNotification',
         ],
-        'App\Events\GuitarWasUpdated' => [],
-        'App\Events\GuitarWasDeleted' => [],
+        'App\Events\GuitarWasUpdated' => [ ],
+        'App\Events\GuitarWasDeleted' => [ ],
 
-        'App\Events\SaleWasCreated' => [],
-        'App\Events\SaleWasDeleted' => [],
+        'App\Events\SaleWasCreated' => [ ],
+        'App\Events\SaleWasDeleted' => [ ],
 
-        'App\Events\NotificationWasCreated'   => [],
-        'App\Events\NotificationWasDismissed' => [],
+        'App\Events\NotificationWasCreated'   => [ ],
+        'App\Events\NotificationWasDismissed' => [ ],
     ];
 
     /**

app/Http/Controllers/API/GuitarController.php

Unused Use Statements -3 removed lines

@@ -4,9 +4,6 @@
 
 use App\Guitar;
 use App\Transformers\GuitarTransformer;
-use Illuminate\Http\Request;
-
-use App\Http\Requests;
 use App\Http\Controllers\Controller;
 
 class GuitarController extends Controller

app/Http/Controllers/GuitarController.php

Spacing +14 added lines, -14 removed lines

@@ -44,14 +44,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',
@@ -60,10 +60,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'),
@@ -85,8 +85,8 @@ 
     {
         $guitar = Guitar::findOrFail($id);
         $this->validate($request, [
-            'rack_id'   => 'required|exists:racks,id,make_id,' . $guitar->make_id,
-            'model_id'  => 'required|exists:models,id,make_id,' . $guitar->make_id,
+            'rack_id'   => 'required|exists:racks,id,make_id,'.$guitar->make_id,
+            'model_id'  => 'required|exists:models,id,make_id,'.$guitar->make_id,
             'colour'    => 'required|string|max:255',
             'damaged'   => 'boolean',
             'condition' => 'required_if:damaged,true',
@@ -95,10 +95,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->update($request->only(['rack_id', 'model_id', 'colour', 'price']) + [
+        $guitar->update($request->only([ 'rack_id', 'model_id', 'colour', 'price' ]) + [
                 'damaged'   => $request->input('damaged'),
                 'condition' => $request->input('damaged') ? $request->input('condition') : null
             ]);
@@ -114,7 +114,7 @@ 
     {
         $guitar = Guitar::findOrFail($id);
         if ($guitar->isSold()) {
-            return redirect('/sales/' . $guitar->sale->id);
+            return redirect('/sales/'.$guitar->sale->id);
         }
 
         abort(404);
@@ -133,7 +133,7 @@ 
         $guitar = Guitar::findOrFail($id);
 
         if ($guitar->isSold()) {
-            return response(['guitar_id' => ['This guitar has already been sold.']], 422);
+            return response([ 'guitar_id' => [ 'This guitar has already been sold.' ] ], 422);
         }
 
         $this->validate($request, [
@@ -144,10 +144,10 @@ 
 
         $shop = null;
         if ($request->input('customer_type') == 'shop') {
-            $shop = Shop::create($request->only(['name', 'address']))->id;
+            $shop = Shop::create($request->only([ 'name', 'address' ]))->id;
         }
 
-        $sale = Sale::create($request->only(['name', 'address']) + ['guitar_id' => $id, 'shop_id' => $shop]);
+        $sale = Sale::create($request->only([ 'name', 'address' ]) + [ 'guitar_id' => $id, 'shop_id' => $shop ]);
 
         event(new SaleWasCreated($sale));
     }

app/Http/routes.php

Spacing +8 added lines, -8 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');
     });
 
@@ -35,13 +35,13 @@ 
     $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('guitars', 'GuitarController', ['only' => ['index', 'update', 'destroy']]);
+    $router->resource('suppliers', 'SupplierController', [ 'except' => [ 'create', 'show', 'edit', 'update' ] ]);
+    $router->resource('warehouses', 'WarehouseController', [ 'except' => [ 'create', 'show', 'edit' ] ]);
+    $router->resource('guitars', 'GuitarController', [ 'only' => [ 'index', 'update', 'destroy' ] ]);
     $router->get('guitars/{id}/sale', 'GuitarController@showSaleFor');
     $router->post('guitars/{id}/sale', 'GuitarController@sale');
 });