Errors when installing package that defines multiple Paths for a PSR-4 Autoload

[Mezzle]

In the file

https://github.com/zendframework/zend-component-installer/blob/master/src/ComponentInstaller.php#L256

This assumes that each PSR-4 Autoload item mentioned in the composer config is a singular item - so they are defined as such:-

{
    "autoload": {
        "psr-4": {
            "Monolog\\": "src/",
            "Vendor\\Namespace\\": ""
        }
    }
}

However, the Composer configuration allows that the path for the code be able to be both a string, and an array - e.g.

{
    "autoload": {
        "psr-4": { "Monolog\\": ["src/", "lib/"] }
    }
}

When defined with an array of paths, and a config provider, this causes the code to error out and the installation to fail.

Whilst this can be fixed in the package (e.g. Mezzle/queuejitsu@17f17a7 ) - this isn't always going to be possible, and should allow for any valid composer configuration to work.

[Ocramius]

Hah, I didn't even know that multiple paths were valid there. An (array) cast on each psr-4 entry should deal with this.

[Mezzle]

Yeah, it's pretty much changing

foreach ($map as $namespace => $path) {

to something like

foreach ($map as $namespace => $paths) {
    $paths = (array)$paths;
    foreach ($paths as $path) {

[weierophinney]

@Mezzle Would you be willing to create a pull request for this change?

[Mezzle]

@weierophinney converted.

[michalbundyra]

@Mezzle @weierophinney I think it will be nice to have some tests for it as well...

[Mezzle]

I've not the time to write them at the moment - don't know when I'll be able to get round to writing some - so if in the meantime anyone wants to - feel free!

[weierophinney]

Thanks, @Mezzle. I added tests and did some refactoring, which I pushed as #44. The fix will release with 1.1.0.