Xetaravel\Models\Repositories\DiscussThreadRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	49
Duplicated Lines	20.41 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	19	4
A	update()	10	10	2

<?php
namespace Xetaravel\Models\Repositories;

use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Auth;
use Xetaravel\Models\DiscussThread;

class DiscussThreadRepository
{

    /**
     * Create the new thread and save it.
     *
     * @param array $data The data used to create the thread.
     *
     * @return \Xetaravel\Models\DiscussThread
     */
    public static function create(array $data): DiscussThread
    {
        $thread = [
            'title' => $data['title'],
            'category_id' => $data['category_id'],
            'content' => $data['content']
        ];

        $user = Auth::user();

        if ($user->hasPermission('manage.discuss.threads')) {
            $thread += [
                'is_locked' => isset($data['is_locked']) ? true : false,
                'is_pinned' => isset($data['is_pinned']) ? true : false,
            ];
        }

        return DiscussThread::create($thread);
    }

    /**
     * Update the article data and save it.
     *
     * @param array $data The data used to update the article.
     * @param \Xetaravel\Models\Article $article The article to update.

Should the type for parameter $article not be Article?

     *
     * @return \Xetaravel\Models\Article
     */
    public static function update(array $data, Article $article): Article

This method seems to be duplicated in your project.

    {
        $article->title = $data['title'];
        $article->category_id = $data['category_id'];
        $article->is_display = isset($data['is_display']) ? true : false;
        $article->content = $data['content'];
        $article->save();

        return $article;
    }
}