Larrock\Core\Traits\GetSeo

Complexity

Total Complexity

6

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	6
Bugs	2	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getSeo()	0	3	1
A	getGetSeoTitleAttribute()	0	13	5

<?php

namespace Larrock\Core\Traits;

use Cache;
use Larrock\Core\Models\Seo;

trait GetSeo
{
    public function getSeo()
    {
        return $this->hasOne(Seo::class, 'seo_id_connect', 'id')->whereSeoTypeConnect($this->config->name);

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

    }

    public function getGetSeoTitleAttribute()
    {
        $cache_key = sha1('getGetSeoTitleAttribute'.$this->id.$this->config->name);

        return Cache::rememberForever($cache_key, function () {
            if (($get_seo = Seo::whereSeoUrlConnect($this->url)->whereSeoTypeConnect($this->config->name)->first()) && $get_seo->seo_title) {
                return $get_seo->seo_title;
            }
            if (($get_seo = Seo::whereSeoIdConnect($this->id)->whereSeoTypeConnect($this->config->name)->first()) && $get_seo->seo_title) {
                return $get_seo->seo_title;
            }

            return $this->title;
        });
    }
}