Distilleries\Contentful\Commands\Sync\SyncFlatten

Complexity

Total Complexity

10

Size/Duplication

Total Lines	144
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	10

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	SyncTrait::dumpSync()	14	14	3
A	SyncTrait::dumpSql()	15	15	1
A	SyncTrait::getConnector()	0	10	3
A	SyncTrait::putSync()	0	11	2
A	SyncTrait::putSql()	15	15	1

<?php

namespace Distilleries\Contentful\Commands\Sync\Traits;

trait SyncTrait
{
    /**
     * Switch to `sync` database.
     *
     * @return void
     */
    protected function switchToSyncDb()
    {
        $this->warn('Switch to sync database');

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

        config([
            'database.default' => 'mysql_sync',
        ]);
    }

    /**
     * Dump `sync` database into a SQL file and return its path.
     *
     * @param  boolean $isPreview
     * @return string
     */
    protected function dumpSync(bool $isPreview, string $connector = 'mysql'): string

This method seems to be duplicated in your project.

    {
        $path = storage_path('dumps/' . date('YmdHis') . '_sync' . ($isPreview ? '_preview' : '') . '.sql');
        $this->warn('Dump "' . basename($path) . '"...');

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

        $dirName = dirname($path);
        if (!is_dir($dirName)) {
            mkdir($dirName, 0777, true);
        }

        $this->dumpSql($path, $this->getConnector($isPreview, $connector));

        return realpath($path);
    }

    /**
     * Dump SQL database in given file path.
     *
     * @param  string $path
     * @param  string $connector
     * @return void
     */
    protected function dumpSql(string $path, string $connector)

This method seems to be duplicated in your project.

    {
        $exec = 'export MYSQL_PWD=\'%s\'; mysqldump --add-drop-table --default-character-set=%s %s -u %s -h %s --port %s > %s';

        $command = sprintf($exec,
            addcslashes(config('database.connections.' . $connector . '.password'), "'"),
            config('database.connections.' . $connector . '.charset'),
            config('database.connections.' . $connector . '.database'),
            config('database.connections.' . $connector . '.username'),
            config('database.connections.' . $connector . '.host'),
            config('database.connections.' . $connector . '.port'),
            $path
        );
        exec($command);
    }

    protected function getConnector(bool $isPreview, string $connector = 'mysql'): string
    {
        $compiledConnector = $connector . ($isPreview ? '_preview' : '');
        if (empty(config('database.connections.' . $compiledConnector . '.username'))) {
            $compiledConnector = $connector;
        }

        return $compiledConnector;

    }


    /**
     * Put previous dump in live-preview database.
     *
     * @param  string $path
     * @param  boolean $isPreview
     * @return void
     */
    protected function putSync(string $path, bool $isPreview, string $connector = 'mysql')
    {
        $compiledConnector = $this->getConnector($isPreview, $connector);

        config([
            'database.default' => $compiledConnector,
        ]);

        $this->warn('Put into "' . $compiledConnector . '" database...');

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

        $this->putSql($path, $compiledConnector);
    }

    /**
     * Put SQL file into given database.
     *
     * @param  string $path
     * @param  string $connector
     * @return void
     */
    protected function putSql(string $path, string $connector)

This method seems to be duplicated in your project.

    {
        $exec = 'export MYSQL_PWD=\'%s\'; mysql -u %s -h %s --port %s %s < %s';

        $command = sprintf($exec,
            addcslashes(config('database.connections.' . $connector . '.password'), "'"),
            config('database.connections.' . $connector . '.username'),
            config('database.connections.' . $connector . '.host'),
            config('database.connections.' . $connector . '.port'),
            config('database.connections.' . $connector . '.database'),
            $path
        );

        exec($command);
    }
}