Distilleries\Contentful\Models\Locale

Complexity

Total Complexity

19

Size/Duplication

Total Lines	138
Duplicated Lines	22.46 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	default()	14	14	3
A	getAppOrDefaultLocale()	0	4	1
A	getAppOrDefaultCountry()	0	4	1
A	defaultCountry()	17	17	3
A	fallback()	0	13	4
A	canBeSave()	0	6	1
A	getLocale()	0	9	2
A	getCountry()	0	9	2
A	getLocaleAttribute()	0	4	1
A	getCountryAttribute()	0	4	1

<?php

namespace Distilleries\Contentful\Models;

use Illuminate\Support\Facades\Cache;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Str;

/**
 * @property integer $id
 * @property string $label
 * @property string $code
 * @property string $fallback_code
 * @property boolean $is_editable
 * @property boolean $is_publishable
 * @property \Illuminate\Support\Carbon $created_at
 * @property \Illuminate\Support\Carbon $updated_at
 */
class Locale extends Model
{
    /**
     * {@inheritdoc}
     */
    protected $table = 'locales';

    /**
     * {@inheritdoc}
     */
    protected $fillable = [
        'label',
        'code',
        'fallback_code',
        'is_default',
        'is_editable',
        'is_publishable',
    ];

    /**
     * {@inheritdoc}
     */
    protected $casts = [
        'is_default' => 'boolean',
        'is_editable' => 'boolean',
        'is_publishable' => 'boolean',
    ];

    /**
     * Return default locale code.
     *
     * @return string
     */
    public static function default(): string

Possible parse error: non-abstract method defined as abstract

This method seems to be duplicated in your project.

    {
        $default = Cache::get('locale_default');

The visibility should be declared for property $default.

        if ($default === null) {

It is generally advisable to only define one property per statement.

The visibility should be declared for property $default.

            $default = static::query()->select('code')->where('is_default', '=', true)->first();

The method select() does not exist on Illuminate\Database\Eloquent\Builder. Did you maybe mean createSelectWithConstraint()?

            $default = !empty($default) ? $default->code : config('contentful.default_locale');

It is generally advisable to only define one property per statement.

The visibility should be declared for property $default.

            $default = self::getLocale($default);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $default.

            // Cache is cleaned in Console\Commands\SyncLocales (run at least daily)
            Cache::forever('locale_default', $default);

The visibility should be declared for property $default.

        }

        return $default;

The visibility should be declared for property $default.

    }


    public static function getAppOrDefaultLocale(): string
    {
        return app()->getLocale() ?? self::default();
    }

    public static function getAppOrDefaultCountry($key = 'app.country'): string
    {
        return config($key, self::defaultCountry());
    }

    /**
     * Return default country code.
     *
     * @return string
     */
    public static function defaultCountry(): string

This method seems to be duplicated in your project.

    {
        $default = Cache::get('country_default');

        if ($default === null) {
            $default = static::query()

The method select() does not exist on Illuminate\Database\Eloquent\Builder. Did you maybe mean createSelectWithConstraint()?

                ->select('code')
                ->where('is_default', '=', true)
                ->first();
            $default = !empty($default) ? $default->code : config('contentful.default_country');
            $default = self::getCountry($default);
            // Cache is cleaned in Console\Commands\SyncLocales (run at least daily)
            Cache::forever('country_default', $default);
        }

        return $default;
    }

    /**
     * Return fallback code for given locale code.
     *
     * @param  string $code
     * @return string
     */
    public static function fallback(string $code): string
    {
        $fallback = Cache::get('locale_fallback_' . $code);

        if ($fallback === null) {
            $locale = static::query()->select('fallback_code')->where('code', '=', $code)->first();

The method select() does not exist on Illuminate\Database\Eloquent\Builder. Did you maybe mean createSelectWithConstraint()?

            $fallback = (!empty($locale) and !empty($locale->fallback_code)) ? $locale->fallback_code : '';

Using logical operators such as and instead of && is generally not recommended.

            Cache::put('locale_fallback_' . $code, $fallback, 5);
        }

        return $fallback;
    }

    public static function canBeSave(string $country,string $locale): bool
    {
        $locales = config('contentful.locales_not_flatten','');
        $locales = explode(',',$locales);
        return !in_array($country.'_'.$locale,$locales);
    }
    public static function getLocale(string $locale): string
    {
        if (Str::contains($locale, '_')) {
            $tab = explode('_', $locale);
            return $tab[1];
        }

        return $locale;
    }

    public static function getCountry(string $locale): string
    {
        if (Str::contains($locale, '_')) {
            $tab = explode('_', $locale);
            return $tab[0];
        }

        return config('contentful.default_country');
    }

    public function getLocaleAttribute(): string
    {
        return self::getLocale($this->code);
    }

    public function getCountryAttribute(): string
    {
        return self::getCountry($this->code);
    }
}