Completed
Push — master ( a464af...dfbb4e )
by Nicolaas
01:53
created

EcommerceSecurityMemberDecoration   A

Complexity

Total Complexity 2

Size/Duplication

Total Lines 18
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 1

Importance

Changes 0
Metric Value
wmc 2
lcom 1
cbo 1
dl 0
loc 18
rs 10
c 0
b 0
f 0

1 Method

Rating   Name   Duplication   Size   Complexity  
A onBeforeWrite() 0 7 2
1
<?php
2
3
class EcommerceSecurityMemberDecoration extends DataExtension
0 ignored issues
show
Coding Style Compatibility introduced by
PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

You can fix this by adding a namespace to your class:

namespace YourVendor;

class YourClass { }

When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.

Loading history...
4
{
5
    private static $db = array(
0 ignored issues
show
Unused Code introduced by
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...
6
        'IsWhitelisted' => 'Boolean',
7
        'IsSecurityRisk' => 'Boolean'
8
    );
9
10
    /**
11
     * Event handler called before writing to the database.
12
     */
13
    public function onBeforeWrite()
14
    {
15
        parent::onBeforeWrite();
16
        if($this->owner->IsSecurityRisk) {
17
            $this->owner->IsWhitelisted = false;
18
        }
19
    }
20
}
21