i-lateral / silverstripe-users

code/extensions/Ext_Users_Group.php

<?php

/**
 * Overwrite group object so we can setup some more default groups
 */
class Ext_Users_Group extends DataExtension

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

{
    public function requireDefaultRecords()
    {
        parent::requireDefaultRecords();

        // Add default author group if no other group exists
        $frontend_group = Group::get()->filter("Code", "users-frontend");

        if (!$frontend_group->exists()) {

This code seems to be duplicated across your project.

            $frontend_group = new Group();
            $frontend_group->Code = 'users-frontend';
            $frontend_group->Title = "Frontend Users";
            $frontend_group->Sort = 1;
            $frontend_group->write();
            Permission::grant($frontend_group->ID, 'USERS_MANAGE_ACCOUNT');

            DB::alteration_message('Front end users group created', 'created');
        }

        // Add a verified users group (only used if we turn on
        // verification)
        $verify_group = Group::get()->filter("Code", "users-verified");

        if (!$verify_group->exists()) {

This code seems to be duplicated across your project.

            $verify_group = new Group();
            $verify_group->Code = 'users-verified';
            $verify_group->Title = "Verified Users";
            $verify_group->Sort = 1;
            $verify_group->write();
            Permission::grant($verify_group->ID, 'USERS_VERIFIED');

            DB::alteration_message('Verified users group created', 'created');
        }
    }
}