Thinktomorrow\AssetLibrary\Commands\ImageToAssetMigrateCommand::handle()

Complexity

Conditions	8
Paths	12

Size

Total Lines	76
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	41
CRAP Score	8.0064

Importance

Changes

0

<?php
    public function handle()
    {
        \ini_set('memory_limit', '256M');
        
        $tableName      = $this->argument('table');
        $imageUrlColumn = $this->argument('urlcolumn');
        $modelIdColumn  = $this->argument('idcolumn');
        $ordercolumn    = $this->argument('ordercolumn');
        $model          = $this->argument('linkedmodel');
        $unreachable    = 0;
        $files          = 0;

        if($ordercolumn){
            $results = DB::table($tableName)->select($imageUrlColumn, $modelIdColumn, $ordercolumn)->get();

It seems like $tableName can also be of type array; however, parameter $table of Illuminate\Support\Facades\DB::table() does only seem to accept string, maybe add an additional type check?

        }else{
            $results = DB::table($tableName)->select($imageUrlColumn, $modelIdColumn)->orderBy($modelIdColumn)->get();

It seems like $modelIdColumn can also be of type array; however, parameter $column of Illuminate\Database\Query\Builder::orderBy() does only seem to accept string, maybe add an additional type check?

        }

        $orderedResults = [];
        $results->each(function($result) use($modelIdColumn, $results, &$orderedResults, $imageUrlColumn){

The import $results is not used and could be removed.

            $orderedResults[$result->$modelIdColumn][] = $result->$imageUrlColumn;
        });

        $orderedResults = collect($orderedResults);

        if($this->option('reset')){
            $this->info('Resetting the asserts on the models');
            $resetbar = $this->output->createProgressBar(count($orderedResults));
            
            $orderedResults->each(function($entry, $key) use($model, $resetbar){
                optional($model::find($key))->deleteAllAssets();
                $resetbar->advance();
            });
            
            $resetbar->finish();
        }
        

        $bar = $this->output->createProgressBar(count($results));
        
        $this->info("\n" . 'Migrating images.');
        foreach($orderedResults as $key => $result)
        {
            $persistedModel = $model::find($key);
            if(!$persistedModel){
                continue;
            }

            foreach($result as $line){
                $bar->advance();
                
                try{
                    $asset = AssetUploader::uploadFromUrl(public_path($line));
                }catch(UnreachableUrl $ex){
                    // increment the amount of unreachable files counter
                    $unreachable++;
                    
                    continue;
                }
                
                $asset->attachToModel($persistedModel);
                
                if($this->option('force')){
                    unlink(public_path($line));
                }
    
                // increment the amount of files migrated counter
                $files++;
            }
        }
       
        $bar->finish();

        $this->info('Migrating done.');
        $this->info('Migrated '. $files . ' files.');
        $this->info('Couldn\'t reach '. $unreachable . ' files.');
    }