Treestoneit\ShoppingCart\Concerns\AttachesToUsers

Complexity

Total Complexity

6

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	loadUserCart()	0	17	3
A	attachTo()	0	12	2
A	overwrite()	0	10	1

<?php

namespace Treestoneit\ShoppingCart\Concerns;

use Illuminate\Contracts\Auth\Authenticatable;
use Treestoneit\ShoppingCart\Models\Cart as CartModel;

trait AttachesToUsers
{
    /**
     * Load the given user's shopping cart.
     *
     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
     * @return static
     */
    public function loadUserCart(Authenticatable $user): self
    {
        // If the user doesn't yet have a saved cart, we'll
        // just attach the current one to the user and exit.
        if (! $cart = CartModel::whereUserId($user->getAuthIdentifier())->with('items')->first()) {
            return $this->attachTo($user);
        }

        // If the current cart is empty, we'll load the saved one.
        if ($this->items()->isEmpty()) {

It seems like items() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return $this->refreshCart($cart);

It seems like refreshCart() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        // Otherwise, we'll overwrite the saved cart with the current one.
        // TODO add a strategy to be able to merge with the saved cart
        return $this->overwrite($user);
    }

    /**
     * Attach the current cart to the given user.
     *
     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
     * @return static
     */
    public function attachTo(Authenticatable $user): self
    {
        $this->cart->fill([

The property cart does not exist. Did you maybe forget to declare it?

            'user_id' => $user->getAuthIdentifier(),
        ]);

        if ($this->cart->exists) {
            $this->cart->save();
        }

        return $this;
    }

    /**
     * Delete any old carts belonging to the given user and attach
     * the current cart to them.
     *
     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
     * @return static
     */
    protected function overwrite(Authenticatable $user): self
    {
        CartModel::whereUserId($user->getAuthIdentifier())

The method whereKeyNot does only exist in Illuminate\Database\Eloquent\Builder, but not in Treestoneit\ShoppingCart\Models\Cart.

                 ->whereKeyNot($this->cart->getKey())
                 // Delete them using Eloquent so that events will be fired
                 // and trigger deletion of cart items as well.
                 ->get()->each->delete();

        return $this->attachTo($user);
    }
}