Another thing I noticed, the function returns a zip://… url which requires the zip stream wrapper to be loaded. This was always the case, so I guess that should generally not be a problem. We could however get rid of that dependency by using ZipArchive::getStream and/or simply return the file content from that method. Or does anyone see a point in returning a path from that method?

Sounds fine to me if you can simplify things a bit that way. But I guess if the ext is present the stream wrapper will be as well so it probably doesn't matter a lot in practice.

What's the status here? I think it was ready to go but it's still marked WIP so not sure :)

Sorry, you're right. I just checked AppVeyor but the failing test looks unrelated to this PR.

Thanks