Opened 6 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 )
See https://wildfiregames.com/forum/index.php?/topic/24979-catapults-cant-move-in-standground-mode/ for a GIF and the original report.
Reproduce:
- Start a singleplayer match with 2 players with siege engine civs (athen, ptol, seleucid, mace...) on a tiny map
- Type "gift from the gods" cheat
- Build the building to train the siege engine and train a siege engine
- Move the siege engine to neutral territory. The siege engine is still unpacked and still in the default standground stance.
- Type "gift from the gods" cheat (so that packing isn't instant anymore)
- Press Alt+D and switch perspective to player 2
- Type "gift from the gods" cheat
- Train some attackers, and let them attack the siege engine
- Switch perspective back to player 1
- Observe that the packed siege engine starts unpacking on it's own as it was attacked by an enemy.
- 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)
Change History (12)
by , 6 years ago
Attachment: | commands.txt added |
---|
comment:1 by , 6 years ago
Description: | modified (diff) |
---|
comment:2 by , 6 years ago
Description: | modified (diff) |
---|
comment:3 by , 6 years ago
Description: | modified (diff) |
---|
comment:4 by , 5 years ago
comment:5 by , 5 years ago
Milestone: | Backlog → Alpha 24 |
---|
comment:6 by , 5 years ago
Component: | UI & Simulation → Simulation |
---|
Move tickets to Simulation
as UI & Simulation
got some sub components.
comment:8 by , 4 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Cancel pack has still this behaviour, it is not more required to move unit.
comment:9 by , 4 years ago
Owner: | removed |
---|---|
Status: | reopened → new |
comment:10 by , 3 years ago
Milestone: | Alpha 24 → Alpha 25 |
---|
(not entirely sure what the status of this is)
dupe at #5175