Method prepareForValidation not work with onEachRow and SkipsEmptyRows interface

[NoubotElbot]

I'm validating the rows of a .csv, but when trying to normalize the data with the prepareForValidation method this doesn't work if SkipsEmptyRows is implemented. Without it it works perfectly.

My import class

class DebtorEmailImport implements
    OnEachRow,
    WithHeadingRow,
    WithValidation,
    SkipsEmptyRows,
    SkipsOnFailure,
    SkipsOnError,
    WithChunkReading,
    WithCustomCsvSettings,
    ShouldQueue,
    WithEvents
{
    use Importable, SkipsErrors, RegistersEventListeners;

    public function prepareForValidation($data, $index)
    {
        // nothing happens here, unless you don't use SkipsEmptyRows interface
        dump($data);
       // code ...
        return $data;
    }

    public function rules(): array
    {
        // my rules
        return $rules;
    }

    public function onRow(Row $row)
    {
       // here I process my row
    }
   # code ...
}

[patrickbrouwers]

Can you provide a failing tests for this via a PR?

[VFAndrew]

I too am encountering this issue. prepareForValidation is not called when using onEachRow and SkipsEmptyRows . If I remove SkipsEmptyRows then prepareForValidation is called.

[comicat-hu]

I think the issue has fixed it in #3573.