App\Repositories\PostRepository

Complexity

Total Complexity

20

Size/Duplication

Total Lines	178
Duplicated Lines	0 %

Test Coverage

Coverage

67.68%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getById()	0	3	1
B	getAll()	0	45	8
A	assignDataAttributes()	0	27	4
A	getBySlug()	0	3	1
A	update()	0	16	3
A	store()	0	14	2
A	delete()	0	3	1

<?php

namespace App\Repositories;

use App\Http\Requests\PostStoreRequest;
use App\Models\Post;
use Carbon\Carbon;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Config;
use Mcamara\LaravelLocalization\Facades\LaravelLocalization;

class PostRepository implements PostRepositoryInterface
{

    /**
     * Get all Posts.
     *
     * @param int|null $recordsPerPage
     * @param array|null $searchParameters
     *
     * @return \Illuminate\Support\Collection|\Illuminate\Contracts\Pagination\LengthAwarePaginator
     */
    public function getAll(int $recordsPerPage = null, array $searchParameters = null)
    {
        $query = Post::orderBy('title', 'asc');

        if (!is_null($searchParameters)) {
            if (!empty($searchParameters['title'])) {
                $query->where(
                    'title',
                    'like',
                    '%' . $searchParameters['title'] . '%'
                );
            }
            if (!empty($searchParameters['categoryId'])) {
                $query->where('category_id', $searchParameters['categoryId']);
            }
            if (!empty($searchParameters['startDate'])) {
                $startDate = Carbon::createFromFormat(
                    'd/m/Y',
                    $searchParameters['startDate']
                );
                $query->where('created_at', '>=', $startDate);
            }
            if (!empty($searchParameters['endDate'])) {
                $endDate = Carbon::createFromFormat(
                    'd/m/Y',
                    $searchParameters['endDate']
                );
                $query->where('created_at', '<=', $endDate);
            }
            if (!empty($searchParameters['status'])) {
                $query->currentStatus($searchParameters['status']);
            }
        }

        // Avoid retrieving the post used for the static image gallery
        $query->where('title->en', '!=', 'Static pages images');


        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;
    }

    /**
     * Get Post by id
     *
     * @param int $postId
     * @return Post
     */
    public function getById(int $postId): Post
    {
        return Post::findOrFail($postId);
    }

    /**
     * Get Post by slug
     *
     * @param string $postSlug
     *
     * @return Post
     */
    public function getBySlug(string $postSlug): Post
    {
        return Post::where('slug', $postSlug)->first();
    }

    /**
     * Store Post
     *
     * @param array $data
     *
     * @return Post
     * @throws \Spatie\ModelStatus\Exceptions\InvalidStatus
     */
    public function store(array $data): Post
    {
        $post = new Post();
        $post = self::assignDataAttributes($post, $data);

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

        // Creator - Logged user id or 1 for factories
        $post->user_id = !is_null(Auth::id()) ? Auth::id() : 1;

The property user_id does not exist on App\Models\Post. Did you mean user?

        $post->save();

        $post->tags()->sync($data['tag_ids'] ?? null);
        $post->setStatus('published');

        return $post->fresh();
    }

    /**
     * Update Post
     *
     * @param array $data
     * @param int $id
     *
     * @return Post
     * @throws \Spatie\ModelStatus\Exceptions\InvalidStatus
     */
    public function update(array $data, int $id): Post
    {
        $post = $this->getById($id);
        $post = self::assignDataAttributes($post, $data);

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

        $post->update();

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

        //$status = ($data['status'] == 'on') ? 'published' : 'unpublished';
        $status = (isset($data['status'])) ? 'published' : 'unpublished';
        if ($post->publishingStatus() != $status) {
            $post->setStatus($status);
        }

        return $post;
    }

    /**
     * Delete Post
     *
     * @param int $id
     * @return void
     */
    public function delete(int $id): void
    {
        Post::destroy($id);
    }

    /**
     * Assign the attributes of the data array to the object
     *
     * @param \App\Models\Post $post
     * @param array $data
     *
     * @return \App\Models\Post
     */
    public function assignDataAttributes(Post $post, array $data): Post
    {
        $post->title = $data['title'] ?? null;
        $post->category_id = $data['category_id'] ?? null;

The property category_id does not exist on App\Models\Post. Did you mean category?

        $post->intro_text = $data['intro_text'] ?? null;

        $post->body = $data['body'] ?? null;
        $post->before_content = $data['before_content'] ?? null;
        $post->after_content = $data['after_content'] ?? null;

        $post->featured = $data['featured'] ?? 0;
        $post->publish_at = $data['publish_at'] ?? null;
        $post->publish_until = $data['publish_until'] ?? null;

        if (isset($data['created_at'])) {
            $post->created_at = Carbon::createFromFormat('d/m/Y', $data['created_at']);
        }

        // Translations
        foreach (LaravelLocalization::getSupportedLocales() as $countryCode => $countryAvTrans) {
            if ($countryCode != Config::get('app.fallback_locale')) {
                $post->setTranslation('title', $countryCode, $data['title_' . $countryCode] ?? null);
                $post->setTranslation('body', $countryCode, $data['body_' . $countryCode] ?? null);
            }
        }

        return $post;
    }
}