Switeck asked me to post here as well the answer I PM'd him. It elaborates on what cim wrote. I quote only the relevant part:
If you want to manipulate oolite-constrictor-hunt-mission.js, you have to do it from the outside, not by putting an identically named script in your OXP. In other words: monkeypatching is what is asked for, only that you don't want to
amend the event handlers, you want to
replace them completely. And you only need to do that for those handlers that you want to actually change.
So, rename your script to something unique (both file name and name tag), and re-organize it thusly (for easiness):
(1) delete all functions and event handlers that are not different from the originals. You don't need to bother with them.
(2) rename the remaining functions and event handlers uniquely. They are not expected to actually run in your script. For instance,
guiScreenChanged should become
this._constrictorScriptReplaceGuiScreenChanged, and equally for all other functions.
(2) Create a new function
this.startUp. It's going to be the only event handler of your script that actually runs. It should look like this:
Code: Select all
this.startUp = function()
{
worldScripts["oolite-constrictor-hunt"].guiScreenChanged = this._constrictorScriptReplaceGuiScreenChanged;
worldScripts["oolite-constrictor-hunt"].missionScreenOpportunity = this._constrictorScriptReplaceMissionScreenOpportunity;
worldScripts["oolite-constrictor-hunt"].shipExitedWitchspace = this._constrictorScriptReplaceShipExitedWitchspace;
worldScripts["oolite-constrictor-hunt"].shipLaunchedFromStation = this._constrictorScriptReplaceShipLaunchedFromStation;
}
Those are the four handlers you actually need to replace.
(@ Switeck: If it's guaranteed that the script inside Oolite is loaded first—which cim confirms—, you don't need to bother with a timer. So forget that part of my PM.)
![[EliteWiki]](/images/elitewikismall.png){kind=small}
New Cargoes