SaveUrlSlugs::__construct() - Code Metrics - Inspection of "Bump vue-server-renderer from 2.5.22 to 2.6.10" - thinktomorrow/chief - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — dependabot/npm_and_yarn/vue-se... ( e78fac...845b60 )

unknown

created 2019-07-25 14:46 UTC

SaveUrlSlugs::__construct() A

↳ Parent: SaveUrlSlugs

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

Metric	Value
eloc	1
dl	0
loc	3
ccs	2
cts	2
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	1

<?php

namespace Thinktomorrow\Chief\Urls\Application;

use Thinktomorrow\Chief\Urls\ProvidesUrl\ProvidesUrl;
use Thinktomorrow\Chief\Urls\UrlRecord;

class SaveUrlSlugs
{
    /** @var bool */
    private $strict = true;

    /** @var ProvidesUrl */
    private $model;

    private $existingRecords;

    public function __construct(ProvidesUrl $model)
    {
        $this->model = $model;
    }

    /**
     * Saving urls slugs in strict mode prevents identical urls to be automatically removed.
     * When set to false, this would remove the identical url records.
     *
     * @param bool $strict
     * @return $this
     */
    public function strict(bool $strict = true)
    {
        $this->strict = $strict;

        return $this;
    }

    public function handle(array $slugs): void
    {
        $this->existingRecords = UrlRecord::getByModel($this->model);

        foreach ($slugs as $locale => $slug) {
            if (!$slug) {
                $this->deleteRecord($locale);
                continue;
            }

            $this->saveRecord($locale, $this->prependBaseUrlSegment($slug, $locale));
        }
    }

    private function deleteRecord(string $locale)
    {
        return $this->saveRecord($locale, null);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    private function saveRecord(string $locale, ?string $slug)
    {
        // Existing ones for this locale?
        $nonRedirectsWithSameLocale = $this->existingRecords->filter(function ($record) use ($locale) {
            return (
                $record->locale == $locale &&
                !$record->isRedirect()
            );
        });

        // If slug entry is left empty, all existing records will be deleted
        if (!$slug) {
            $nonRedirectsWithSameLocale->each(function ($existingRecord) {
                $existingRecord->delete();
            });

            return;
        }

        $this->cleanupExistingRecords($locale, $slug);

        // If slug entry is left empty, all existing records will be deleted
        if ($nonRedirectsWithSameLocale->isEmpty()) {
            $this->createRecord($locale, $slug);
            return;
        }

        // Only replace the existing records that differ from the current passed slugs
        $nonRedirectsWithSameLocale->each(function ($existingRecord) use ($slug) {
            if ($existingRecord->slug != $slug) {
                $existingRecord->replaceAndRedirect(['slug' => $slug]);
            }
        });
    }

    private function createRecord($locale, $slug)
    {
        UrlRecord::create([
            'locale'              => $locale,
            'slug'                => $slug,
            'model_type'          => $this->model->getMorphClass(),
            'model_type'          => $this->model->/** @scrutinizer ignore-call */ getMorphClass(),
            'model_id'            => $this->model->id,

        ]);
    }

    /**
     * @param string $locale
     * @param string|null $slug
     */
    private function cleanupExistingRecords(string $locale, string $slug): void
    {
        // In the case where we have any redirects that match the given slug, we need to
        // remove the redirect record in favour of the newly added one.
        $this->deleteIdenticalRedirects($this->existingRecords, $locale, $slug);

        $sameExistingRecords = UrlRecord::where('slug', $slug)->where('locale', $locale)->get();

        // Also delete any redirects that match this locale and slug but are related to another model
        $this->deleteIdenticalRedirects($sameExistingRecords, $locale, $slug);

        // Also delete any urls that match this locale and slug but are related to another model
        $this->deleteIdenticalRecords($sameExistingRecords);
    }

    /**
     * Remove any redirects owned by this model that equal the new slug.
     *
     * @param $existingRecords
     * @param $locale
     * @param $slug
     */
    private function deleteIdenticalRedirects($existingRecords, $locale, $slug): void
    {
        $existingRecords->filter(function ($record) use ($locale) {
            return (
                $record->locale == $locale &&
                $record->isRedirect()
            );
        })->each(function ($existingRecord) use ($slug) {
            if ($existingRecord->slug == $slug) {
                $existingRecord->delete();
            }
        });
    }

    private function deleteIdenticalRecords($existingRecords): void
    {
        if ($this->strict) {
            return;
        }

        // The old homepage url should be removed since this is no longer in effect.
        // In case of any redirect to this old homepage, the last used redirect is now back in effect.
        $existingRecords->reject(function ($existingRecord) {
            return (
                $existingRecord->model_type == $this->model->getMorphClass() &&
                $existingRecord->model_id == $this->model->id);

        })->each(function ($existingRecord) {

            // TODO: if there is a redirect to this page, we'll take this one as the new url
            $existingRecord->delete();
        });
    }

    /**
     * @param string $slug
     * @param $locale
     * @return string
     */
    private function prependBaseUrlSegment(string $slug, $locale): string
    {
        $slugWithBaseSegment = $this->model->baseUrlSegment($locale) . '/' . $slug;
        $slugWithBaseSegment = trim($slugWithBaseSegment, '/');

        // If slug with base segment is empty string, it means that the passed slug was probably a "/" character.
        // so we'll want to return it in case the base segment is not added.
        return $slugWithBaseSegment ?: '/';
    }
}


thinktomorrow / chief

Push — dependabot/npm_and_yarn/vue-se... ( e78fac...845b60 )

SaveUrlSlugs::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like