Trade issue with 1.15

[bbeach]

There is a problem with the detach_from_masstraffic command in aiscripts/trade.performplayertraderun.xml:

<detach_from_masstraffic object="this.ship" wait="true" /> can hang and cause cargo ships not to function... changing all occurrences to:

<do_if value="this.ship.isclass.ship_xl or this.ship.isclass.ship_l">
<run_script name="'move.unpark'" />
</do_if>
<do_else>
<run_script name="'move.undock'" />
</do_else>

works.

Though if the ship is out of budget or something like that, you'll get the notification from the captain that he's waiting further instructions, but then it still won't work and will just loop back around and try to trade again.

Until there is a system in place for the player to interact and actually resolve issues, the default should be that the trade is canceled.

I'm going to test it out on the aiscripts/trade.performtraderun.xml and see if it improves ai trading also.

[EnemyWithin]

Could you put your fix in a small mod package?
Would really appreciate it.