MemberExtension::updateCMSFields() - Code Metrics - Inspection of "Fix .env issues encountered earlier. This is only..." - Firesphere/silverstripe-graphql-jwt - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 023bd4...0eb96d )

by Simon

created 2017-08-13 04:06 UTC

MemberExtension::updateCMSFields() A

↳ Parent: MemberExtension

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	13
rs	9.4285
c	1
b	0
f	0
cc	2
eloc	7
nc	2
nop	1

<?php

namespace Firesphere\GraphQLJWT;

use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\FieldList;
use SilverStripe\ORM\DataExtension;

/**
 * Class MemberExtension
 * Add a unique token to the Member for extra validation
 */
class MemberExtension extends DataExtension
{
    private static $db = [

        'JWTUniqueID' => 'Varchar(255)',
    ];

    private static $indexes = [

        'JWTUniqueID' => 'unique'
    ];

    public function updateCMSFields(FieldList $fields)
    {
        parent::updateCMSFields($fields);
        $fields->removeByName(['JWTUniqueID']);
        if ($this->owner->JWTUniqueID) {
            $fields->addFieldsToTab(
                'Root.Main',
                [
                    CheckboxField::create('reset', 'Reset the Token ID to disable this user\'s remote login')
                ]
            );
        }
    }

    public function onBeforeWrite()
    {
        parent::onBeforeWrite();
        if ($this->owner->reset) {
            $this->owner->JWTUniqueID = null;
        }
    }
}


1			<?php
2
3			namespace Firesphere\GraphQLJWT;
4
5			use SilverStripe\Forms\CheckboxField;
6			use SilverStripe\Forms\FieldList;
7			use SilverStripe\ORM\DataExtension;
8
9			/**
10			* Class MemberExtension
11			* Add a unique token to the Member for extra validation
12			*/
13			class MemberExtension extends DataExtension
14			{
15			private static $db = [
			0 ignored issues – show Unused Code introduced 2017-08-12 01:00 UTC by Report Bug Copy Issue Report The property `$db` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
16			'JWTUniqueID' => 'Varchar(255)',
17			];
18
19			private static $indexes = [
			0 ignored issues – show Unused Code introduced 2017-08-12 01:00 UTC by Report Bug Copy Issue Report The property `$indexes` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
20			'JWTUniqueID' => 'unique'
21			];
22
23			public function updateCMSFields(FieldList $fields)
24			{
25			parent::updateCMSFields($fields);
26			$fields->removeByName(['JWTUniqueID']);
27			if ($this->owner->JWTUniqueID) {
28			$fields->addFieldsToTab(
29			'Root.Main',
30			[
31			CheckboxField::create('reset', 'Reset the Token ID to disable this user\'s remote login')
32			]
33			);
34			}
35			}
36
37			public function onBeforeWrite()
38			{
39			parent::onBeforeWrite();
40			if ($this->owner->reset) {
41			$this->owner->JWTUniqueID = null;
42			}
43			}
44			}
45

Firesphere / silverstripe-graphql-jwt

Push — master ( 023bd4...0eb96d )

MemberExtension::updateCMSFields() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like