Project

General

Profile

Bug #693380

Ship-to-ship cargo transfer inconsistent in request_transport() aware clients

Added by Jacob Nevins 6 months ago. Updated about 8 hours ago.

Status:
Closed
Priority:
Normal
Category:
Server
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

In the attached savegame, with S2_6, if I move [l tgt="unit" id=199 name="Transport" /] loaded with Spies out to sea next to other transports, I see the following behaviour:
  • In Gtk3:
    • Attempting to move a Spy onto another transport with arrow keys doesn't work.
      • If the destination tile has more than one transport, I get prompted to choose one of them, but then nothing happens.
      • If the destination tile only has one transport, nothing happens.
    • Moving a Spy onto another transport with [G]oto works (but doesn't prompt me to choose a transport).
  • Qt is similar.
  • In Gtk2 (which is blissfully unaware of transport choice), any of these actions does end up with a Spy on the destination transport (of course I've no choice of which).

However, moving a Spy from shore to ship with arrow keys does work in all clients (prompting or no).

I can't remember whether we ever intended to prevent ship-to-ship transfers; right now they are clearly possible.

(For the record: I can't see a way to move units between transports on the same tile; checking back, we never got around to that; any further discussion/work on that should go in a new ticket resurrecting gna patch #4805 .)

26_multiload.sav.bz2 (165 KB) 26_multiload.sav.bz2 S2_6 example savegame Jacob Nevins, 2017-08-27 12:38 PM
0027-Unload-unit-from-current-transport-when-it-tries-to-.patch (949 Bytes) 0027-Unload-unit-from-current-transport-when-it-tries-to-.patch Marko Lindqvist, 2018-02-04 05:24 PM

History

#1 Updated by Marko Lindqvist 15 days ago

#2 Updated by Marko Lindqvist 8 days ago

Jacob Nevins wrote:

(For the record: I can't see a way to move units between transports on the same tile; checking back, we never got around to that; any further discussion/work on that should go in a new ticket resurrecting gna patch #4805 .)

-> Feature #732489

#3 Updated by Marko Lindqvist about 8 hours ago

  • Status changed from Resolved to Closed
  • Assignee set to Marko Lindqvist

Also available in: Atom PDF