RuthgerIdema\UrlRewrite\Traits\HasUrlRewrite

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getUrlAttribute()	0	8	2
A	getUrlRewriteAttributesArray()	0	10	2
A	getUrlRewrite()	0	7	1

<?php

declare(strict_types=1);

namespace RuthgerIdema\UrlRewrite\Traits;

use RuthgerIdema\UrlRewrite\Facades\UrlRewrite;

trait HasUrlRewrite
{
    public function getUrlAttribute(): string
    {
        if (! $urlRewrite = $this->getUrlRewrite()) {
            return '';
        }

        return route('url.rewrite', $urlRewrite->request_path, false);
    }

    public function getUrlRewriteAttributesArray(): ?array
    {
        $mapped = [];

        foreach (config("url-rewrite.types.$this->urlRewriteType.attributes") as $attribute) {

The property urlRewriteType does not exist. Did you maybe forget to declare it?

            $mapped[$attribute] = $this->getAttribute($attribute);

It seems like getAttribute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $mapped;
    }

    public function getUrlRewrite(): ?object
    {
        return UrlRewrite::getByTypeAndAttributes(
            config("url-rewrite.types.$this->urlRewriteType.route"),
            $this->getUrlRewriteAttributesArray()
        );
    }
}