forked from Torann/laravel-geoip
 
 
 - 
  Notifications
 You must be signed in to change notification settings 
- Fork 4
-
Hello and thanks for maintaining this fork!
I've tried to update MaxMind's geoip database through a Artisan::call('geoip:update'); call but it returned the following message: The command "geoip:update" does not exist.
Upon further inspection, in GeoIPServiceProvider.php the following codeblock is responsible:
if ($this->app->runningInConsole()) { $this->registerResources(); $this->registerGeoIpCommands(); }
Is there any reason for it to check whether it's running through CLI and only then to register the commands?
I mean wouldn't it be better for the end-user to be able to update the database without resorting to php artisan commands?
Beta Was this translation helpful? Give feedback.
All reactions
Replies: 0 comments
 
 Sign up for free
 to join this conversation on GitHub.
 Already have an account?
 Sign in to comment