Opened 3 years ago
Closed 3 years ago
#6046 closed defect (fixed)
order queue not flushed
Reported by: | snelius | Owned by: | Freagarach |
---|---|---|---|
Priority: | Must Have | Milestone: | Alpha 25 |
Component: | Simulation | Keywords: | UnitAI |
Cc: | snelius | Patch: | Phab:D3587 |
Description
test scenario:
do A
then unselect unit
then select same unit again
do B (give another order queue)
A) give unit orders in a queue to
- build a barrack (no SHIFT pressed)
- build a house (with SHIFT button pressed => add next step to new queue "A")
- go to gather berries (with SHIFT button pressed => next step to A queue)
B) give unit orders in a queue to
- build a barrack (no SHIFT pressed => queue A should be flushed like in a23)
In this step simply right click on a barrack which is actually being build without SHIFT button pressed. - build a house (with SHIFT button pressed => add next step to new queue "B")
- go cut wood (with SHIFT button pressed => next step to B queue)
Expected result: after building house unit should gather wood BUT will gather berries, i.e. queue A is remembered but should have been erased in step B.1 - this step should overwrite or flush old queue A.
Please see attached replay from a24 revision 24933
Attachments (1)
Change History (6)
by , 3 years ago
Attachment: | 2021-02-20_order queue not flushed.zip added |
---|
comment:1 by , 3 years ago
comment:2 by , 3 years ago
Component: | Core engine → Simulation |
---|---|
Keywords: | UnitAI added; order queue removed |
Milestone: | Backlog → Work In Progress |
Patch: | → Phab:D3587 |
Priority: | Release Blocker → Must Have |
comment:4 by , 3 years ago
Milestone: | Work In Progress → Alpha 25 |
---|
Decided not to fix for A24 as the issue should not really affect many people, and there's an easy "manual" fix.
Note:
See TracTickets
for help on using tickets.
In r24855, we don't readd an order given what is essentially the same order, to prevent entities not doing stuff when ordered repeatedly.
We didn't take into account to flush the rest of the queue.