Alvo\User\HTMLForm\UpdateUserForm

Complexity

Total Complexity

5

Size/Duplication

Total Lines	113
Duplicated Lines	6.19 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	callbackSubmit()	0	29	2
A	loadUserData()	6	6	1
A	__construct()	0	54	2

<?php

namespace Alvo\User\HTMLForm;

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

/**
 * Example of FormModel implementation.
 */
class UpdateUserForm extends FormModel
{
    /**
     * Constructor injects with DI container.
     *
     * @param Anax\DI\DIInterface $di a service container

The type Alvo\User\HTMLForm\Anax\DI\DIInterface was not found. Did you mean Anax\DI\DIInterface? If so, make sure to prefix the type with \.

     */
    public function __construct(DIInterface $di, $user = null)
    {
        parent::__construct($di);

        $this->user = $user;

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

        if (!$this->user) {
            $this->user = $this->loadUserData($di->get('session')->get('userId'));
        }
        $this->di = $di;

It seems like $di of type Anax\DI\DIInterface is incompatible with the declared type Anax\HTMLForm\Anax\DI\DIInterface of property $di.

        $this->form->create(
            [
                "id" => __CLASS__,
                "class" => "center form",
                "wrapper-element" => "div",
                "use_fieldset" => false,
                // "br-after-label" => false,
            ],
            [
                "email" => [
                    "type" => "email",
                    "value" => esc($this->user->email),
                    "validation" => [
                    //     "custom_test" => [
                    //         "message" => "User with this email is already registered",
                    //         "test" => function ($email)
                    //         {
                    //             $check = $this->user->getUser('email', $email);
                    //             return !$check;
                    //         }
                    //     ],
                        "not_empty"
                    ]
                ],

                "password" => [
                    "type"        => "password",
                    "validation" => [
                        "not_empty"
                    ],
                ],

                "password-again" => [
                    "type"        => "password",
                    "validation" => [
                        "match" => "password"
                    ],
                ],

                "submit" => [
                    "type" => "submit",
                    "value" => "Update User",
                    "callback" => [$this, "callbackSubmit"],
                    "class" => "btn btn-warning"
                ],
            ]
        );
    }



    public function loadUserData($id)

This method seems to be duplicated in your project.

    {
        $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()
    {
        // Get values from the submitted form
        $email         = $this->form->value("email");
        $password      = $this->form->value("password");
        $passwordAgain = $this->form->value("password-again");

        // Check password matches
        if ($password !== $passwordAgain) {
            $this->form->rememberValues();
            $this->form->addOutput("Password did not match.");
            return false;
        }

        $user = new User();
        $user->setDb($this->di->get("db"));
        $user->find("id", $this->user->id);
        $user->email = $email;
        $user->setPassword($password);
        $user->updated = date("Y-m-d H:i:s");
        $user->save();

        // $this->di->get("user")->logout();
        // $t = $this->di->get("user")->login($email, $password);
        // debug($t);

        $this->form->addOutput("Profile has been updated.");

        return true;
    }
}