KnockoutPasswordField

Complexity

Total Complexity

1

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 10 and the first side effect is on line 2.

require_once('Common.php');

/**
 * KnockoutPasswordField
 *
 * Creates a {@link PasswordField} with an additional data-bind attribute that links to a Knockout obervable
 * @uses 'password' as the default observable
 */
class KnockoutPasswordField extends PasswordField

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    use \Knockout\Common;

    /**
     * bindingType
     *
     * KnockoutPasswordField needs either 'value' or 'textInput' as a key for the 'data-bind' HTML attribute
     *
     * @var string data-bind attribute key
     * @example  data-bind="input: name, valueUpdate: 'input'" - the binding type is: input.
     */
    protected $bindingType = "textInput";

    /**
     * casting of variables for security purposes
     *
     * @see http://docs.silverstripe.org/en/3.1/developer_guides/security/secure_coding/
     */
    protected $casting = array(
        "Observable" => "Varchar",
        "BindingType" => "Varchar",
        "OtherBindings" => "Varchar",
        "HasFocus" => "Boolean"
    );

    /**
     * Constructor
     *
     * @param string $name
     * @param null|string $title
     * @param string $value
     */
    public function __construct($name, $title = null, $value = '')
    {
        parent::__construct($name, $title, $value);
        $this->addExtraClass('password');
        $this->setTemplate('KnockoutTextField');
        $this->observable = 'password';
        $this->setAttribute('spellcheck', false);

false is of type boolean, but the function expects a string.

    }
}