Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1716 closed defect (fixed)

[PATCH] Some queue orders for citizen-soldiers still need implemented

Reported by: michael Owned by: leper
Priority: Must Have Milestone: Alpha 12
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

The queue (shift-click) order for citizen-soldiers to drop resources, then go build something does not work. The soldier will skip the drop off part once the 2nd order is made, as if the player never queued the orders to begin with.

Attachments (2)

formation-order-queues.patch (10.7 KB ) - added by leper 11 years ago.
Proper formation order queueing
formation-order-queues_formation-atack.patch (20.2 KB ) - added by Simon 11 years ago.
Adds a fix for #1624 based on the prior submitted patch by leper

Download all attachments as: .zip

Change History (7)

comment:1 by leper, 11 years ago

Some orders (gather-near-position (used by rallypoints), return-resource, attack, heal and garrison) still use the old behaviour. They issue the command to all formation members and the formation disbands, if you add another order a new formation is created and thus the previous tasks (for every unit) are discarded.

The problem is that you can't wait for every unit to finish the task and just queue the new "get-in-formation" order as that isn't what one would expect.

Completely migrating to the new behaviour (keeping the formation intact, while each unit carries out the order on their own) has the problem that defining when a task is finished isn't obvious.

See #592, #593 and #1670.

by leper, 11 years ago

Proper formation order queueing

comment:2 by leper, 11 years ago

Keywords: patch review added
Summary: Some queue orders for citizen-soldiers still need implemented[PATCH] Some queue orders for citizen-soldiers still need implemented

The attached patch should fix #592, #593 and this ticket. #1670 should be done like Deiz outlined in that ticket.

Some testing/review would be appreciated.

comment:3 by leper, 11 years ago

The special handling for building can be removed too.

by Simon, 11 years ago

Adds a fix for #1624 based on the prior submitted patch by leper

comment:4 by leper, 11 years ago

Owner: set to leper
Resolution: fixed
Status: newclosed

In 12911:

Formation order queueing. Fixes #592, #593 and #1716.
Allow changing the formation, while under attack. Fixes #1624. Patch by Simikolon.

comment:5 by leper, 11 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.