dnadesign / silverstripe-elemental

code/extensions/ElementDuplicationExtension.php

<?php

/**
 * @package elemental
 */
class ElementDuplicationExtension extends Extension

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

{

    /**
     * Duplicate items
     *
     */
    public function onAfterDuplicate($original, $doWrite=true)

The parameter $doWrite is not used and could be removed.

    {
        $thisClass = $this->owner->ClassName;
        $duplicateRelations = Config::inst()->get($thisClass, 'duplicate_relations');

        if ($duplicateRelations && !empty($duplicateRelations)) {
            foreach ($duplicateRelations as $relation) {
                $items = $original->$relation();
                foreach ($items as $item) {
                    $duplicateItem = $item->duplicate(false);
                    $duplicateItem->{$thisClass.'ID'} = $this->owner->ID;
                    $duplicateItem->write();
                }
            }
        }
    }

    public function onBeforeDuplicate($original, $doWrite=true)

The parameter $original is not used and could be removed.

The parameter $doWrite is not used and could be removed.

    {
        $thisClass = $this->owner->ClassName;
        $clearRelations = Config::inst()->get($thisClass, 'duplicate_clear_relations');

        if ($clearRelations && !empty($clearRelations)) {
            foreach ($clearRelations as $clearRelation) {
                $clearRelation = $clearRelation . 'ID';
                $this->owner->$clearRelation = 0;
            }
        }
    }
}