SilverStripe\LDAP\Forms\LDAPChangePasswordForm

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	55	9

<?php

namespace SilverStripe\LDAP\Forms;

use Exception;
use SilverStripe\LDAP\Authenticators\LDAPAuthenticator;
use SilverStripe\LDAP\Services\LDAPService;
use SilverStripe\Control\Director;
use SilverStripe\Control\HTTP;
use SilverStripe\Control\HTTPResponse;
use SilverStripe\Control\RequestHandler;
use SilverStripe\Core\Config\Config;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\TextField;
use SilverStripe\ORM\ValidationResult;
use SilverStripe\Security\Member;
use SilverStripe\Security\MemberAuthenticator\ChangePasswordForm;
use SilverStripe\Security\Security;

class LDAPChangePasswordForm extends ChangePasswordForm
{
    /**
     * The sole purpose for overriding the constructor is surfacing the username to the user.
     * @param RequestHandler $controller
     * @param string $name
     * @param FieldList $fields
     * @param FieldList $actions
     */
    public function __construct($controller, $name, $fields = null, $actions = null)
    {
        parent::__construct($controller, $name, $fields, $actions);

        // Obtain the Member object. If the user got this far, they must have already been synced.
        $member = Security::getCurrentUser();
        if (!$member) {
            if ($this->getSession()->get('AutoLoginHash')) {
                $member = Member::member_from_autologinhash($this->getSession()->get('AutoLoginHash'));

It seems like $this->getSession()->get('AutoLoginHash') can also be of type array; however, parameter $hash of SilverStripe\Security\Member::member_from_autologinhash() does only seem to accept string, maybe add an additional type check?

            }

            // The user is not logged in and no valid auto login hash is available
            if (!$member) {
                $this->getSession()->clear('AutoLoginHash');
                return $this->controller->redirect($this->controller->Link('login'));
            }
        }

        $data = Injector::inst()
            ->get(LDAPService::class)
            ->getUserByGUID($member->GUID, ['samaccountname']);

        $emailField = null;
        $usernameField = null;
        if (Config::inst()->get(
            LDAPAuthenticator::class,
            'allow_email_login'
        ) === 'yes'
            && !empty($member->Email)
        ) {
            $emailField = TextField::create(
                'Email',

'Email' of type string is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

                _t(__CLASS__ . '.USERNAMEOREMAIL', 'Email'),
                $member->Email,
                null,
                $this

$this of type SilverStripe\LDAP\Forms\LDAPChangePasswordForm is incompatible with the type array expected by parameter $args of SilverStripe\View\ViewableData::create().

            );
        }
        if (!empty($data['samaccountname'])) {
            $usernameField = TextField::create(
                'Username',
                _t(__CLASS__ . '.USERNAME', 'Username'),
                $data['samaccountname'],
                null,
                $this
            );
        }

        if ($emailField) {
            $emailFieldReadonly = $emailField->performDisabledTransformation();
            $this->Fields()->unshift($emailFieldReadonly);
        }
        if ($usernameField) {
            $usernameFieldReadonly = $usernameField->performDisabledTransformation();
            $this->Fields()->unshift($usernameFieldReadonly);
        }
    }
}