Turahe\Master\Commands\SyncCoordinateCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	handle()	0	69	3

<?php

namespace Turahe\Master\Commands;

use GuzzleHttp\Exception\ServerException;
use Illuminate\Console\Command;
use Turahe\Master\Models\City;
use Turahe\Master\Models\District;
use Turahe\Master\Models\Province;
use Turahe\Master\Models\Village;

class SyncCoordinateCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'turahe:master:sync-coordinate';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Synchronize latitude longitude data in database directly using Google\'s geocoding service';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    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();
        });
    }
}