Rinvex\Cacheable\BelongsToMany

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	get()	0	39	4

<?php

declare(strict_types=1);

namespace Rinvex\Cacheable;

use Illuminate\Database\Eloquent\Relations\BelongsToMany as BaseBelongsToMany;

class BelongsToMany extends BaseBelongsToMany
{
    /**
     * Execute the query as a "select" statement.
     *
     * @param  array $columns
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function get($columns = ['*'])
    {
        $builder = $this->query->applyScopes();

        $closure = function () use ($builder, $columns) {
            // First we'll add the proper select columns onto the query so it is run with
            // the proper columns. Then, we will get the results and hydrate out pivot
            // models with the result of those columns as a separate model relation.
            $columns = $this->query->getQuery()->columns ? [] : $columns;

Consider using a different name than the imported variable $columns, or did you forget to import by reference?

            $models = $builder->addSelect(
                $this->shouldSelect($columns)
            )->getModels();

            $this->hydratePivotRelation($models);

            // If we actually found models we will also eager load any relationships that
            // have been specified as needing to be eager loaded. This will solve the
            // n + 1 query problem for the developer and also increase performance.
            if (count($models) > 0) {
                $models = $builder->eagerLoadRelations($models);
            }

            return $this->related->newCollection($models);
        };

        // Check if cache is enabled
        if ($builder->getModel()->getCacheLifetime()) {
            return $builder->getModel()->cacheQuery($builder, $columns, $closure);
        }

        // Cache disabled, just execute query & return result
        $results = call_user_func($closure);

        // We're done, let's clean up!
        $builder->getModel()->resetCacheConfig();

        return $results;
    }
}