CreatorLastEditorExtension

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

2 Methods

Rating	Name	Duplication	Size	Complexity
A	onBeforeWrite()	0	7	2
A	updateCMSFields()	0	9	2

<?php

class CreatorLastEditorExtension extends DataExtension

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

{
    private static $has_one = array(

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

        'Creator' => 'Member',
        'LastEditor' => 'Member',
    );

    /*
    1) Save the creator as the current editing member if there is not creator already assigned
    2) Save the last editor (e.g. an admin) as the person who last edited this document
    */
    public function onBeforeWrite()
    {
        if ($this->owner->CreatorID == 0) {
            $this->owner->CreatorID = Member::currentUserID();
        }
        $this->owner->LastEditorID = Member::currentUserID();
    }

    /*
    Allow the admin to override the creator and last editor
    */
    public function updateCMSFields(FieldList $fields)
    {
        if (Permission::check('ADMIN')) {
            $memberField = new DropdownField('CreatorID', 'Creator', Member::get()->map('ID', 'Username', '-- Please select --'));

The call to DataList::map() has too many arguments starting with '-- Please select --'.

            $fields->addFieldToTab('Root.Creator', $memberField);
        }

        return $fields;
    }
}