Generator not being run

[Superpat]

Hello, after trying to use prepareRows without success (see #4270) I tried switching to using Generators instead.

So I have an array that needs to be flattened and I pass it to the constructor which stores it as a private variable of the export class, then using FromGenerator I loop through the array and yield for each flattened item.

But for some reason my generator() function never runs ? I've got logs in the constructor and at the beginning of the generator function (before the loop starts) as well as in my map() function. But only the logging in the constructor is run.

[patrickbrouwers]

Is this an export or import? Can you show some code

[Superpat]

It's an export, this would be the simplest example (I had two exports, but both the complicated one and the simple one where not working)

<?php

namespace App\Exports;

use Generator;
use App\Models\MyModel;
use Maatwebsite\Excel\Concerns\FromGenerator;
use Maatwebsite\Excel\Concerns\WithHeadings;
use Maatwebsite\Excel\Concerns\WithMapping;
use Maatwebsite\Excel\Concerns\WithStrictNullComparison;
use Maatwebsite\Excel\Concerns\Exportable;

class MyModelExport implements FromGenerator, WithHeadings, WithMapping, WithStrictNullComparison
{
    use Exportable;

    private iterator $myModels;
 
    /**
    * @param iterator<MyModel>
    */
    public function __construct(iterator $myModels)
    {
        $this->myModels = $myModels;
    }

    /**
    * @return Generator<MyModel>
    */
    public function generator(): Generator
    {
        foreach ($this->myModels as $model) {
            yield $model;
        }
    }
}

[patrickbrouwers]

https://github.com/SpartnerNL/Laravel-Excel/blob/3.1/tests/Concerns/FromGeneratorTest.php

According to this test it should work. If you are able to add a failing unit test to that test class, it might help understand what goes wrong

[Superpat]

For the record, I got it working with collections instead, though I do need to run an extra flatmap around it, so generators where a bit nicer.