CommentAnswerAdd::check() - Code Metrics - phpffcms/ffcms - Measure and Improve Code Quality continuously with Scrutinizer

CommentAnswerAdd::check() C
last analyzed 2019-02-04 19:14 UTC

↳ Parent: CommentAnswerAdd

Complexity

Conditions	16
Paths	18

Size

Total Lines	54
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	16
eloc	29
nc	18
nop	0
dl	0
loc	54
rs	5.5666
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

namespace Apps\Model\Api\Comments;

use Apps\ActiveRecord\CommentAnswer;
use Apps\ActiveRecord\CommentPost;
use Apps\Model\Front\Profile\EntityAddNotification;
use Ffcms\Core\App;
use Ffcms\Core\Arch\Model;
use Ffcms\Core\Exception\JsonException;
use Ffcms\Core\Helper\Date;
use Ffcms\Core\Helper\Text;
use Ffcms\Core\Helper\Type\Str;

/**
 * Class CommentAnswerAdd. Model to add comment answer to database and build response active record row.
 * @package Apps\Model\Api\Comments
 */
class CommentAnswerAdd extends Model
{
    public $message;
    public $replayTo = 0;
    public $guestName;
    public $ip;

    private $_configs;
    private $_userId = 0;

    public function __construct(array $configs)
    {
        $this->_configs = $configs;
        parent::__construct();
    }

    public function before()
    {
        $this->ip = App::$Request->getClientIp();
        if (App::$User->isAuth()) {
            $this->_userId = App::$User->identity()->getId();
        }
    }

    /**
     * Check if comment answer conditions is ok. Will throw exception if not.
     * @return bool
     * @throws JsonException
     */
    public function check()
    {
        // check if user is auth'd or guest name is defined
        if (!App::$User->isAuth() && ((int)$this->_configs['guestAdd'] !== 1 || Str::length($this->guestName) < 2)) {
            throw new JsonException(__('Guest name is not defined'));
        }

        // guest moderation
        if (!App::$User->isAuth() && (bool)$this->_configs['guestModerate']) {
            $captcha = App::$Request->request->get('captcha');
            if (!App::$Captcha->validate($captcha)) {
                throw new JsonException(__('Captcha is incorrect! Click on image to refresh and try again'));
            }
        }

        // check if replayTo is defined
        if ($this->replayTo < 1) {
            throw new JsonException(__('Comment post thread is not founded'));
        }

        // check if message length is correct
        if (Str::length($this->message) < (int)$this->_configs['minLength'] || Str::length($this->message) > (int)$this->_configs['maxLength']) {
            throw new JsonException(__('Message length is incorrect. Current: %cur%, min - %min%, max - %max%', [
                'cur' => Str::length($this->message),
                'min' => $this->_configs['minLength'],
                'max' => $this->_configs['maxLength']
            ]));
        }

        $count = CommentPost::where('id', '=', $this->replayTo)->count();
        if ($count !== 1) {
            throw new JsonException(__('Comment post thread is not founded'));
        }

        // check to prevent spam
        $query = CommentAnswer::where('user_id', $this->_userId)
            ->orWhere('ip', $this->ip)
            ->orderBy('created_at', 'DESC')
            ->first();

        // something is founded :D
        if ($query) {
            $isModerator = false;
            if (App::$User->isAuth() && App::$User->identity()->role->can('global/modify')) {
                $isModerator = true;
            }
            $answerTime = Date::convertToTimestamp($query->created_at);
            $answerTime = Date::convertToTimestamp(/** @scrutinizer ignore-type */ $query->created_at);
            $delay = $answerTime + $this->_configs['delay'] - time();
            if ($delay > 0 && !$isModerator) { // sounds like config time is not passed now
                throw new JsonException(__('Spam protection: please, wait %sec% seconds', ['sec' => $delay]));
            }
        }

        return true;
    }

    /**
     * Add comment answer to database and return active record object
     * @return CommentAnswer
     */
    public function buildRecord()
    {
        $record = new CommentAnswer();
        $record->comment_id = $this->replayTo;
        $record->user_id = $this->_userId;
        $record->guest_name = $this->guestName;
        $record->message = $this->message;
        $record->lang = App::$Request->getLanguage();
        $record->ip = $this->ip;
        // check if premoderation is enabled and user is guest
        if ((bool)$this->_configs['guestModerate'] && $this->_userId < 1) {
            $record->moderate = 1;
$answer = 42;

$correct = false;

$correct = (bool) $answer;
        }
        $record->save();

        // add notification for comment post owner
        $commentPost = $record->post;
        if ($commentPost !== null && (int)$commentPost->user_id !== 0 && (int)$commentPost->user_id !== $this->_userId) {
            $notify = new EntityAddNotification((int)$commentPost->user_id);
            $notify = new EntityAddNotification(/** @scrutinizer ignore-type */ (int)$commentPost->user_id);
            $notify->add($commentPost->pathway, EntityAddNotification::MSG_ADD_COMMENTANSWER, [

                'snippet' => Text::snippet($this->message, 50),
                'post' => Text::snippet($commentPost->message, 50)
            ]);
        }

        return $record;
    }
}


phpffcms / ffcms

CommentAnswerAdd::check() C last analyzed 2019-02-04 19:14 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

CommentAnswerAdd::check() C
last analyzed 2019-02-04 19:14 UTC