Forum\Comment\CommentController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	77
Duplicated Lines	41.56 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	questionAction()	16	16	1
A	answerAction()	16	16	1
A	indexActionGet()	0	8	2

<?php

namespace Forum\Comment;

use Anax\Commons\ContainerInjectableInterface;
use Anax\Commons\ContainerInjectableTrait;
use Forum\Comment\HTMLForm\CreateForm;
use Forum\Answer\Answer;
use Forum\Question\Question;

// use Anax\Route\Exception\ForbiddenException;
// use Anax\Route\Exception\NotFoundException;
// use Anax\Route\Exception\InternalErrorException;

/**
 * A sample controller to show how a controller class can be implemented.
 */
class CommentController implements ContainerInjectableInterface
{
    use ContainerInjectableTrait;


    /**
     * @var $data description
     */
    //private $data;


    // /**
    //  * The initialize method is optional and will always be called before the
    //  * target method/action. This is a convienient method where you could
    //  * setup internal properties that are commonly used by several methods.
    //  *
    //  * @return void
    //  */
    // public function initialize() : void
    // {
    //     ;
    // }


    /**
     * Description.
     *
     * @param datatype $variable Description

There is no parameter named $variable. Was it maybe removed?

     *
     * @return object as a response object
     * @throws Exception
     *
     */
    public function indexActionGet(): object
    {
        $user_id = $this->di->get("session")->get("user_id");
        var_dump($user_id);

var_dump($user_id); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        if (!$user_id) {
            $this->di->get("response")->redirect("user/login");
        }
    }

    public function questionAction(int $questionId): object

This method seems to be duplicated in your project.

    {
        $page = $this->di->get("page");
        $form = new CreateForm($this->di, $questionId, "");
        $question = new Question();
        $question->setDb($this->di->get("dbqb"));
        $form->check();
        $text = $question->findById($questionId)->question;
        $data = [
            "form" => $form->getHTML(),
            "text" => $text,
            "title" => "create comment",
        ];
        $page->add("forum/create_comment", $data);
        return $page->render($data);
    }


    public function answerAction(int $answerId): object

This method seems to be duplicated in your project.

    {
        $page = $this->di->get("page");
        $form = new CreateForm($this->di, "", $answerId);
        $answer = new Answer();
        $answer->setDb($this->di->get("dbqb"));
        $form->check();
        $text = $answer->findById($answerId)->answer;
        $data = [
            "form" => $form->getHTML(),
            "text" => $text,
            "title" => "create comment",
        ];
        $page->add("forum/create_comment", $data);
        return $page->render($data);
    }
}