Ergare17\Articles\Http\Controllers\APIArticlesController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	4	1
A	store()	0	14	1
A	destroy()	0	5	1
A	update()	0	9	1
A	show()	0	4	1

<?php

namespace Ergare17\Articles\Http\Controllers;

use Ergare17\Articles\Http\Requests\DestroyArticle;
use Ergare17\Articles\Http\Requests\ListArticle;
use Ergare17\Articles\Http\Requests\ShowArticle;
use Ergare17\Articles\Http\Requests\StoreArticle;
use Ergare17\Articles\Http\Requests\UpdateArticle;
use Ergare17\Articles\Models\Article;

class APIArticlesController extends Controller
{
    public function index(ListArticle $request)
    {
        return Article::all();
    }

    public function show(ShowArticle $request, Article $article)
    {
        return $article;
    }

    // Injeccció de depèndències
    public function store(StoreArticle $request)
    {
        $request->validate([
            'title' => 'required',
        ]);

        $article = Article::create([

The method create() does not exist on Ergare17\Articles\Models\Article. Did you maybe mean created()?

            'title' => $request->title,
            'description' => $request->description,
            'user_id' => $request->user_id
        ]);

        return $article;
    }

    public function destroy(DestroyArticle $request, Article $article)
    {
        $article->delete();
        return $article;
    }

    public function update(UpdateArticle $request, Article $article)
    {
        $request->validate([
            'title' => 'required'
        ]);
        $article->title = $request->title;

The property title does not exist on object<Ergare17\Articles\Models\Article>. Since you implemented __set, maybe consider adding a @property annotation.

        $article->save();
        return $article;
    }
}