Apps\Controller\Api\Comments\ActionAdd::add()

Complexity

Conditions	4
Paths	4

Size

Total Lines	34
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function add(): ?string
    {
        $this->setJsonHeader();
        $configs = AppRecord::getConfigs('widget', 'Comments');

        $replayTo = (int)$this->request->request->get('replay-to');
        $model = null;
        // check if its a answer (comment answer type)
        if ($replayTo > 0) {
            $model = new CommentAnswerAdd($configs);

It seems like $configs can also be of type null and string; however, parameter $configs of Apps\Model\Api\Comments\CommentAnswerAdd::__construct() does only seem to accept array, maybe add an additional type check?

            $model->replayTo = $replayTo;
        } else { // sounds like new comment row
            $model = new CommentPostAdd($configs);

It seems like $configs can also be of type null and string; however, parameter $configs of Apps\Model\Api\Comments\CommentPostAdd::__construct() does only seem to accept array, maybe add an additional type check?

            $model->appName = App::$Security->strip_tags($this->request->request->get('app_name'));
            $model->appId = (int)$this->request->request->get('app_id');
        }

        // pass general comment params to model
        $model->message = App::$Security->secureHtml((string)$this->request->request->get('message'));
        $model->guestName = App::$Security->strip_tags($this->request->request->get('guest-name'));

        // check model conditions before add new row
        if (!$model|| !$model->check()) {
            throw new NativeException('Unknown error');
        }

        // add comment post or answer to database and get response active record row
        $record = $model->buildRecord();
        // pass row to entity builder model
        $response = new EntityCommentData($record);

        return json_encode([
            'status' => 1,
            'data' => $response->make() // build row to standard format
        ]);