Replaced Box/Spout by OpenSpout (fix #206).

[Daniel-KM]

No description provided.

[zerocrates]

Have you tried this out?

I'm getting

OpenSpout\Reader\ODS\RowIterator::current(): Return value must be of type OpenSpout\Common\Entity\Row, null returned

... it looks like a bug that happens when calling current() right when starting the iterator... When they introduced typehinting on their returns, they no longer allow current() to return null, which it will do before any calls to next() or rewind().

What it wants to happen is a single call to rewind before using current (as PHP does when it starts foreach-ing an iterator). Unfortunately If we just rewind() right when we load the iterator, then any later foreach we do will hit an error because Spout/Openspout's iterator only allows a single rewind.

You already did some workaround-ing of this in the current code: getRows uses the direct iterator calls rather than foreach, and just does an extra next call at the beginning to skip past the initial null return but not have to call rewind.

Probably there's ultimately a solution where we just call our reset and get a fresh Iterator again even more than we already do, and then we can just use foreach everywhere.

[zerocrates]

I've made changes to account for this problem (which also revealed a minor latent problem with the existing code for the ODS source).