App\Http\Api\Form\ArticleLikeForm::handleInternal()

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    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());
        }
}