App\Http\Api\Form\ArticleLikeForm

Complexity

Total Complexity

3

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handleInternal()	0	11	3

<?php
namespace App\Http\Api\Form;

use App\Factory\ArticleLikeFactory;

class ArticleLikeForm extends BaseArticleLikeForm
{
    use UserTrait;
    
    protected function handleInternal()
    {
        $like = $this->getLike();
        if ($like) {
            return;
        }

        // creates new like record
        $like = ArticleLikeFactory::create($this->id, $this->getUser()->getId());

It seems like $this->getUser()->getId() can also be of type string; however, parameter $userId of App\Factory\ArticleLikeFactory::create() does only seem to accept integer, maybe add an additional type check?

        if (!$like->save()) {
            $this->addErrors($like->getErrors());
        }
    }
}