LaraGeoData\Models\HasTableWithSuffix

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getTable()	0	11	3
A	useSuffix()	0	9	2

<?php

namespace LaraGeoData\Models;

trait HasTableWithSuffix
{
    abstract public function getTableNameRoot(): string;

    public function getTable()
    {
        if ($this->table) {
            return $this->table;
        }
        $tableName = $this->getTableNameRoot();
        if ($suffix = config('geonames.database.default_suffix')) {
            $tableName = "{$tableName}_{$suffix}";
        }

        return $tableName;
    }

    public static function useSuffix(?string $suffix = null): static
    {
        $instance  = new static();
        $tableName = $instance->getTableNameRoot();
        if ($suffix) {
            $tableName = "{$tableName}_{$suffix}";
        }

        return $instance->setTable($tableName);

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

    }
}