LaraGeoData\Models\HasTableWithSuffix

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
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	makeUsingSuffix()	0	9	2

<?php

namespace LaraGeoData\Models;

trait HasTableWithSuffix
{
    /**
     * Get base table name (root name, without suffix).
     *
     * @return string
     */
    abstract public function getTableNameRoot(): string;

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

        return $tableName;
    }

    /**
     * Initialise model with suffixed table name.
     *
     * @param string|null $suffix
     * @return static
     */
    public static function makeUsingSuffix(?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?

    }
}