Nicklas\Comment\HTMLForm\Admin\EditUserForm

Complexity

Total Complexity

4

Size/Duplication

Total Lines	72
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

12.5%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	callbackSubmit()	0	18	3
B	__construct()	0	38	1

<?php

namespace Nicklas\Comment\HTMLForm\Admin;

use \Anax\HTMLForm\FormModel;
use \Anax\DI\DIInterface;
use \Nicklas\Comment\Modules\User;

/**
 * Example of FormModel implementation.
 */
class EditUserForm extends FormModel
{
    /**
     * Constructor injects with DI container.
     *
     * @param Anax\DI\DIInterface $di a service container
     */
    public function __construct(DIInterface $di, $id)
    {
        parent::__construct($di);
        $this->user = new User();

The property user does not exist. Did you maybe forget to declare it?

        $this->user->setDb($this->di->get("db"));
        $this->user->find("id", $id);
        $user = $this->user;
        $this->form->create(
            [
                "id" => __CLASS__,
                "fieldset" => true,
                "legend" => "Update user: $user->name"
            ],
            [

                "name" => [
                    "type"        => "text",
                    "readonly" => true,
                    "value" => $user->name,
                ],
                "email" => [
                    "type"        => "text",
                    "value" => $user->email,
                ],
                "select" => [
                    "type"        => "select",
                    "label"       => "Select authority",
                    "options"     => ["$user->authority" => $user->authority, "admin" => "admin", "user" => "user"],
                ],
                "submit" => [
                    "type" => "submit",
                    "value" => "Update",
                    "onclick"=>"return confirm('Fick du verkligen allt rätt?');",
                    "callback" => [$this, "callbackSubmit"]
                ],
            ]
        );
    }


    /**
     * 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");

        if (strpos($email, '%') !== false) {
            $this->form->addOutput("% is not allowed");
            return false;
        }

          $user = $this->user;

          $user->email = $email;
          $user->authority = $this->form->value("select") ?: "user";
          $user->save();
          $this->form->addOutput("Du uppdaterade användaren");
          return true;
    }
}