Firesphere\HaveIBeenPwnd\Extensions\MemberExtension::updateCMSFields()

Complexity

Conditions	6
Paths	8

Size

Total Lines	41
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function updateCMSFields(FieldList $fields)
    {
        // PwndDisabled always needs to be false
        $this->owner->PwndDisabled = false;

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

        $fields->removeByName(['BreachedSites', 'PasswordIsPwnd']);
        if ($this->owner->BreachedSites || $this->owner->PasswordIsPwnd) {
            $fields->findOrMakeTab('Root.HaveIBeenPwnd', _t(self::class . '.PWNDTAB', 'Have I Been Pwnd?'));
        }
        if ($this->owner->PasswordIsPwnd > 0 || $this->owner->BreachedSites) {
            $text = _t(
                self::class . '.PWNDHelp',
                'If the error says that you "have been Pwnd", it means your password appears in the <a href="https://haveibeenpwned.com/Privacy">Have I Been Pwnd</a> database. ' .
                'Therefore, we can not accept your password, because it is insecure or known to have been breached. ' .
                'Before a password is safely stored in our database, we test if the password has been breached. We do not share your password. ' .
                'We run a safe test against the HaveIBeenPwnd database to. None of your data is shared or stored at HaveIBeenPwnd. ' .
                'For more information, you can read up on "Password safety", and we strongly recommend installing a password manager if you haven\'t already. ' .
                'Several options are LastPass, BitWarden and 1Password. These services are also able to test your passwords against the HaveIBeenPwnd database, ' .
                'to see if your passwords are secure and safe.<br />' .
                'Furthermore, <a href="https://www.troyhunt.com/introducing-306-million-freely-downloadable-pwned-passwords/">Troy Hunt explains why and how this service is important</a>.'
            );

            $help = LiteralField::create('Helptext', '<p>' . $text . '</p>');
            $fields->addFieldToTab('Root.HaveIBeenPwnd', $help);
        }

        if ($this->owner->BreachedSites) {
            $fields->addFieldToTab(
                'Root.HaveIBeenPwnd',
                ReadonlyField::create('BreachedSites', _t(self::class . '.BREACHEDSITES', 'Breached sites'))
            );
        }

        $fields->addFieldToTab('Root.Main', $countField = ReadonlyField::create('PasswordIsPwnd', 'Pwnd Count'));
        $countField->setDescription(_t(
            self::class . '.AMOUNT',
            'Amount of times the password appears in the Have I Been Pwnd database'
        ));

        $fields->addFieldToTab('Root.Main', $tmpDisable = CheckboxField::create('PwndDisabled', _t(self::class . '.TMPDISABLE', 'Disable "Have I Been Pwnd" temporarily')));
        $tmpDisable->setDescription(_t(self::class . '.TMPDISABLEDESCR', 'Allow the password to be a compromised password once (only from the CMS), to reset a users password manually and let the user reset the password on first login.'));
    }