UserManagement\Page\UserRegistrationPageController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

69.23%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	8	4
A	SignUpForm()	0	8	3

<?php
namespace UserManagement\Page;

use PageController;

The type PageController was not found. Maybe you did not declare it correctly or list all dependencies?

use UserManagement\Forms\SignUpForm;
use SilverStripe\SiteConfig\SiteConfig;
use SilverStripe\Security\Security;

/**
 * Class UserRegistrationPageController
 *
 * @package user-management
 */
class UserRegistrationPageController extends PageController
{
    /**
     * @var string
     */
    private static $url_segment = 'user-registration';

The private property $url_segment is not used, and could be removed.

    /**
     * @var array
     */
    private static $allowed_actions = array('SignUpForm');

The private property $allowed_actions is not used, and could be removed.

    public function init()
    {
        parent::init();
        $member = Security::getCurrentUser();
        if ($member && $member->exists()) {
            $config = SiteConfig::current_site_config();
            if ($config->LoginCallBackUrl()->URLSegment) {
                return $this->redirect($config->LoginCallBackUrl()->URLSegment);
            }
        }
    }

    /**
     * @method string enableSpamProtection()
     * @return object
     */
    public function SignUpForm()
    {
        $form = new SignUpForm($this, 'SignUpForm');
        $config = SiteConfig::current_site_config();
        if ($config->EnableSpamProtection && class_exists('UndefinedOffset\NoCaptcha\Forms\NocaptchaField')) {
            $form->enableSpamProtection();

The method enableSpamProtection() does not exist on UserManagement\Forms\SignUpForm. Since you implemented __call, consider adding a @method annotation.

        }
        return $form;
    }
}