SelectWorksheetsStep::PreProcess() - Code Metrics - Inspection of "1b8e354d4c0dad913f1b592a148a59ad0348eb9c" - jonphipps/Metadata-Registry - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — beta (1b8e35)

by Jon

created 2018-01-22 22:04 UTC

SelectWorksheetsStep::PreProcess() D

↳ Parent: SelectWorksheetsStep

Complexity

Conditions	11
Paths	130

Size

Total Lines	36
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	36
rs	4.9629
c	0
b	0
f	0
cc	11
eloc	29
nc	130
nop	2

How to fix Complexity

<?php

/** Created by PhpStorm,  User: jonphipps,  Date: 2017-05-31,  Time: 11:14 AM */

namespace App\Wizard\Import\ProjectSteps;

use App\Jobs\ParseVocabulary;
use App\Models\Batch;
use App\Models\ConceptAttribute;
use App\Models\ElementAttribute;
use App\Models\Export;
use App\Models\Import;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Validator;
use Smajti1\Laravel\Step;
use Smajti1\Laravel\Wizard;
use function dispatch;
use function explode;

class SelectWorksheetsStep extends Step
{
    public static $label = 'Choose the Worksheets...';
    public static $slug  = 'worksheets';
    public static $view  = 'frontend.import.project.steps.worksheets';

    public function fields(): array
    {
        return [
            [
                'name'  => 'worksheets',
                'label' => 'Select the worksheets to import',
                'type'  => 'jqxgrid_select',
            ],
        ];
    }

    public function PreProcess(Request $request, Wizard $wizard)
    public function PreProcess(/** @scrutinizer ignore-unused */ Request $request, Wizard $wizard)
    {
        $worksheets = [];
        $sheets     = $wizard->data()['googlesheets'];
        //it's already in the correct form
        if (is_array($sheets) && isset($sheets[0])) {
            return;
        }
        foreach ($sheets as $key => $worksheet) {
            $export               = Export::find($worksheet['id']);
            $sheet['worksheet']   = $export->worksheet;
            $sheet['languages']   = $export->languages;
            $sheet['exported_at'] = $export->created_at->toDayDateTimeString();
            $sheet['exported_at'] = $export->created_at->toDayDateTimeString();
            if ($export->elementset) {
                $id                     = $export->elementset->id;
                $sheet['last_edit']     = ElementAttribute::getLatestDateForElementSet($id);
                $sheet['elementset_id'] = $id;
            }
            if ($export->vocabulary) {
                $id                     = $export->vocabulary->id;
                $sheet['last_edit']     = ConceptAttribute::getLatestDateForVocabulary($id);
                $sheet['vocabulary_id'] = $id;
            }
            $sheet['last_edit']            =
                $sheet['last_edit'] ? $sheet['last_edit']->toDayDateTimeString() : 'Never Edited';

            $sheet['id']                   = $export->id . '::' . $key;
            $lastImport                    = $export->getLatestImport();
            $sheet['last_import']          =
                ($lastImport && $lastImport->imported_at) ? $lastImport->imported_at->toDayDateTimeString() :
                    'Never Imported';
            $sheet['last_import_batch_id'] = ($lastImport && $lastImport->batch_id) ? $lastImport->batch_id : '';
            $worksheets[]                  = $sheet;
        }
        $data                 = $wizard->data();
        $data['googlesheets'] = $worksheets;
        $wizard->data($data);
    }

    public function process(Request $request): void
    {
        $worksheets[] = json_decode($request->selected_worksheets);

        $spreadsheet  = $this->wizard->dataGet('spreadsheet');
        $batch_id     = $this->wizard->dataGet('batch_id');
        /** @var Batch $batch */
        $batch                = Batch::findOrFail($batch_id);
        $batch->total_count   = count($worksheets[0]);
        $unfinishedImports    = $batch->imports()->whereNull('imported_at');
        if ($unfinishedImports) {
            $unfinishedImports->delete();
        }
        $batch->save();

        // setup a job for each worksheet
        foreach ($worksheets[0] as $worksheet) {
            [ $export_id, $worksheet_id ] = explode('::', $worksheet);
            $export                       = Export::find($export_id);
            $import                       =
                Import::create([
                    'worksheet'     => $worksheet_id,
                    'source'        => 'Google',
                    'user_id'       => auth()->id(),
                    'batch_id'      => $batch_id,
                    'schema_id'     => $export->schema_id,
                    'vocabulary_id' => $export->vocabulary_id,
                ]);
            $export->addImports($import);
            // setup a job to run the worksheet processor to get the change instructions for each vocab
            dispatch(new ParseVocabulary($import, $worksheet_id, $spreadsheet));
            // each worksheet being processed will have its own checkbox table, hidden as a master->detail
            //it should look as if each of the worksheet selection rows had simply expanded to show the instructions
            // the row itself should have the master of the master-detail row just below with a summary of the changes
            $foo = 'bar';

        }

        // save progress to session
        $this->saveProgress($request, ['googlesheets' => $this->wizard->dataGet('googlesheets')]);
    }

    public function validate(Request $request)
    {
        Validator::make(['selected_worksheets' => json_decode($request->selected_worksheets)],
            ['selected_worksheets'          => 'required'],
            ['selected_worksheets.required' => 'You must select at least one worksheet before you can move to the next step.'])
            ->validate();
    }

    public function rules(Request $request = null)
    {
        return [
            'selected_worksheets' => 'required',
        ];
    }
}


jonphipps / Metadata-Registry

Branch — beta (1b8e35)

SelectWorksheetsStep::PreProcess() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like