DavideCasiraghi\LaravelEventsCalendar\Console\RetrieveAllGpsCoordinates

Complexity

Total Complexity

3

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

27.27%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	9	2
A	__construct()	0	3	1

<?php
namespace DavideCasiraghi\LaravelEventsCalendar\Console;

use Illuminate\Console\Command;
use DavideCasiraghi\LaravelEventsCalendar\Facades\LaravelEventsCalendar;
use DavideCasiraghi\LaravelEventsCalendar\Models\Country;
use DavideCasiraghi\LaravelEventsCalendar\Models\EventVenue;

class RetrieveAllGpsCoordinates extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature =  'retrieve-all-gps-coordinates';
    
    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Assign to all the venues the corresponding GPS coordinates';
    
    /**
    * Create a new command instance.
    *
    * @return void
    */
   public function __construct()
   {
       parent::__construct();
   }
   
    /**
    * Execute the console command.
    *
    * @param  
    * @return mixed
    */

The doc comment @return at position 0 could not be parsed: Unknown type name '@return' at position 0 in @return.

    public function handle(){
        $eventVenues = EventVenue::all();
        foreach ($eventVenues as $key => $eventVenue) {
            // Get GPS coordinates
            $address = Country::getCountryName($eventVenue->country_id).', '.$eventVenue->city.', '.$eventVenue->address;
            $gpsCoordinates = LaravelEventsCalendar::getVenueGpsCoordinates($address);

The method getVenueGpsCoordinates() does not exist on DavideCasiraghi\LaravelEventsCalendar\Facades\LaravelEventsCalendar. Since you implemented __callStatic, consider adding a @method annotation.

            $eventVenue->lat = $gpsCoordinates['lat'];
            $eventVenue->lng = $gpsCoordinates['lng'];
            $eventVenue->save();
        }
    }
}