DMSUploadField_ItemHandler

Complexity

Total Complexity

2

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	EditForm()	0	19	1
A	getItem()	0	4	1

<?php

class DMSUploadField_ItemHandler extends UploadField_ItemHandler
{
    private static $allowed_actions = array(

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

        'delete',
        'edit',
        'EditForm',
    );

    /**
     * Gets a DMS document by its ID
     *
     * @return DMSDocument

Should the return type not be DMSDocument|null?

     */
    public function getItem()
    {
        return DMSDocument::get()->byId($this->itemID);
    }

    /**
     * @return Form
     */
    public function EditForm()
    {
        $file = $this->getItem();

        // Get form components
        $fields = $this->parent->getDMSFileEditFields($file);

The method getDMSFileEditFields() does not exist on UploadField. Did you maybe mean getFileEditFields()?

        $actions = $this->parent->getDMSFileEditActions($file);

The method getDMSFileEditActions() does not exist on UploadField. Did you maybe mean getFileEditActions()?

        $validator = $this->parent->getDMSFileEditValidator($file);

The method getDMSFileEditValidator() does not exist on UploadField. Did you maybe mean getFileEditValidator()?

        $form = new Form(
            $this,

$this is of type this<DMSUploadField_ItemHandler>, but the function expects a object<Controller>.

            __FUNCTION__,
            $fields,
            $actions,
            $validator
        );
        $form->loadDataFrom($file);

It seems like $file defined by $this->getItem() on line 26 can be null; however, Form::loadDataFrom() does not accept null, maybe add an additional type check?

        return $form;
    }
}