DavideCasiraghi\LaravelEventsCalendar\Models\Region

Complexity

Total Complexity

2

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRegionsByCountry()	0	9	1
A	getRegionName()	0	5	1

<?php

namespace DavideCasiraghi\LaravelEventsCalendar\Models;

use Astrotomic\Translatable\Translatable;
use Illuminate\Database\Eloquent\Model;

class Region extends Model
{
    /***************************************************************************/
    /**
     * The table associated with the model.
     *
     * @var string
     */
    protected $table = 'regions';

    /***************************************************************************/

    use Translatable;

The trait Astrotomic\Translatable\Translatable requires the property $each which is not provided by DavideCasiraghi\LaravelEventsCalendar\Models\Region.

    public $translatedAttributes = ['name', 'slug'];
    protected $fillable = ['country_id', 'timezone'];
    public $useTranslationFallback = true;

    /***************************************************************************/

    /*
     * Return the region name.
     *
     * @param  int  $regionId
     * @return string
     */
    public static function getRegionName($regionId): string
    {
        $ret = self::find($regionId)->name;

The property name does not seem to exist on DavideCasiraghi\LaravelEventsCalendar\Models\Region. Are you sure there is no database migration missing?

        return $ret;
    }

    /***************************************************************************/

    /**
     * Return all the countries ordered by name.
     *
     * @return \Illuminate\Support\Collection
     */
    public static function getRegionsByCountry($countryId)
    {
        $ret = self::join('region_translations', 'regions.id', '=', 'region_translations.region_id')
                    ->where('locale', 'en')
                    ->where('country_id', $countryId)
                    ->orderBy('name')
                    ->pluck('name', 'region_translations.region_id AS id');

        return $ret;
    }
}