App\Http\Requests\CheckinRequest::rules()

Complexity

Conditions	9
Paths	8

Size

Total Lines	65
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function rules()
    {
        $library = \Auth::user();
        $alma = app(AlmaClient::class);

        // 1. If we're given a loan id, look for that.

        if ($this->input('loan')) {
            $loan = Loan::with(['item', 'item.thing', 'user'])
                ->find($this->input('loan'));
            if (is_null($loan)) {
                // Bail out
                return [
                    'loan' => [new LoanExists()],
                ];
            }
            // Success, we have located an active local loan!
            $this->loan = $loan;

The property loan does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return [];
        }

        // 2. Otherwise, start by looking up the item by barcode, locally or in Alma.

        $barcode = $this->input('barcode');

        if (is_null($barcode)) {
            // Bail out, but no need for error, controller will take care of this case.
            return [];
        }

        $item = Item::withTrashed()->where('barcode', '=', $barcode)->first();

        if (is_null($item)) {
            // Item doesn't exist locally, but perhaps in Alma?
            if (empty($library->library_code) || is_null($alma->key)) {

The property library_code does not exist on App\User. Did you mean library_id?

                // Alma integration is not configured. Bail out.
                return [
                    'barcode' => [new ThingExists()],
                ];
            }

            $almaItem = $alma->items->fromBarcode($barcode);
            if (is_null($almaItem)) {
                // Nope, not in Alma either. Bail out.
                return [
                    'barcode' => [new ThingExists()],
                ];
            }
            // Success, we have located an item in Alma that does not exist locally.
            $this->almaItem = $almaItem;

The property almaItem does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return [];
        }

        // 3. At this point we have a local item (deleted or not). Let's see if it's on loan, or have been.

        $lastLoan = $item->loans()->withTrashed()->first();

        if (!is_null($lastLoan)) {
            $this->loan = $lastLoan; // This will be the newest one
            return [];
        }

        // 4. The item exists, but has never been loaned out.
        return [
            'barcode' => [new NeverLoanedOut($item)],
        ];