UserManagement\Page\UserRegistrationPageController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?php
namespace UserManagement\Page;

use PageController;

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

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

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

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

    private static $allowed_actions = array('SignUpForm');

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

    public function init()
    {
        parent::init();
        $member = Member::currentUser();

The function SilverStripe\Security\Member::currentUser() has been deprecated: 5.0.0 use Security::getCurrentUser()

        if ($member && $member->exists()) {
            $config = SiteConfig::current_site_config();
            if ($config->LoginCallBackUrl()->URLSegment) {
                return $this->redirect($config->LoginCallBackUrl()->URLSegment);
            }
        }
    }

    /**
     * @return object
     */
    public function SignUpForm()
    {
        $form = new SignUpForm($this, 'SignUpForm');
        $config = SiteConfig::current_site_config();
        if ($config->EnableSpamProtection) {
            $form->enableSpamProtection();

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

        }
        return $form;
    }
}