App\Repositories\InsightRepository

Complexity

Total Complexity

12

Size/Duplication

Total Lines	89
Duplicated Lines	0 %

Test Coverage

Coverage

83.78%

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	8	1
A	delete()	0	3	1
A	store()	0	10	1
A	getAll()	0	26	5
A	getById()	0	3	1
A	assignDataAttributes()	0	7	2
A	getLatest()	0	3	1

<?php

namespace App\Repositories;

use App\Models\Insight;
use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Config;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;

class InsightRepository implements InsightRepositoryInterface
{
    public function getAll(
        int $recordsPerPage = null,
        array $searchParameters = null
    ) {
        $query = Insight::orderBy('created_at', 'desc');

        if (!is_null($searchParameters)) {
            if (!empty($searchParameters['title'])) {
                $query->where(
                    'title',
                    'like',
                    '%' . $searchParameters['title'] . '%'
                );
            }
            if (!is_null($searchParameters['is_published'])) {
                $query->where('is_published', $searchParameters['is_published']);
            }
        }

        if ($recordsPerPage) {

The expression $recordsPerPage of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            $results = $query->paginate($recordsPerPage)->withQueryString();
        } else {
            $results = $query->get();
        }

        return $results;
    }

    public function getById($insightId)
    {
        return Insight::findOrFail($insightId);
    }

    public function store(array $data)
    {
        $insight = new Insight();
        $insight = self::assignDataAttributes($insight, $data);

The method App\Repositories\InsightRepository::assignDataAttributes() is not static, but was called statically.

        $insight->save();

        $insight->tags()->sync($data['tag_ids'] ?? null);

        return $insight->fresh();
    }

    public function update(array $data, int $id)
    {
        $insight = $this->getById($id);
        $insight = self::assignDataAttributes($insight, $data);

The method App\Repositories\InsightRepository::assignDataAttributes() is not static, but was called statically.

        $insight->update();

        return $insight;
    }

    public function delete(int $id)
    {
        Insight::destroy($id);
    }

    /**
     * Assign the attributes of the data array to the object
     *
     * @param \App\Models\Insight $insight
     * @param array $data
     *
     * @return \App\Models\Insight
     */
    public function assignDataAttributes(Insight $insight, array $data): Insight
    {
        $insight->title = $data['title'] ?? null;
        $insight->body = $data['body'] ?? null;
        $insight->is_published = (isset($data['is_published'])) ? 1 : 0;

        return $insight;
    }

    /**
     * Get the last 5 Insights.
     *
     * @param int $numberOfInsights
     *
     * @return Collection
     */
    public function getLatest(int $numberOfInsights): Collection
    {
        return Insight::orderBy('created_at', 'desc')->take($numberOfInsights)->get();
    }

}