nikhil-pandey / guitar-shop

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/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/Console/Commands/Init.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
     {
         $this->info('Starting Installation...');
 
-        if ( ! env('APP_KEY')) {
+        if (!env('APP_KEY')) {
             $this->info('Generating app key');
             Artisan::call('key:generate');
         } else {
@@ -39,11 +39,11 @@ 
         }
 
         $this->info('Migrating database');
-        Artisan::call('migrate', ['--force' => true]);
+        Artisan::call('migrate', [ '--force' => true ]);
 
-        if ( ! User::count()) {
+        if (!User::count()) {
             $this->info('Seeding initial data');
-            Artisan::call('db:seed', ['--force' => true]);
+            Artisan::call('db:seed', [ '--force' => true ]);
         } else {
             $this->comment('Skipping - Data already seeded');
         }

app/Events/Event.php

Spacing +2 added lines, -2 removed lines

@@ -13,11 +13,11 @@
      */
     public function broadcastOn()
     {
-        return ['default'];
+        return [ 'default' ];
     }
 
     public function broadcastWith()
     {
-        return [];
+        return [ ];
     }
 }

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

app/Providers/EventServiceProvider.php

Spacing +17 added lines, -17 removed lines

@@ -13,33 +13,33 @@
      * @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\Events\GuitarWasUpdated' => [],
-        'App\Events\GuitarWasDeleted' => [],
+        'App\Events\GuitarWasCreated' => [ ],
+        '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/GuitarController.php

Spacing +13 added lines, -13 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
             ]);
@@ -118,7 +118,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, [
@@ -129,10 +129,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));
     }

database/factories/ModelFactory.php

Spacing +12 added lines, -12 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-$factory->define(App\User::class, function (Faker\Generator $faker) {
+$factory->define(App\User::class, function(Faker\Generator $faker) {
     return [
         'name'           => $faker->name,
         'email'          => $faker->email,
@@ -9,47 +9,47 @@ 
     ];
 });
 
-$factory->define(App\Warehouse::class, function (Faker\Generator $faker) {
+$factory->define(App\Warehouse::class, function(Faker\Generator $faker) {
     return [
         'name' => 'Warehouse #'.$faker->numberBetween(),
     ];
 });
 
-$factory->define(App\Make::class, function (Faker\Generator $faker) {
+$factory->define(App\Make::class, function(Faker\Generator $faker) {
     return [
         'name' => $faker->words(2, true),
     ];
 });
 
-$factory->define(App\Supplier::class, function (Faker\Generator $faker) {
+$factory->define(App\Supplier::class, function(Faker\Generator $faker) {
     return [
         'name'     => $faker->company,
         'location' => $faker->address,
     ];
 });
 
-$factory->define(App\Shop::class, function (Faker\Generator $faker) {
+$factory->define(App\Shop::class, function(Faker\Generator $faker) {
     return [
         'name'    => $faker->company,
         'address' => $faker->address,
     ];
 });
 
-$factory->define(App\Rack::class, function (Faker\Generator $faker) {
+$factory->define(App\Rack::class, function(Faker\Generator $faker) {
     return [
         'name'     => 'Rack #'.$faker->numberBetween(1, 20),
         'capacity' => $faker->randomNumber(3),
     ];
 });
 
-$factory->define(App\Model::class, function (Faker\Generator $faker) {
+$factory->define(App\Model::class, function(Faker\Generator $faker) {
     return [
         'name'  => 'Model #'.$faker->numberBetween(1, 20),
         'photo' => config('shop.images.dir').$faker->image(config('shop.images.path'), 640, 480, 'technics', null),
     ];
 });
 
-$factory->define(App\Purchase::class, function (Faker\Generator $faker) {
+$factory->define(App\Purchase::class, function(Faker\Generator $faker) {
     return [
         'quantity'       => $faker->randomNumber(2),
         'price'          => $faker->randomFloat(2, 1000),
@@ -58,7 +58,7 @@ 
     ];
 });
 
-$factory->defineAs(App\Purchase::class, 'delivered', function (Faker\Generator $faker) use ($factory) {
+$factory->defineAs(App\Purchase::class, 'delivered', function(Faker\Generator $faker) use ($factory) {
     $user = $factory->raw(App\Purchase::class);
 
     return array_merge($user, [
@@ -67,7 +67,7 @@ 
     ]);
 });
 
-$factory->define(App\Guitar::class, function (Faker\Generator $faker) {
+$factory->define(App\Guitar::class, function(Faker\Generator $faker) {
     return [
         'colour'    => $faker->colorName,
         'damaged'   => $faker->boolean(),
@@ -76,14 +76,14 @@ 
     ];
 });
 
-$factory->define(App\Sale::class, function (Faker\Generator $faker) {
+$factory->define(App\Sale::class, function(Faker\Generator $faker) {
     return [
         'name'    => $faker->name,
         'address' => $faker->address,
     ];
 });
 
-$factory->defineAs(App\Sale::class, 'shop', function (Faker\Generator $faker) {
+$factory->defineAs(App\Sale::class, 'shop', function(Faker\Generator $faker) {
     return [
         'shop_id' => factory(App\Shop::class)->create()->id,
     ];

app/Guitar.php

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
     /**
      * @var array
      */
-    protected $dates = ['deleted_at'];
+    protected $dates = [ 'deleted_at' ];
 
     /**
      * We do not want any timestamps.
@@ -25,7 +25,7 @@ 
      *
      * @var array
      */
-    protected $guarded = [];
+    protected $guarded = [ ];
 
     /**
      * Sold guitars.
@@ -107,6 +107,6 @@ 
      */
     public function isSold()
     {
-        return ! is_null($this->sale);
+        return !is_null($this->sale);
     }
 }