Pamo\Comment\HTMLForm\UpdateCommentForm

Complexity

Total Complexity

6

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	callbackSuccess()	0	3	1
A	__construct()	0	39	1
A	callbackSubmit()	0	9	2
A	getItemDetails()	0	6	2

<?php

namespace Pamo\Comment\HTMLForm;

use Anax\HTMLForm\FormModel;
use Psr\Container\ContainerInterface;
use Pamo\Comment\CommentQ;
use Pamo\Comment\CommentA;

/**
 * Form to update an item.
 */
class UpdateCommentForm extends FormModel
{
    /**
     * Constructor injects with DI container and the id to update.
     *
     * @param Psr\Container\ContainerInterface $di a service container

The type Pamo\Comment\HTMLForm\Psr\Container\ContainerInterface was not found. Did you mean Psr\Container\ContainerInterface? If so, make sure to prefix the type with \.

     * @param integer             $id to update
     */
    public function __construct(ContainerInterface $di, $type, $commentId)
    {
        parent::__construct($di);
        $comment = $this->getItemDetails($type, $commentId);

        $this->questionid = $comment->questionid;

The property questionid does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->form->create(
            [
                "id" => __CLASS__,
                "legend" => "Update a comment",
            ],
            [
                "id" => [
                    "type" => "hidden",
                    "value" => $comment->id,
                ],

                "type" => [
                    "type" => "hidden",
                    "value" => $type,
                ],

                "text" => [
                    "type" => "text",
                    "label" => false,
                    "value" => $comment->text
                ],

                "submit" => [
                    "type" => "submit",
                    "value" => "Save",
                    "callback" => [$this, "callbackSubmit"]
                ],

                "cancel" => [
                    "type" => "button",
                    "value" => "Cancel",
                    "onclick" => "location.href='$this->questionid';"
                ],
            ]
        );
    }



    /**
     * Get details on item to load form with.
     *
     * @param integer $id get details on item with id.
     *
     * @return $comment

The doc comment $comment at position 0 could not be parsed: Unknown type name '$comment' at position 0 in $comment.

     */
    public function getItemDetails($type, $commentId) : object
    {
        $comment = $type === "question" ? new CommentQ() : new CommentA();
        $comment->setDb($this->di->get("dbqb"));
        $comment->find("id", $commentId);
        return $comment;
    }



    /**
     * Callback for submit-button which should return true if it could
     * carry out its work and false if something failed.
     *
     * @return bool true if okey, false if something went wrong.
     */
    public function callbackSubmit() : bool
    {
        $type = $this->form->value("type");
        $comment = $type === "question" ? new CommentQ() : new CommentA();
        $comment->setDb($this->di->get("dbqb"));
        $comment->find("id", $this->form->value("id"));
        $comment->text = $this->form->value("text");
        $comment->save();
        return true;
    }



    /**
     * Callback what to do if the form was successfully submitted, this
     * happen when the submit callback method returns true. This method
     * can/should be implemented by the subclass for a different behaviour.
     */
    public function callbackSuccess()
    {
        $this->di->get("response")->redirect("game/question/$this->questionid")->send();
    }



    // /**
    //  * Callback what to do if the form was unsuccessfully submitted, this
    //  * happen when the submit callback method returns false or if validation
    //  * fails. This method can/should be implemented by the subclass for a
    //  * different behaviour.
    //  */
    // public function callbackFail()
    // {
    //     $this->di->get("response")->redirectSelf()->send();
    // }
}