Apps\Model\Front\Feedback\FormAnswerAdd

Complexity

Total Complexity

11

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	rules()	0	7	1
A	before()	0	8	2
A	make()	0	26	5
A	__construct()	0	7	2
A	labels()	0	6	1

<?php

namespace Apps\Model\Front\Feedback;

use Apps\ActiveRecord\FeedbackAnswer;
use Apps\Model\Front\Profile\EntityAddNotification;
use Ffcms\Core\App;
use Ffcms\Core\Arch\Model;
use Ffcms\Core\Helper\Text;

/**
 * Class FormAnswerAdd. Model to work with add answer to feedback post thread
 * @package Apps\Model\Front\Feedback
 */
class FormAnswerAdd extends Model
{
    public $name;
    public $email;
    public $message;

    /** @var \Apps\ActiveRecord\FeedbackPost */
    protected $_post;
    protected $_userId;
    protected $_ip;

    /**
     * FormAnswerAdd constructor. Pass active record of comment post and user id
     * @param $recordPost
     */
    public function __construct($recordPost)
    {
        $this->_post = $recordPost;
        if (App::$User->isAuth()) {
            $this->_userId = App::$User->identity()->getId();
        }
        parent::__construct();
    }

    /**
     * Define local properties if user is authorized
     */
    public function before()
    {
        if ($this->_userId > 0) {
            $user = App::$User->identity($this->_userId);
            $this->name = $user->profile->nick;
            $this->email = $user->getParam('email');
        }
        $this->_ip = App::$Request->getClientIp();
    }

    /**
     * Labels for display form
     * @return array
     */
    public function labels(): array
    {
        return [
            'name' => __('Name'),
            'email' => __('Email'),
            'message' => __('Message')
        ];
    }

    /**
     * Form validation rules
     * @return array
     */
    public function rules(): array
    {
        return [
            [['name', 'email', 'message'], 'required'],
            ['name', 'length_min', 2],
            ['email', 'email'],
            ['message', 'length_min', 10]
        ];
    }

    /**
     * Add new row to database and set post is unreaded
     */
    public function make()
    {
        // update readed marker
        $this->_post->readed = 0;

The property $readed was declared of type boolean, but 0 is of type integer. Maybe add a type cast?

        $this->_post->save();

        // add new answer row in database
        $record = new FeedbackAnswer();
        $record->feedback_id = $this->_post->id;
        $record->name = $this->name;
        $record->email = $this->email;
        $record->message = $this->message;
        if ($this->_userId > 0) {
            $record->user_id = $this->_userId;
        }
        $record->ip = $this->_ip;
        $record->save();

        // add notification msg
        $targetId = $this->_post->user_id;
        if ($targetId !== null && (int)$targetId > 0 && $targetId !== $this->_userId) {
            $notify = new EntityAddNotification($targetId);

$targetId of type integer is incompatible with the type boolean expected by parameter $targetId of Apps\Model\Front\Profile\EntityAddNotification::__construct().

            $uri = '/feedback/read/' . $this->_post->id . '/' . $this->_post->hash . '#feedback-answer-' . $record->id;
            $notify->add($uri, EntityAddNotification::MSG_ADD_FEEDBACKANSWER, [
                'snippet' => Text::snippet($this->message, 50),
                'post' => Text::snippet($this->_post->message, 50)
            ]);
        }
    }
}