Update Installer Script (run postInstall before shortcut creation)

[Zemogiter]

Description

Thanks to #1144 we have discovered that scripts will fail if .executable() points to a file that only exist after postInstall step is done. Since @plata mentioned something about Installer Script working this way for a reason (though I was unable to find that reason in closed PRs and issues) this is a WIP draft untill me or someone else test this with other scripts that use postInstall.

What works

Heroes of Might & Magic IV no longer crash due to non-existing .executable()

What was not tested

Other scripts with postInstall

Test

• Operating system (and linux kernel version): Ubuntu 19.10
• Hardware (GPU/CPU): GTX 1080 ti, i7-7700K

Ready for review

• Script tested as a regular phoenicis user and working (if you have a problem -> create as draft and ask for help).
• json-align and eslint run according to the documentation.
• Codacy and travis checked.

[plata]

@qparis can you remember if there was a reason to use this order?

[madoar]

I don't see a reason why we should not be able to change the order. We should only face an issue with this if inside postInstall the created shortcut is accessed or the internal values of QuickScript are changed, which should not be the case anyway.

[plata]

For me it's fine to try it. Could you take a look at the postInstall calls to see if there's something which might get broken?

[madoar]

This is interesting. It seems like some installers use a different order:

scripts/Engines/Wine/QuickScript/Origin Script/script.js

Lines 48 to 65 in 5fe4e1f

	setupWizard.message(tr("Download \"{0}\" in Origin and shut it down once \"{0}\" is installed", this._name));
	wine.run(tempFile, [], null, false, true);
	// wait until Origin and Wine are closed
	wine.wait();
	// Origin installation has finished
	setupWizard.wait(tr("Please wait..."));
	this._preInstall(wine, setupWizard);
	// back to generic wait (might have been changed in preInstall)
	setupWizard.wait(tr("Please wait..."));
	this._postInstall(wine, setupWizard);
	// create shortcut after installation (if executable is specified, it does not exist earlier)
	this._createShortcut(wine.prefix());

[madoar]

ZipScript as well

[plata]

Ok, then let's merge this.