Radchasay\User\HTMLForm\UpdateProfileForm

Complexity

Total Complexity

3

Size/Duplication

Total Lines	91
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	44	44	1
A	getItemDetails()	7	7	1
A	callbackSubmit()	14	14	1

<?php

namespace Radchasay\User\HTMLForm;

use \Anax\HTMLForm\FormModel;
use \Anax\DI\DIInterface;
use \Radchasay\User\User;

/**
 * Form to update an item.
 */
class UpdateProfileForm extends FormModel

This class seems to be duplicated in your project.

{
    /**
     * Constructor injects with DI container and the id to update.
     *
     * @param Anax\DI\DIInterface $di a service container
     * @param integer             $id to update
     */
    public function __construct(DIInterface $di, $id)
    {
        parent::__construct($di);
        $profileInfo = $this->getItemDetails($id);
        $this->form->create(
            [
                "id"     => __CLASS__,
                "legend" => "Update details of the item",
            ],
            [
                "id"    => [
                    "type"     => "hidden",
                    "value"    => $profileInfo->id,
                ],
                "email" => [
                    "type"       => "text",
                    "validation" => ["not_empty"],
                    "value"      => $profileInfo->email,
                ],

                "name" => [
                    "type"       => "text",
                    "validation" => ["not_empty"],
                    "value"      => $profileInfo->name,
                ],

                "age" => [
                    "type"       => "number",
                    "validation" => ["not_empty"],
                    "value"      => $profileInfo->age,
                ],

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

                "reset" => [
                    "type" => "reset",
                ],
            ]
        );
    }


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

The doc-type $user could not be parsed: Unknown type name "$user" at position 0. (view supported doc-types)

     */
    public function getItemDetails($id)
    {
        $user = new User();
        $user->setDb($this->di->get("db"));
        $user->find("id", $id);
        return $user;
    }


    /**
     * Callback for submit-button which should return true if it could
     * carry out its work and false if something failed.
     *
     * @return boolean true if okey, false if something went wrong.
     */
    public function callbackSubmit()
    {
        $user = new User();
        $user->setDb($this->di->get("db"));
        $user->find("id", $this->form->value("id"));
        $user->email = htmlentities($this->form->value("email"));
        $user->name = htmlentities($this->form->value("name"));
        $user->age = htmlentities($this->form->value("age"));
        $user->save();
        $this->di->get("session")->set("email", $user->email);
        $url = $this->di->get("url")->create("user/login");
        $this->di->get("response")->redirect($url);
        //$this->di->get("response")->redirect($url . "/update/{$book->id}");

67% of this comment could be valid code. Did you maybe forget this after debugging?

    }
}