+global\CacheDecorator::find()

Complexity

Conditions	6
Paths	8

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function find(string $id, string $class, array $columns = []): ?Storable
    {
        $hash = $this->getHash($class, $id);
        $model = $this->cacheRepository->find($id, $class, $columns);
        if ($model) {
            if ($columns) {

The expression $columns of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                // If you wanna specified columns...
                if (!array_diff($columns, array_keys($model->toArray()))) {
                    return $model;
                }
            } else {
                // If you wanna all columns...
                if ($this->allColumns->get($hash, false)) {
                    return $model;
                }
            }
        }
        $model = $this->repository->find($id, $class, $columns);
        $this->cacheRepository->update($model);
        if (!$columns) {

The expression $columns of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->allColumns->put($hash, true);
        }
        return $model;
    }
}