SAMLMemberExtension

Complexity

Total Complexity

1

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	updateCMSFields()	0	5	1

<?php
/**
 * Class SAMLMemberExtension
 *
 * Adds mappings from IdP claim rules to SilverStripe {@link Member} fields.
 */
class SAMLMemberExtension extends DataExtension

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

{
    /**
     * @var array
     */
    private static $db = [

The property $db is not used and could be removed.

        // Pointer to the session object held by the IdP
        'SAMLSessionIndex' => 'Varchar(255)',
        // Unique user identifier, same field is used by LDAPMemberExtension
        'GUID' => 'Varchar(50)',
    ];

    /**
     * These are used by {@link SAMLController} to map specific IdP claim rules
     * to {@link Member} fields. Availability of these claim rules are defined
     * on the IdP.
     *
     * @var array
     * @config
     */
    private static $claims_field_mappings = [

The property $claims_field_mappings is not used and could be removed.

        'http://schemas.xmlsoap.org/ws/2005/05/identity/claims/givenname' => 'FirstName',
        'http://schemas.xmlsoap.org/ws/2005/05/identity/claims/surname' => 'Surname',
        'http://schemas.xmlsoap.org/ws/2005/05/identity/claims/emailaddress' => 'Email'
    ];

    /**
     * @param FieldList $fields
     */
    public function updateCMSFields(FieldList $fields)
    {
        $fields->replaceField('GUID', new ReadonlyField('GUID'));
        $fields->removeFieldFromTab('Root', 'SAMLSessionIndex');
    }
}