Thinktomorrow\Chief\Templates\PageTemplateApplicator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	7	2
A	shouldApply()	0	3	1
A	applyRelations()	0	18	5
A	shouldApplyRelations()	0	3	2

<?php

declare(strict_types=1);

namespace Thinktomorrow\Chief\Templates;

use Thinktomorrow\Chief\Modules\Module;
use Thinktomorrow\Chief\Pages\Page;
use Thinktomorrow\Chief\Relations\ActsAsParent;
use Webmozart\Assert\Assert;

class PageTemplateApplicator implements TemplateApplicator
{
    /** @var ModuleTemplateApplicator */
    private $moduleTemplateApplicator;

    public function __construct(ModuleTemplateApplicator $moduleTemplateApplicator)
    {
        $this->moduleTemplateApplicator = $moduleTemplateApplicator;
    }

    public function handle($sourceModel, $targetModel): void
    {
        /** @var Page $sourceModel */
        Assert::isInstanceOf($sourceModel, Page::class);

        if ($this->shouldApplyRelations($sourceModel, $targetModel)) {
            $this->applyRelations($sourceModel, $targetModel);
        }
    }

    public function shouldApply($sourceModel, $targetModel): bool
    {
        return ($sourceModel instanceof Page);
    }

    private function shouldApplyRelations($sourceModel, $targetModel): bool
    {
        return ($sourceModel instanceof ActsAsParent && $targetModel instanceof ActsAsParent);
    }

    private function applyRelations(ActsAsParent $sourceModel, ActsAsParent $targetModel): void
    {
        foreach ($sourceModel->children() as $child) {
            $duplicatedChild = null;

            if ($child instanceof Module && $child->isPageSpecific()) {
                $duplicatedChild = $child::create([
                    'slug'           => $targetModel->title ? $targetModel->title . '-' . $child->slug : $child->slug . '-copy',

Accessing title on the interface Thinktomorrow\Chief\Relations\ActsAsParent suggest that you code against a concrete implementation. How about adding an instanceof check?

                    'owner_id'        => $targetModel->id,

Accessing id on the interface Thinktomorrow\Chief\Relations\ActsAsParent suggest that you code against a concrete implementation. How about adding an instanceof check?

                    'owner_type' => $targetModel->getMorphClass()

The method getMorphClass() does not exist on Thinktomorrow\Chief\Relations\ActsAsParent. Since it exists in all sub-types, consider adding an abstract or default implementation to Thinktomorrow\Chief\Relations\ActsAsParent.

                ]);

                $this->moduleTemplateApplicator->handle($child, $duplicatedChild);
            } else {
                $duplicatedChild = $child;
            }

            $targetModel->adoptChild($duplicatedChild, ['sort' => $child->relation->sort]);

The property relation does not exist on Thinktomorrow\Chief\Modules\Module. Did you mean relations?

        }
    }
}