nikhil-pandey / guitar-shop

app/Http/Controllers/GuitarController.php

<?php

namespace App\Http\Controllers;

use App\Events\GuitarWasCreated;
use App\Events\WarehouseWasUpdated;
use App\Guitar;
use App\Purchase;
use App\Rack;
use Illuminate\Http\Request;

class GuitarController extends Controller
{
    /**
     * Save the guitar.
     *
     * @param $id
     */
    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.']],
                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,
            'colour'    => 'required|string|max:255',
            'damaged'   => 'boolean',
            'condition' => 'required_if:damaged,true',
            'price'     => 'required|numeric|min:1',
        ]);

        $rack = Rack::find($request->input('rack_id'));
        if ($rack->used >= $rack->capacity) {
            return response(['rack_id' => ['The selected rack is full.']], 422);
        }

        $guitar = Guitar::create($request->only(['rack_id', 'model_id', 'colour', 'price']) + [
                'make_id'     => $purchase->make_id,
                'purchase_id' => $purchase->id,
                'damaged'     => $request->input('damaged'),
                'condition'   => $request->input('damaged') ? $request->input('condition') : null
            ]);

        event(new GuitarWasCreated($guitar));
        event(new WarehouseWasUpdated($rack->warehouse));

The call to WarehouseWasUpdated::__construct() has too many arguments starting with $rack->warehouse.

    }
}