Fix incompatibility with --classmap-authoritative

[enumag]

I tried to replace --optimize-autoloader with --classmap-authoritative for my CI and got this exception:

  Fatal error: Class 'PackageVersions\Versions' not found 

It seems that the Versions class was generated but is missing in the composer autoloading classmap because when I added the file manually in composer.json it works fine:

    "autoload": {
        //...
        "files": [
            "vendor/ocramius/package-versions/src/PackageVersions/Versions.php"
        ]
    },

[Ocramius]

Any test case/scenario that can be used to automate this failure and verify its correctness?

[enumag]

@Ocramius I tried to add --classmap-authoritative to .travis.install.sh (and removed the fix). It caused the build to fail with:

1) PackageVersionsTest\VersionsTest::testValidVersions
Error: Class 'PackageVersions\Versions' not found

Which is the error we wanted to test. So I'll just commit the fix along with the .travis.installer.sh change and that should be it.

[Ocramius]

The test should be an additional one, not a replacement for current installation...

[enumag]

I don't think it's necessary in this case (if something works with --classmap-authoritative I'm sure it will work without it as well). But alright, have it your way. It has to be a separate travis build though.

[Ocramius]

I don't think it's necessary

Well, people usually run it without --classmap-authoritative, so we have to test with a separate flag for this particular scenario :-)

Anyway, I will have to take a closer look at this patch, since I might be able to also fix #18 in a similar way.

[Ocramius]

Manually rebased and merged, thanks!

[stof]

using files is quite bad, because this forces loading the file all the time instead of using autoloading

[stof]

btw, due to the change done in #33, it is useless, as the classmap generator will contain the class due to the fallback class

[Ocramius]

Yeah, can be reverted...

[Ocramius]

86f2636

[stof]

The proper way to ensure the class made it in the classmap (forgetting #33 which creates the file all the time) would have been to generate the file on the ScriptEvents::PRE_AUTOLOAD_DUMP instead

[itscaro]

Hello,

With 1.5.1 and --classmap-authoritative, I encounter this issue, is there any fix? Thank you

[Ocramius]

@itscaro please write a test case reproducing the failure first