Fomvasss\UrlAliases\Traits\UrlAliasable

Complexity

Total Complexity

10

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	urlAlias()	0	5	1
A	urlAliases()	0	5	1
A	getLocaleboundStr()	0	7	2
A	scopeByLocale()	0	5	2
A	scopeByLocales()	0	6	4

<?php
/**
 * Created by PhpStorm.
 * User: fomvasss
 * Date: 21.08.18
 * Time: 11:34
 */

namespace Fomvasss\UrlAliases\Traits;

trait UrlAliasable
{
    /**
     * @return mixed
     */
    public function urlAlias()
    {
        $model = config('url-aliases.model', \Fomvasss\UrlAliases\Models\UrlAlias::class);

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

        return $this->morphOne($model, 'model');

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

    }

    /**
     * @return mixed
     */
    public function urlAliases()
    {
        $model = config('url-aliases.model', \Fomvasss\UrlAliases\Models\UrlAlias::class);

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

        return $this->morphMany($model, 'model');

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

    }

    /**
     * @return |null

The doc comment |null at position 0 could not be parsed: Unknown type name '|' at position 0 in |null.

     */
    public function getLocaleboundStr()
    {
        if ($this->urlAlias) {
            return $this->urlAlias->locale_bound;
        } 
        
        return null;
    }


    /**
     * @param $query
     * @param null $locale

Are you sure the doc-type for parameter $locale is correct as it would always require null to be passed?

     * @return mixed
     */
    public function scopeByLocale($query, $locale = null)
    {
        $locale = $locale ?: \UrlAliasLocalization::getCurrentLocale();

$locale is of type null, thus it always evaluated to false.

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

        return $query->whereLocale($locale);
    }
    

    /**
     * @param $query
     * @param null $locales

Are you sure the doc-type for parameter $locales is correct as it would always require null to be passed?

     * @return mixed
     */
    public function scopeByLocales($query, $locales = null)
    {
        $locales = $locales ?: session('app_locales');

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

$locales is of type null, thus it always evaluated to false.

        if ($locales) {
            return $query->whereIn('locale', is_array($locales) ? $locales : [$locales]);
        }
    }

}