DNADesign\Elemental\TopPage\FluentExtension

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	assignTopPage()	0	5	1
A	clearTopPage()	0	5	1

<?php

namespace DNADesign\Elemental\TopPage;

use DNADesign\Elemental\Models\BaseElement;
use DNADesign\Elemental\Models\ElementalArea;
use Page;

The type Page was not found. Maybe you did not declare it correctly or list all dependencies?

use TractorCow\Fluent\State\FluentState;

The type TractorCow\Fluent\State\FluentState was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Class FluentExtension
 *
 * Use this extension in case you use the Fluent module (https://github.com/tractorcow-farm/silverstripe-fluent)
 * for page localisation
 * this will keep track of the locale the nested data object is stored in
 *
 * @property string $TopPageLocale
 * @property BaseElement|ElementalArea|$this $owner
 * @package DNADesign\Elemental\TopPage
 */
class FluentExtension extends DataExtension
{
    /**
     * @var array
     */
    private static $db = [

The private property $db is not used, and could be removed.

        'TopPageLocale' => 'Varchar',
    ];

    protected function assignTopPage(Page $page): void
    {
        parent::assignTopPage($page);

        $this->owner->TopPageLocale = FluentState::singleton()->getLocale();

The property TopPageLocale does not exist on DNADesign\Elemental\Models\ElementalArea. Since you implemented __set, consider adding a @property annotation.

The property TopPageLocale does not exist on DNADesign\Elemental\Models\BaseElement. Since you implemented __set, consider adding a @property annotation.

    }

    protected function clearTopPage(): void
    {
        parent::clearTopPage();

        $this->owner->TopPageLocale = null;

The property TopPageLocale does not exist on DNADesign\Elemental\Models\BaseElement. Since you implemented __set, consider adding a @property annotation.

The property TopPageLocale does not exist on DNADesign\Elemental\Models\ElementalArea. Since you implemented __set, consider adding a @property annotation.

    }
}