Laravel BYR to BYN currency converter command

convertBYRToBYN.php

<?php

namespace App\Console\Commands;

use Illuminate\Console\Command;

class ConvertBYRToBYN extends Command
{
   
    protected $signature = 'convert-byr-to-byn';

    protected $description = 'Convert BYR to BYN in database.';

    private $count = 0;

    public function handle()
    {
        $query = \DB::table('ads')->where('currency', 'BYR');

        $this->line("We have {$query->count()} ads with BYR.");

        $query->chunk(1000, function ($ads) {
            foreach ($ads as $ad) {
                $this->updateOne($ad);
                $this->count++;
            }
            $this->line($this->count);
        });

        $this->comment("Done updating BYR to BYN. Total: $this->count ads.");
    }

    /**
    * Обновляем валюту для конкретного объявления.
    */
    private function updateOne($ad){
        $price = $ad->ad_price < 10000 ? $ad->ad_price : $ad->ad_price/10000;

        \DB::table('ads')->where([
            'id' => $ad->id
        ])->update([
            'ad_price' => $price,
            'currency' => 'BYN',
        ]);
    }
}

/*
Результаты выполнения комманды:
1) Почему-то обработана только половина выдачи.
We have 18356 ads with BYR.
1000
2000
3000
4000
5000
6000
7000
8000
9000
9356
Done updating BYR to BYN. Total: 9356 ads.

2) Снова только половина
We have 9000 ads with BYR.
1000
2000
3000
4000
5000
Done updating BYR to BYN. Total: 5000 ads.

3) Для этого испытания я закомментировал строчку `$this->updateOne($ad);`. В итоге count базы и цикла совпали.
We have 4000 ads with BYR.
1000
2000
3000
4000
Done updating BYR to BYN. Total: 4000 ads.

4) Откомментировал назад — снова половина выдачи.
We have 4000 ads with BYR.
1000
2000
Done updating BYR to BYN. Total: 2000 ads.
*/

Решение:

do {
    $ads = $query->limit(1000)->get();
    foreach ($ads as $ad) {
        $this->updateOne($ad);
        $this->count++;
    }
    $this->line($this->count);
} while(! empty($ads));