Rinvex\Support\Traits\SeederHelper

Complexity

Total Complexity

9

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	seedResources()	0	17	4
A	ensureExistingDatabaseTables()	0	9	2
A	hasDatabaseTables()	0	12	3

<?php

declare(strict_types=1);

namespace Rinvex\Support\Traits;

use Closure;
use Exception;
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Eloquent\Model;

trait SeederHelper
{
    /**
     * Seed the given resources.
     *
     * @param \Illuminate\Database\Eloquent\Model $model
     * @param string                              $seeder
     * @param array                               $initialExclude
     * @param \Closure                            $callback

Should the type for parameter $callback not be null|Closure?

     *
     * @throws \Exception
     *
     * @return void
     */
    protected function seedResources(Model $model, string $seeder, array $initialExclude = [], Closure $callback = null)
    {
        if (! file_exists($seeder)) {
            throw new Exception("Resources seeder file '{$seeder}' does NOT exist!");
        }

        $this->warn('Seeding: '.str_after($seeder, $this->laravel->basePath().'/'));

The property laravel does not exist. Did you maybe forget to declare it?

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

        // Create new resources
        foreach (json_decode(file_get_contents($seeder), true) as $resource) {
            $model->firstOrCreate(array_except($resource, $initialExclude), array_only($resource, $initialExclude));
        }

        $this->info('Seeded: '.str_after($seeder, $this->laravel->basePath().'/'));

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

        ! $callback || call_user_func($callback);
    }

    /**
     * Ensure existing database tables.
     *
     * @param string $package
     *
     * @return bool
     */
    protected function ensureExistingDatabaseTables(string $package)
    {
        if (! $this->hasDatabaseTables($package)) {
            $package = explode('/', $package);
            $this->call("{$package[0]}:migrate:{$package[1]}");

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

        }

        return true;
    }

    /**
     * Check if all required database tables exists.
     *
     * @param string $package
     *
     * @return bool
     */
    protected function hasDatabaseTables(string $package)
    {
        $package = explode('/', $package);

        foreach (config("{$package[0]}.{$package[1]}.tables") as $table) {
            if (! Schema::hasTable($table)) {
                return false;
            }
        }

        return true;
    }
}