UpdateSecondHandProduct

Complexity

Total Complexity

11

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	MyPermissionCheck()	0	6	1
A	init()	0	7	2
A	unpublish()	0	13	3
B	archive()	0	20	5

<?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.

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

The method init() contains an exit expression.

        }
    }

    public function unpublish($request)
    {
        $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);
    }

    public function archive($request)
    {
        $archived = false;
        $otherID = $request->param("OtherID");
        if (isset($otherID)) {
            $archived = null;
            $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');
            } elseif (! is_null($secondHandProduct)) {
                $archived = $secondHandProduct->delete();

Are you sure the assignment to $archived is correct as $secondHandProduct->delete() (which targets DataObject::delete()) seems to always return null.

            }
        }
        return json_encode($archived);
    }

    /**
     * @return Boolean

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

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