Fix global memory limit set in constructor

Instead, set it when running.

Having the set in the constructor was causing memory_limit to be changed
for "artisan queue:work" processes.
This commit is contained in:
Dan Church 2021-09-27 14:57:18 -05:00
parent dbf314151e
commit c685e3e9f4
No known key found for this signature in database
GPG key ID: EA2BF379CD2CDBD0

View file

@ -65,7 +65,6 @@ class ImportCities extends Command
public function __construct() public function __construct()
{ {
parent::__construct(); parent::__construct();
ini_set('memory_limit', '256M');
} }
/** /**
@ -75,6 +74,8 @@ class ImportCities extends Command
*/ */
public function handle() public function handle()
{ {
$old_memory_limit = ini_get('memory_limit');
ini_set('memory_limit', '256M');
$path = storage_path('app/cities.json'); $path = storage_path('app/cities.json');
if(hash_file('sha512', $path) !== self::CHECKSUM) { if(hash_file('sha512', $path) !== self::CHECKSUM) {
@ -136,6 +137,7 @@ class ImportCities extends Command
$this->line(''); $this->line('');
$this->info('Successfully imported ' . $cityCount . ' entries!'); $this->info('Successfully imported ' . $cityCount . ' entries!');
$this->line(''); $this->line('');
ini_set('memory_limit', $old_memory_limit);
return; return;
} }