Charcoal\Attachment\Object\Container

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	15.15 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setDependencies()	10	10	3
A	attachments()	0	15	3
A	preDelete()	0	9	1

<?php

namespace Charcoal\Attachment\Object;

// From Pimple
use Pimple\Container as ServiceContainer;

// From 'charcoal-config'
use Charcoal\Config\ConfigurableInterface;

// From 'charcoal-attachment'
use Charcoal\Attachment\Interfaces\AttachmentAwareInterface;
use Charcoal\Attachment\Interfaces\AttachmentContainerInterface;
use Charcoal\Attachment\Traits\AttachmentAwareTrait;
use Charcoal\Attachment\Traits\AttachmentContainerTrait;
use Charcoal\Attachment\Traits\ConfigurableAttachmentsTrait;

/**
 * Gallery Attachment Type
 *
 * This type allows for nesting of additional attachment types.
 */
class Container extends Attachment implements

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: hasProperty, p, properties, property

    AttachmentAwareInterface,
    AttachmentContainerInterface,
    ConfigurableInterface
{
    use AttachmentAwareTrait;
    use AttachmentContainerTrait;
    use ConfigurableAttachmentsTrait;

    /**
     * Inject dependencies from a DI Container.
     *
     * @param  ServiceContainer $container A dependencies container instance.
     * @return void
     */
    protected function setDependencies(ServiceContainer $container)

This method seems to be duplicated in your project.

    {
        parent::setDependencies($container);

        if (isset($container['attachments/config'])) {
            $this->setConfig($container['attachments/config']);
        } elseif (isset($container['config']['attachments'])) {
            $this->setConfig($container['config']['attachments']);
        }
    }

    /**
     * Retrieve the objects associated to the current object.
     *
     * @param  mixed ...$args Filter the attachments;
     *     options accepted by {@see AttachmentAwareTrait::attachments()}.
     * @return Collection|Attachment[]
     */
    public function attachments(...$args)
    {
        $attachables = $this->attachableObjects();
        $attachments = call_user_func_array([ $this, 'getAttachments' ], $args);

        foreach ($attachments as $attachment) {
            if (isset($attachables[$attachment->objType()])) {
                $attachment->attachmentType = $attachables[$attachment->objType()];
            } else {
                $attachment->attachmentType = [];
            }
        }

        return $attachments;
    }

    /**
     * Event called before _deleting_ the attachment.
     *
     * @return boolean
     * @see    Charcoal\Attachment\Traits\AttachmentAwareTrait::removeJoins
     * @see    Charcoal\Source\StorableTrait::preDelete() For the "create" Event.
     */
    public function preDelete()
    {
        // Delete nested attachments
        array_map(function($attachment) {
            $attachment->delete();
        }, $this->attachments()->values());

        return parent::preDelete();
    }
}