SecondHandProductGroup::onBeforeWrite() - Code Metrics - sunnysideup/silverstripe-ecommerce_second_hand_product - Measure and Improve Code Quality continuously with Scrutinizer

SecondHandProductGroup::onBeforeWrite() B
last analyzed 2018-06-20 01:12 UTC

↳ Parent: SecondHandProductGroup

Complexity

Conditions	6
Paths	6

Size

Total Lines	20
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	20
rs	8.8571
c	0
b	0
f	0
cc	6
eloc	13
nc	6
nop	0

<?php


class SecondHandProductGroup extends ProductGroup
namespace YourVendor;

class YourClass { }
{
    private static $db = array(

        'RootParent' => 'Boolean'
    );

    private static $allowed_children = array(

        'SecondHandProductGroup',
        'SecondHandProduct'
    );

    private static $icon = 'ecommerce_second_hand_product/images/treeicons/SecondHandProductGroup';


    /**
     * Standard SS variable.
     */
    private static $singular_name = 'Second Hand Product Holder';

    public function i18n_singular_name()
    {
        return self::$singular_name;
    }

    /**
     * Standard SS variable.
     */
    private static $plural_name = 'Second Hand Product Holders';

    public function i18n_plural_name()
    {
        return self::$plural_name;
    }

    /**
     * Standard SS variable.
     *
     * @var string
     */
    private static $description = 'A product category page specifically for second had products';


    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->addFieldToTab(
            'Root.SecondHand',
            CheckboxField::create(
                'RootParent',
                _t('SecondHandProductGroup.LANDING_PAGE', 'Landing Page')
            )
        );
        return $fields;
    }

    /**
     * Event handler called before writing to the database.
     */
    public function onBeforeWrite()
    {
        parent::onBeforeWrite();
        if ($this->ParentID) {
            $parent = SiteTree::get()->byID($this->ParentID);
            if ($parent) {
                if ($parent instanceof SecondHandProductGroup) {
                    $this->RootParent = false;
                } else {
                    if (! $this->hasOtherSecondHandProductGroupsOnThisLevel()) {
                        $this->RootParent = true;
                    }
                }
            }
        } else {
            if (! $this->hasOtherSecondHandProductGroupsOnThisLevel()) {
                $this->RootParent = true;
            }
        }
    }

    /**
     * Level is within SiteTree hierarchy
     * @return boolean
     */
    protected function hasOtherSecondHandProductGroupsOnThisLevel()
    {
        if (!$this->ParentID) {
            $this->ParentID = 0;
        }
        return SecondHandProductGroup::get()
            ->filter(array('ParentID' => $this->ParentID))
            ->exclude(array('ID' => $this->ID))
            ->count() > 0 ? true : false;
    }

    /**
     * @return SecondHandProductGroup
     */
    public function BestRootParentPage()
    {
        $obj = DataObject::get_one(
            'SecondHandProductGroup',
            array('RootParent' => 1)
        );
        if ($obj) {
            return $obj;
        } else {
            return SecondHandProductGroup::get()->first();
        }
    }

    /**
     * Returns the class we are working with.
     *
     * @return string
     */
    protected function getBuyableClassName()
    {
        return 'SecondHandProduct';
    }
}

class SecondHandProductGroup_Controller extends ProductGroup_Controller
namespace YourVendor;

class YourClass { }
{
    private static $allowed_actions = array(

        'SearchSecondHandProducts',
        'search'
    );

    public function init()
    {
        Config::inst()->update(
            'ProductGroup',
            'base_buyable_class',
            'SecondHandProduct'
        );
        parent::init();
        $this->showFullList = true;
    }

    public function SearchSecondHandProducts()
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        $fields = new FieldList(
            new TextField('searchterm', 'Keyword', isset($_GET['searchterm']) ? $_GET['searchterm'] : '')
        );
        $actions = new FieldList(
            new FormAction('doSearchSecondHandProducts', 'Search')
        );
        $validator = new RequiredFields('searchterm');
        $form = Form::create($this, 'SearchSecondHandProducts', $fields, $actions, $validator);
        $form->setFormMethod('GET');
        $form->disableSecurityToken();

        return $form;
    }

    public function doSearchSecondHandProducts($data, $form)

    {
        $page = SecondHandProductGroup::get()->first();
        if ($page) {
            return $this->redirect($this->link('search').'?searchterm='.$data['searchterm']);
        }
    }

    public function search($request)
    {
        $term = Convert::raw2sql($request->param('searchterm'));
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        //uncompleted
    }

    public function HasSearchFilterAndSort()
    {
        return true;
    }
}


1			<?php
2
3
4			class SecondHandProductGroup extends ProductGroup
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report 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...
5			{
6			private static $db = array(
			0 ignored issues – show Comprehensibility introduced 2017-05-17 01:55 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-05-17 01:55 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...
7			'RootParent' => 'Boolean'
8			);
9
10			private static $allowed_children = array(
			0 ignored issues – show Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The property `$allowed_children` 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...
11			'SecondHandProductGroup',
12			'SecondHandProduct'
13			);
14
15			private static $icon = 'ecommerce_second_hand_product/images/treeicons/SecondHandProductGroup';
			0 ignored issues – show Comprehensibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The property `$icon` 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
17			/**
18			* Standard SS variable.
19			*/
20			private static $singular_name = 'Second Hand Product Holder';
			0 ignored issues – show Comprehensibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
21			public function i18n_singular_name()
22			{
23			return self::$singular_name;
24			}
25
26			/**
27			* Standard SS variable.
28			*/
29			private static $plural_name = 'Second Hand Product Holders';
			0 ignored issues – show Comprehensibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
30			public function i18n_plural_name()
31			{
32			return self::$plural_name;
33			}
34
35			/**
36			* Standard SS variable.
37			*
38			* @var string
39			*/
40			private static $description = 'A product category page specifically for second had products';
			0 ignored issues – show Comprehensibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The property `$description` 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...
41
42			public function getCMSFields()
43			{
44			$fields = parent::getCMSFields();
45			$fields->addFieldToTab(
46			'Root.SecondHand',
47			CheckboxField::create(
48			'RootParent',
49			_t('SecondHandProductGroup.LANDING_PAGE', 'Landing Page')
50			)
51			);
52			return $fields;
53			}
54
55			/**
56			* Event handler called before writing to the database.
57			*/
58			public function onBeforeWrite()
59			{
60			parent::onBeforeWrite();
61			if ($this->ParentID) {
62			$parent = SiteTree::get()->byID($this->ParentID);
63			if ($parent) {
64			if ($parent instanceof SecondHandProductGroup) {
65			$this->RootParent = false;
66			} else {
67			if (! $this->hasOtherSecondHandProductGroupsOnThisLevel()) {
68			$this->RootParent = true;
69			}
70			}
71			}
72			} else {
73			if (! $this->hasOtherSecondHandProductGroupsOnThisLevel()) {
74			$this->RootParent = true;
75			}
76			}
77			}
78
79			/**
80			* Level is within SiteTree hierarchy
81			* @return boolean
82			*/
83			protected function hasOtherSecondHandProductGroupsOnThisLevel()
84			{
85			if (!$this->ParentID) {
86			$this->ParentID = 0;
87			}
88			return SecondHandProductGroup::get()
89			->filter(array('ParentID' => $this->ParentID))
90			->exclude(array('ID' => $this->ID))
91			->count() > 0 ? true : false;
92			}
93
94			/**
95			* @return SecondHandProductGroup
96			*/
97			public function BestRootParentPage()
98			{
99			$obj = DataObject::get_one(
100			'SecondHandProductGroup',
101			array('RootParent' => 1)
102			);
103			if ($obj) {
104			return $obj;
105			} else {
106			return SecondHandProductGroup::get()->first();
107			}
108			}
109
110			/**
111			* Returns the class we are working with.
112			*
113			* @return string
114			*/
115			protected function getBuyableClassName()
116			{
117			return 'SecondHandProduct';
118			}
119			}
120
121			class SecondHandProductGroup_Controller extends ProductGroup_Controller
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report 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...
122			{
123			private static $allowed_actions = array(
			0 ignored issues – show Comprehensibility introduced 2017-05-17 01:55 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The property `$allowed_actions` 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...
124			'SearchSecondHandProducts',
125			'search'
126			);
127
128			public function init()
129			{
130			Config::inst()->update(
131			'ProductGroup',
132			'base_buyable_class',
133			'SecondHandProduct'
134			);
135			parent::init();
136			$this->showFullList = true;
137			}
138
139			public function SearchSecondHandProducts()
			0 ignored issues – show Coding Style introduced 2017-08-24 01:32 UTC by Report Bug Copy Issue Report `SearchSecondHandProducts` uses the super-global variable `$_GET` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
140			{
141			$fields = new FieldList(
142			new TextField('searchterm', 'Keyword', isset($_GET['searchterm']) ? $_GET['searchterm'] : '')
143			);
144			$actions = new FieldList(
145			new FormAction('doSearchSecondHandProducts', 'Search')
146			);
147			$validator = new RequiredFields('searchterm');
148			$form = Form::create($this, 'SearchSecondHandProducts', $fields, $actions, $validator);
149			$form->setFormMethod('GET');
150			$form->disableSecurityToken();
151
152			return $form;
153			}
154
155			public function doSearchSecondHandProducts($data, $form)
			0 ignored issues – show Documentation introduced 2017-08-24 01:32 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
156			{
157			$page = SecondHandProductGroup::get()->first();
158			if ($page) {
159			return $this->redirect($this->link('search').'?searchterm='.$data['searchterm']);
160			}
161			}
162
163			public function search($request)
164			{
165			$term = Convert::raw2sql($request->param('searchterm'));
			0 ignored issues – show Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report `$term` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
166			//uncompleted
167			}
168
169			public function HasSearchFilterAndSort()
170			{
171			return true;
172			}
173			}
174

sunnysideup / silverstripe-ecommerce_second_hand_product

GitHub Access Token became invalid

SecondHandProductGroup::onBeforeWrite() B last analyzed 2018-06-20 01:12 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SecondHandProductGroup::onBeforeWrite() B
last analyzed 2018-06-20 01:12 UTC