Thinktomorrow\Chief\Pages\PageManager

Complexity

Total Complexity

32

Size/Duplication

Total Lines	238
Duplicated Lines	0 %

Test Coverage

Coverage

96.72%

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	delete()	0	7	2
A	filters()	0	4	1
A	details()	0	11	2
A	enforceUniqueSlug()	0	8	1
A	saveFields()	0	8	2
A	pageBuilderField()	0	7	2
A	storeRequest()	0	14	3
A	can()	0	9	2
A	authorize()	0	14	5
A	updateRequest()	0	19	3
A	addDefaultShortDescription()	0	7	2
A	fields()	0	36	2
A	afterUpdate()	0	5	1
A	fieldArrangement()	0	13	2
A	afterStore()	0	5	1

<?php

namespace Thinktomorrow\Chief\Pages;

use Illuminate\Http\Request;
use Thinktomorrow\Chief\Audit\Audit;
use Thinktomorrow\Chief\Fields\Fields;
use Thinktomorrow\Chief\Filters\Filters;
use Thinktomorrow\Chief\Fields\FieldsTab;
use Thinktomorrow\Chief\Management\Manager;
use Thinktomorrow\Chief\Fields\Types\TextField;
use Thinktomorrow\Chief\Fields\FieldArrangement;
use Thinktomorrow\Chief\Fields\Types\InputField;
use Thinktomorrow\Chief\Fields\Types\MediaField;
use Thinktomorrow\Chief\Management\Registration;
use Thinktomorrow\Chief\Fields\RemainingFieldsTab;
use Thinktomorrow\Chief\Management\AbstractManager;
use Thinktomorrow\Chief\Management\Details\Details;
use Thinktomorrow\Chief\Pages\Application\DeletePage;
use Thinktomorrow\Chief\Concerns\Sluggable\UniqueSlug;
use Thinktomorrow\Chief\Management\Exceptions\DeleteAborted;
use Thinktomorrow\Chief\Management\Assistants\ArchiveAssistant;
use Thinktomorrow\Chief\Management\Assistants\PublishAssistant;
use Thinktomorrow\Chief\Management\Exceptions\NotAllowedManagerRoute;

class PageManager extends AbstractManager implements Manager
{
    /** @var \Thinktomorrow\Chief\Concerns\Sluggable\UniqueSlug */
    private $uniqueSlug;

    /** @var PageBuilderField */

The type Thinktomorrow\Chief\Pages\PageBuilderField was not found. Maybe you did not declare it correctly or list all dependencies?

    private $pageBuilderField;

    protected $assistants = [
        'archive' => ArchiveAssistant::class,
        'publish' => PublishAssistant::class,
    ];

    public function __construct(Registration $registration)
    {
        parent::__construct($registration);

        $this->uniqueSlug = UniqueSlug::make(new PageTranslation)->slugResolver(function ($value) {
            return str_slug_slashed($value);
        });
    }

    public function can($verb): bool
    {
        try {
            $this->authorize($verb);
        } catch (NotAllowedManagerRoute $e) {
            return false;
        }

        return parent::can($verb);
    }

    /**
     * @param $verb
     * @throws NotAllowedManagerRoute
     */
    private function authorize($verb)
    {
        $permission = 'update-page';

        if (in_array($verb, ['index','show'])) {
            $permission = 'view-page';
        } elseif (in_array($verb, ['create','store'])) {
            $permission = 'create-page';
        } elseif (in_array($verb, ['delete'])) {
            $permission = 'delete-page';
        }

        if (! auth()->guard('chief')->user()->hasPermissionTo($permission)) {

The method hasPermissionTo() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of Illuminate\Contracts\Auth\Authenticatable such as Illuminate\Foundation\Auth\User.

            throw NotAllowedManagerRoute::notAllowedPermission($permission, $this);
        }
    }

    /**
     * The set of fields that should be manageable for a certain model.
     *
     * Additionally, you should:
     * 1. Make sure to setup the proper migrations and
     * 2. For a translatable field you should add this field to the $translatedAttributes property of the model as well.
     *
     * @return Fields
     */
    public function fields(): Fields
    {
        return new Fields([
            $this->pageBuilderField(),
            InputField::make('title')->translatable($this->model->availableLocales())

The method availableLocales() does not exist on Illuminate\Contracts\Foundation\Application.

                                     ->validation('required-fallback-locale|max:200', [], [
                                         'trans.'.config('app.fallback_locale', 'nl').'.title' => 'title',
                                     ])
                                     ->label('De titel van je '.$this->model->labelSingular ?? 'pagina')

The method label() does not exist on Thinktomorrow\Chief\Fields\Types\InputField. Since you implemented __call, consider adding a @method annotation.

                                     ->description('Dit is de titel die zal worden getoond in de overzichten en modules.<br> Deze zal gebruikt worden als interne titel en slug van de nieuwe pagina.'),

The method description() does not exist on Thinktomorrow\Chief\Fields\Types\InputField. Since you implemented __call, consider adding a @method annotation.

This line exceeds maximum limit of 120 characters; contains 200 characters

            InputField::make('slug')
                ->translatable($this->model->availableLocales())
                ->validation($this->model->id
                    ? 'required-fallback-locale|unique:page_translations,slug,' . $this->model->id . ',page_id'
                    : 'required-fallback-locale|unique:page_translations,slug', [], [
                    'trans.'.config('app.fallback_locale', 'nl').'.slug' => 'slug'
                ])
                ->label('Link')
                ->description('De unieke url verwijzing naar deze pagina.')
                ->prepend(collect($this->model->availableLocales())->mapWithKeys(function ($locale) {

The method prepend() does not exist on Thinktomorrow\Chief\Fields\Types\InputField. Since you implemented __call, consider adding a @method annotation.

                    return [$locale => url($this->model->baseUrlSegment($locale)).'/'];

Are you sure url($this->model->baseUrlSegment($locale)) of type Illuminate\Contracts\Routing\UrlGenerator|string can be used in concatenation?

The method baseUrlSegment() does not exist on Illuminate\Contracts\Foundation\Application.

                })->all()),
            InputField::make('seo_title')
                ->translatable($this->model->availableLocales())
                ->label('Zoekmachine titel'),
            TextField::make('seo_description')
                ->translatable($this->model->availableLocales())
                ->label('Zoekmachine omschrijving')

The method label() does not exist on Thinktomorrow\Chief\Fields\Types\TextField. Since you implemented __call, consider adding a @method annotation.

                ->description('omschrijving van de pagina zoals in search engines (o.a. google) wordt weergegeven.'),

The method description() does not exist on Thinktomorrow\Chief\Fields\Types\TextField. Since you implemented __call, consider adding a @method annotation.

            InputField::make('seo_keywords')
                ->translatable($this->model->availableLocales())
                ->label('Zoekmachine sleutelwoorden')
                ->description('sleutelwoorden van de pagina waarop in search engines (o.a google) gezocht kan worden.'),
            MediaField::make('seo_image')
                ->label('Zoekmachine foto')

The method label() does not exist on Thinktomorrow\Chief\Fields\Types\MediaField. Since you implemented __call, consider adding a @method annotation.

                ->description('foto die bij het delen van deze pagina getoont word. (afmeting: 1200x627px)')

The method description() does not exist on Thinktomorrow\Chief\Fields\Types\MediaField. Since you implemented __call, consider adding a @method annotation.

        ]);
    }

    public static function filters(): Filters
    {
        return new Filters([
            PublishedFilter::class
        ]);
    }

    private function pageBuilderField()
    {
        if ($this->pageBuilderField) {
            return $this->pageBuilderField;
        }

        return $this->pageBuilderField = $this->createPagebuilderField();

It seems like $this->createPagebuilderField() of type Thinktomorrow\Chief\Fields\Types\PagebuilderField is incompatible with the declared type Thinktomorrow\Chief\Pages\PageBuilderField of property $pageBuilderField.

    }

    public function fieldArrangement($key = null): FieldArrangement
    {
        if ($key == 'create') {
            return new FieldArrangement($this->fields()->filterBy(function ($field) {
                return $field->key == 'title';
            }));
        }

        return new FieldArrangement($this->fields(), [
            new FieldsTab('pagina', ['sections']),
            new RemainingFieldsTab('inhoud'),
            new FieldsTab('eigen modules', [], 'chief::back.pages._partials.modules'),
            new FieldsTab('seo', ['seo_title', 'seo_description', 'seo_keywords', 'seo_image']),
        ]);
    }

    public function details(): Details
    {
        // For existing model
        if ($this->model->id) {
            return parent::details()
                ->set('title', $this->model->title)
                ->set('intro', 'laatst aangepast op ' . $this->model->updated_at->format('d/m/Y H:i'))
                ->set('context', '<span class="inline-s">' . $this->assistant('publish')->publicationStatusAsLabel() . '</span>');

The method publicationStatusAsLabel() does not exist on Thinktomorrow\Chief\Management\Assistants\Assistant. It seems like you code against a sub-type of Thinktomorrow\Chief\Management\Assistants\Assistant such as Thinktomorrow\Chief\Management\Assistants\PublishAssistant.

This line exceeds maximum limit of 120 characters; contains 130 characters

        }

        return parent::details();
    }

    public function saveFields(): Manager
    {
        // Store the morph_key upon creation
        if (! $this->model->morph_key) {
            $this->model->morph_key = $this->model->morphKey();

The method morphKey() does not exist on Illuminate\Contracts\Foundation\Application.

        }

        return parent::saveFields();
    }

    public function delete()
    {
        if (request()->get('deleteconfirmation') !== 'DELETE') {
            throw new DeleteAborted();
        }

        app(DeletePage::class)->handle($this->model->id);
    }

    public function storeRequest(Request $request): Request
    {
        $trans = [];
        foreach ($request->get('trans', []) as $locale => $translation) {
            if (is_array_empty($translation)) {
                continue;
            }

            $translation = $this->enforceUniqueSlug($request->get('trans'), $locale, $this->model);

It seems like $this->model can also be of type Illuminate\Contracts\Foundation\Application; however, parameter $page of Thinktomorrow\Chief\Pages\PageManager::enforceUniqueSlug() does only seem to accept Thinktomorrow\Chief\Pages\Page, maybe add an additional type check?

It seems like $request->get('trans') can also be of type null; however, parameter $translations of Thinktomorrow\Chief\Pages\PageManager::enforceUniqueSlug() does only seem to accept array, maybe add an additional type check?

            $trans[$locale] = $this->addDefaultShortDescription($translation);
        }

        // Merge with request...
        return $request->merge(['trans' => $trans]);
    }

    public function updateRequest(Request $request): Request
    {
        $trans = [];
        foreach ($request->get('trans', []) as $locale => $translation) {
            if (is_array_empty($translation)) {

                // Nullify all values
                $trans[$locale] = array_map(function ($value) {

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

                    return null;
                }, $translation);
                continue;
            }

            $translation = $this->enforceUniqueSlug($request->get('trans'), $locale, $this->model);

It seems like $request->get('trans') can also be of type null; however, parameter $translations of Thinktomorrow\Chief\Pages\PageManager::enforceUniqueSlug() does only seem to accept array, maybe add an additional type check?

It seems like $this->model can also be of type Illuminate\Contracts\Foundation\Application; however, parameter $page of Thinktomorrow\Chief\Pages\PageManager::enforceUniqueSlug() does only seem to accept Thinktomorrow\Chief\Pages\Page, maybe add an additional type check?

            $trans[$locale] = $this->addDefaultShortDescription($translation);
        }

        // Merge with request...
        return $request->merge(['trans' => $trans]);
    }

    public function afterStore($request)

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

    {
        Audit::activity()
            ->performedOn($this->model)

It seems like $this->model can also be of type Illuminate\Contracts\Foundation\Application; however, parameter $model of Spatie\Activitylog\ActivityLogger::performedOn() does only seem to accept Illuminate\Database\Eloquent\Model, maybe add an additional type check?

            ->log('created');
    }

    public function afterUpdate($request)

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

    {
        Audit::activity()
            ->performedOn($this->model)

It seems like $this->model can also be of type Illuminate\Contracts\Foundation\Application; however, parameter $model of Spatie\Activitylog\ActivityLogger::performedOn() does only seem to accept Illuminate\Database\Eloquent\Model, maybe add an additional type check?

            ->log('edited');
    }

    private function enforceUniqueSlug(array $translations, string $locale, Page $page): array
    {
        $translation = $translations[$locale];

        $translation['slug']    = $translation['slug'] ?? $translation['title'];
        $translation['slug']    = $this->uniqueSlug->get($translation['slug'], $page->getTranslation($locale));

It seems like $page->getTranslation($locale) can also be of type Illuminate\Database\Eloquent\Model; however, parameter $entity of Thinktomorrow\Chief\Concerns\Sluggable\UniqueSlug::get() does only seem to accept Thinktomorrow\Chief\Concerns\Sluggable\SluggableContract|null, maybe add an additional type check?

        return $translation;
    }

    /**
     * @param array $translation
     * @return array
     */
    private function addDefaultShortDescription(array $translation): array
    {
        if (isset($translation['content'])) {
            $translation['short'] = $translation['short'] ?? teaser($translation['content'], 100);
        }

        return $translation;
    }
}