Distilleries\Contentful\Models\Traits\Localable

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	scopeLocale()	0	8	3

<?php

namespace Distilleries\Contentful\Models\Traits;

use Illuminate\Database\Eloquent\Builder;
use Distilleries\Contentful\Models\Locale;

trait Localable
{
    abstract public function getTable();

    /**
     * Scope a query to a given locale.
     *
     * @param  \Illuminate\Database\Eloquent\Builder  $query
     * @param  string|null  $locale
     * @param  string|null  $country
     * @return \Illuminate\Database\Eloquent\Builder
     */
    public function scopeLocale($query, ?string $locale = '', ?string $country = ''): Builder
    {
        $locale = ! empty($locale) ? $locale : Locale::getAppOrDefaultLocale();
        $country = ! empty($country) ? $country : Locale::getAppOrDefaultCountry();

        return $query

The expression return $query->whereRaw('LOWER(' . $this->getTable() . '.country) LIKE LOWER("' . $country . '")')->whereRaw('LOWER(' . $this->getTable() . '.locale) LIKE LOWER("' . $locale . '")') could return the type Illuminate\Database\Query\Builder which is incompatible with the type-hinted return Illuminate\Database\Eloquent\Builder. Consider adding an additional type-check to rule them out.

            ->whereRaw('LOWER(' . $this->getTable() . '.country) LIKE LOWER("' . $country . '")')
            ->whereRaw('LOWER(' . $this->getTable() . '.locale) LIKE LOWER("' . $locale . '")');
    }
}