UpdateSecondHandProduct

Complexity

Total Complexity

10

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	7	2
A	unpublish()	0	13	3
A	archive()	0	19	4
A	MyPermissionCheck()	0	6	1

<?php


class UpdateSecondHandProduct extends Controller

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    private static $allowed_actions = array(

Consider using a different property name as you override a private property of the parent class.

The property $allowed_actions is not used and could be removed.

        'unpublish' => '->MyPermissionCheck',
        'archive' => '->MyPermissionCheck'
    );


    /**
     * make the page less easy to access
     * (but still accessible)
     * - code => ip address
     * @var string
     */
    private static $secret_codes = array();

The property $secret_codes is not used and could be removed.

    function init()

It is generally recommended to explicitly declare the visibility for methods.

    {
        parent::init();
        if(! $this->MyPermissionCheck()) {
            die('you do not have access');

The method init() contains an exit expression.

        }
    }

    function unpublish($request)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $unpublished = false;
        $otherID = $request->param("OtherID");
        if(isset($otherID)) {
            $internalItemID = Convert::raw2sql($otherID);
            $secondHandProduct = DataObject::get_one('SecondHandProduct', ['InternalItemID' => $internalItemID]);
            if($secondHandProduct){
                $unpublished = $secondHandProduct->deleteFromStage('Live');
            }
        }
        return json_encode($unpublished);
    }

    function archive($request)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $archived = false;
        $otherID = $request->param("OtherID");
        if(isset($otherID)) {
            $internalItemID = Convert::raw2sql($otherID);
            $secondHandProduct = DataObject::get_one('SecondHandProduct', ['InternalItemID' => $internalItemID]);
            if(!$secondHandProduct){
                $secondHandProduct = Versioned::get_one_by_stage('SecondHandProduct', 'Stage', ['InternalItemID' => $internalItemID]);

array('InternalItemID' => $internalItemID) is of type array<string,array|string,{"InternalItemID":"array|string"}>, but the function expects a string.

            }
            if (is_a($secondHandProduct, Object::getCustomClass('SiteTree'))) {
                $archived = $secondHandProduct->deleteFromStage('Live');

$archived is not used, you could remove the assignment.

                $archived = $secondHandProduct->deleteFromStage('Stage');
            } else {
                $archived = $secondHandProduct->delete();
            }
        }
        return json_encode($archived);
    }

    /**
     * @return Boolean

Should the return type not be boolean|string|null?

     */
    function MyPermissionCheck()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $codesWithIPs = $this->Config()->get('secret_codes');
        $code = $this->request->param('ID');
        return ControllerPermissionChecker::permissionCheck($codesWithIPs, $code);
    }
}