SecondHandProductAdmin::restore() - Code Metrics - Inspection of "MINOR: adding restore button to archived products" - sunnysideup/silverstripe-ecommerce_second_hand_product - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 296583...4f1ba4 )

unknown

created 2017-03-29 23:20 UTC

SecondHandProductAdmin::restore() B

↳ Parent: SecondHandProductAdmin

Complexity

Conditions	4
Paths	4

Size

Total Lines	22
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	22
rs	8.9197
c	0
b	0
f	0
cc	4
eloc	17
nc	4
nop	1

<?php

/**
 * @description: for the management of Product and Product Groups only
 *
 *
 * @authors: Nicolaas [at] Sunny Side Up .co.nz
 * @package: ecommerce
 * @sub-package: cms
 **/

class SecondHandProductAdmin extends ModelAdminEcommerceBaseClass
namespace YourVendor;

class YourClass { }
{
    private static $menu_priority = 3.2;


    private static $url_segment = 'secondhandproducts';


    private static $menu_title = 'Second Hand';


    private static $managed_models = array(

        'SecondHandProduct',
        'SecondHandArchive'
    );

    private static $allowed_actions = array(

        "editinsitetree",
        "ItemEditForm",
        "restore" => true,
    );

    /**
     * standard SS variable
     * @var String
     */
    private static $menu_icon = "ecommerce/images/icons/product-file.gif";



    public function getEditForm($id = null, $fields = null)

    {
        foreach (GoogleAddressField::js_requirements() as $jsFile) {
            Requirements::javascript($jsFile);
        }
        $form = parent::getEditForm();
        if (singleton($this->modelClass) instanceof SiteTree) {
            if ($gridField = $form->Fields()->dataFieldByName($this->sanitiseClassName($this->modelClass))) {
                if ($gridField instanceof GridField) {
                    $gridField->setConfig(GridFieldEditOriginalPageConfigSecondHandPage::create());
                }
            }
        }
        return $form;
    }


    public function doCancel($data, $form)

    {
        return $this->redirect(singleton('CMSMain')->Link());
    }

    function restore($request){
// 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);
    }
}
        if(isset($_GET['productid'])){
            $id = intval($_GET['productid']);
            if($id) {
                $restoredPage = Versioned::get_latest_version("SiteTree", $id);
                if(!$restoredPage) 	return new SS_HTTPResponse("SiteTree #$id not found", 400);
                $restoredPage = $restoredPage->doRestoreToStage();
                $restoredPage->doPublish();
                $this->getResponse()->addHeader(
                    'X-Status',
                    rawurlencode(_t(
                        'CMSMain.RESTORED',
                        "Restored '{title}' successfully",
                        array('title' => $restoredPage->Title)
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                    ))
                );
                $cmsEditLink = '/admin/secondhandproducts/SecondHandProduct/EditForm/field/SecondHandProduct/item/'.$id.'/edit';
                return Controller::curr()->redirect($cmsEditLink);
            }
        }
        return new SS_HTTPResponse("ERROR!", 400);
    }
}


1			<?php
2
3			/**
4			* @description: for the management of Product and Product Groups only
5			*
6			*
7			* @authors: Nicolaas [at] Sunny Side Up .co.nz
8			* @package: ecommerce
9			* @sub-package: cms
10			**/
11
12			class SecondHandProductAdmin extends ModelAdminEcommerceBaseClass
			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...
13			{
14			private static $menu_priority = 3.2;
			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 `$menu_priority` 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...
15
16			private static $url_segment = 'secondhandproducts';
			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 `$url_segment` 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...
17
18			private static $menu_title = 'Second Hand';
			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 `$menu_title` 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...
19
20			private static $managed_models = array(
			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 `$managed_models` 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...
21			'SecondHandProduct',
22			'SecondHandArchive'
23			);
24
25			private static $allowed_actions = array(
			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 `$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...
26			"editinsitetree",
27			"ItemEditForm",
28			"restore" => true,
29			);
30
31			/**
32			* standard SS variable
33			* @var String
34			*/
35			private static $menu_icon = "ecommerce/images/icons/product-file.gif";
			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 `$menu_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...
36
37
38			public function getEditForm($id = null, $fields = null)
			0 ignored issues – show Documentation introduced 2017-01-03 01:12 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...
39			{
40			foreach (GoogleAddressField::js_requirements() as $jsFile) {
41			Requirements::javascript($jsFile);
42			}
43			$form = parent::getEditForm();
44			if (singleton($this->modelClass) instanceof SiteTree) {
45			if ($gridField = $form->Fields()->dataFieldByName($this->sanitiseClassName($this->modelClass))) {
46			if ($gridField instanceof GridField) {
47			$gridField->setConfig(GridFieldEditOriginalPageConfigSecondHandPage::create());
48			}
49			}
50			}
51			return $form;
52			}
53
54
55			public function doCancel($data, $form)
			0 ignored issues – show Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The parameter `$data` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-01-03 01:12 UTC by Report Bug Copy Issue Report The parameter `$form` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
56			{
57			return $this->redirect(singleton('CMSMain')->Link());
58			}
59
60			function restore($request){
			0 ignored issues – show Unused Code introduced 2017-03-29 23:22 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Best Practice introduced 2017-03-29 23:22 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Coding Style introduced 2017-03-29 23:22 UTC by Report Bug Copy Issue Report `restore` 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...
61			if(isset($_GET['productid'])){
62			$id = intval($_GET['productid']);
63			if($id) {
64			$restoredPage = Versioned::get_latest_version("SiteTree", $id);
65			if(!$restoredPage) return new SS_HTTPResponse("SiteTree #$id not found", 400);
66			$restoredPage = $restoredPage->doRestoreToStage();
67			$restoredPage->doPublish();
68			$this->getResponse()->addHeader(
69			'X-Status',
70			rawurlencode(_t(
71			'CMSMain.RESTORED',
72			"Restored '{title}' successfully",
73			array('title' => $restoredPage->Title)
			0 ignored issues – show Documentation introduced 2017-03-29 23:22 UTC by Report Bug Copy Issue Report `array('title' => $restoredPage->Title)` is of type `array<string,?,{"title":"?"}>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
74			))
75			);
76			$cmsEditLink = '/admin/secondhandproducts/SecondHandProduct/EditForm/field/SecondHandProduct/item/'.$id.'/edit';
77			return Controller::curr()->redirect($cmsEditLink);
78			}
79			}
80			return new SS_HTTPResponse("ERROR!", 400);
81			}
82			}
83

sunnysideup / silverstripe-ecommerce_second_hand_product

GitHub Access Token became invalid

Push — master ( 296583...4f1ba4 )

SecondHandProductAdmin::restore() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like