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

EcommerceSecurityMemberDecoration::onBeforeWrite()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 7
Code Lines 4

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
dl 0
loc 7
rs 9.4285
c 0
b 0
f 0
cc 2
eloc 4
nc 2
nop 0
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