Turahe\Master\Commands\SyncCoordinateCommand::handle()

Complexity

Conditions	3
Paths	1

Size

Total Lines	69
Code Lines	56

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle()
    {
        $count = Province::count();
        $this->info("Processing {$count} Province");
        $bar = $this->output->createProgressBar($count);

It seems like $count can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $max of Symfony\Component\Console\Style\SymfonyStyle::createProgressBar() does only seem to accept integer, maybe add an additional type check?

        $bar->start();
        Province::cursor()->each(function ($item) use ($bar) {
            $meta = $item->meta;
            $geocoding = \Geocoder::getCoordinatesForAddress($item->address);

The type Geocoder was not found. Maybe you did not declare it correctly or list all dependencies?

            $meta['lat'] = $geocoding['lat'] ?? null;
            $meta['long'] = $geocoding['lng'] ?? null;
            $item->meta = $meta;
            $item->save();
            $bar->advance();
        });

        $this->line('');

        $count = City::count();
        $this->info("Processing {$count} City");
        $bar = $this->output->createProgressBar($count);

It seems like $count can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $max of Symfony\Component\Console\Style\SymfonyStyle::createProgressBar() does only seem to accept integer, maybe add an additional type check?

        $bar->start();
        City::with('provinsi')->cursor()->each(function ($item) use ($bar) {
            $meta = $item->meta;
            $geocoding = \Geocoder::getCoordinatesForAddress($item->address);
            $meta['lat'] = $geocoding['lat'] ?? null;
            $meta['long'] = $geocoding['lng'] ?? null;
            $item->meta = $meta;
            $item->save();
            $bar->advance();
        });

        $this->line('');

        $count = District::count();
        $this->info("Processing {$count} District");
        $bar = $this->output->createProgressBar($count);

It seems like $count can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $max of Symfony\Component\Console\Style\SymfonyStyle::createProgressBar() does only seem to accept integer, maybe add an additional type check?

        $bar->start();
        District::with('kabupaten.provinsi')->cursor()->each(function ($item) use ($bar) {
            $meta = $item->meta;
            $geocoding = \Geocoder::getCoordinatesForAddress($item->address);
            $meta['lat'] = $geocoding['lat'] ?? null;
            $meta['long'] = $geocoding['lng'] ?? null;
            $item->meta = $meta;
            $item->save();
            $bar->advance();
        });

        $this->line('');

        $count = Village::whereNull('meta')->count();
        $this->info("Processing {$count} Village");
        $bar = $this->output->createProgressBar($count);

It seems like $count can also be of type Illuminate\Database\Eloquent\Builder and Illuminate\Database\Query\Builder; however, parameter $max of Symfony\Component\Console\Style\SymfonyStyle::createProgressBar() does only seem to accept integer, maybe add an additional type check?

        $bar->start();
        Village::whereNull('meta')->cursor()->each(function ($item) use ($bar) {
            $meta = $item->meta;
            if (!$meta) {
                try {
                    $geocoding = \Geocoder::getCoordinatesForAddress($item->address);
                    $meta['lat'] = $geocoding['lat'] ?? null;
                    $meta['long'] = $geocoding['lng'] ?? null;
                    $item->meta = $meta;
                    $item->save();
                } catch (ServerException $e) {
                    $this->error($e->getMessage());
                    sleep(1);
                }
            }
            $bar->advance();
        });