Installer fails with platform extensions

[stevenrombauts]

The installer will throw an exception when trying to install platform-xyz extensions into platform along with the default dependencies on a first composer install during installation of custom projects. This happens because the dependencies required to bootstrap the application (such as phptdotenv) are missing. We need to make sure that:

• the required dependencies are installed first
• the autoloader is present before we bootstrap the application

Originally reported at joomlatools/joomlatools-platform#240

Todo's:

• [x] Install/update after autoload is generated
• [x] Check if installed before queuing
• [x] Separate queue from ExtensionInstaller class
• [x] Fix uninstall?

[stevenrombauts]

@raeldc If you have 2 minutes to spare, could you test this? I've created a gist that sets everything up for you, all you need to do is verify that the installation works on your Joomlatools Box:

The following sets up Joomlatools Platform develop, adds extra components to composer.json and then installs it all in one go. Run this command:

curl -s https://gist.githubusercontent.com/stevenrombauts/fefee1e50135d0c766fd/raw/f0595859b423b54313397f388c993d760ec0e7e8/test-composer.sh | bash

Login at http://joomla.box/composertest/web/administrator and check if com_content, com_media and com_finder have been installed (and are working)

Thanks!

[raeldc]

@stevenrombauts,

I'm having this error when running the script.

[stevenrombauts]

Thanks @raeldc for testing! Looks like this constant has only recently been added to Composer in 49d7d659. Not many people will have such a recent version so I went back to relying on the Symfony Console constants.

Could you give that test command above another run please?

[raeldc]

@stevenrombauts, it's working perfectly now. Awesome!

[johanjanssens]

@stevenrombauts @raeldc This issue is still marked as test. Did we release this? If so where, if not where should this go?

[stevenrombauts]

@johanjanssens We still need to release this. This should be working but to make it work properly in platform we also need to fix #23. But this should be mergeable.

[johanjanssens]

@stevenrombauts Can you check with @ercanozkaya This code was committed in develop and it seems it has been released with 1.1.0. Trying to clean this up a bit so the state of the repo reflects the work done, or things will get blurry after a while.

[ercanozkaya]

This was released as part of v1.1.0