Opened 5 years ago

Last modified 3 years ago

#5328 new defect

Siege engines can't cancel unpacking

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

See https://wildfiregames.com/forum/index.php?/topic/24979-catapults-cant-move-in-standground-mode/ for a GIF and the original report.

Reproduce:

  1. Start a singleplayer match with 2 players with siege engine civs (athen, ptol, seleucid, mace...) on a tiny map 
  2. Type "gift from the gods" cheat
  3. Build the building to train the siege engine and train a siege engine
  4. Move the siege engine to neutral territory. The siege engine is still unpacked and still in the default standground stance.
  5. Type "gift from the gods" cheat (so that packing isn't instant anymore)
  6. Press Alt+D and switch perspective to player 2
  7. Type "gift from the gods" cheat
  8. Train some attackers, and let them attack the siege engine
  9. Switch perspective back to player 1 
  10. Observe that the packed siege engine starts unpacking on it's own as it was attacked by an enemy.
  11. You want the siege engine to retreat intead of continue to unpack and attack. Therefore you click on "cancel unpacking".

Expected behavior: Clicking on "Cancel unpack" stops the unpack process and allows the still packed siege engine to process move orders. 

Current behavior: Clicking on "Cancel unpack" resets the the unpacking progress bar to zero, but the siege engine starts the unpack progress again on its own.

The user has to wait to wait one full unpack cycle and then one full pack cycle to move the siege engine that was packed already. 

Issue was introduced in this line https://code.wildfiregames.com/rP21630#inline-2523

It is said that setting the Passive stance avoids the problem.

Attachments (1)

commands.txt (17.6 KB ) - added by elexis 5 years ago.

Download all attachments as: .zip

Change History (12)

by elexis, 5 years ago

Attachment: commands.txt added

comment:1 by elexis, 5 years ago

Description: modified (diff)

comment:2 by elexis, 5 years ago

Description: modified (diff)

comment:3 by elexis, 5 years ago

Description: modified (diff)

comment:4 by elexis, 5 years ago

dupe at #5175

comment:5 by elexis, 5 years ago

Milestone: BacklogAlpha 24

comment:6 by Imarok, 5 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

comment:7 by Silier, 4 years ago

Owner: set to Silier
Resolution: fixed
Status: newclosed

In 23298:

Automatically cancel packing or unpacking based on given order

Cancel packing when next command requires unpacked state and cancel unpacking when next command requires packed state.

Fixing unit refusing to move because keeps unpacking to attack targets in range.

Partially Fixes: #4015, #5328
Differential Revision: https://code.wildfiregames.com/D1520
Patch by: @causative
Comments by: elexis, Freagarach, Stan

comment:8 by Silier, 4 years ago

Resolution: fixed
Status: closedreopened

Cancel pack has still this behaviour, but it is not more required to move unit.

Last edited 4 years ago by Silier (previous) (diff)

comment:9 by Silier, 4 years ago

Owner: Silier removed
Status: reopenednew

comment:10 by wraitii, 3 years ago

Milestone: Alpha 24Alpha 25

(not entirely sure what the status of this is)

comment:11 by wraitii, 3 years ago

Milestone: Alpha 25Backlog

Backlogging for inactivity

Note: See TracTickets for help on using tickets.