Opened 9 years ago

Closed 4 years ago

Last modified 4 years ago

#3274 closed defect (fixed)

Order is replaced if resource-carrying men in formation and women are selected

Reported by: elexis Owned by: wraitii
Priority: Should Have Milestone: Alpha 24
Component: Simulation Keywords:
Cc: Patch: Phab:D2702

Description (last modified by Freagarach)

  1. Have some men with formation, and some women (f.e. starting units)
  2. Gather some resources with all of them (or only some men)
  3. Stop the units (hotkey H)
  4. Task them to gather a resource and shift click the building of a barracks

Expected result: All units gather stone and then build the barracks. Actual result: The units in formation immediately go to the barracks immediately, women gather.

It also seems to be necessary that the selection contains women. If you use men without formation, the bug doesnt occur.

Originally reported by ffm, even before a18 was released.

Change History (5)

comment:1 by Imarok, 5 years ago

Component: UI & SimulationSimulation

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

comment:2 by Freagarach, 4 years ago

Description: modified (diff)

Also reproducible by having the formation and women already within the range (10 m) of the resource when first giving the gather order:

  1. Have some men with formation and some women.
  2. Stand near a resource with them.
  3. Task them to gather resources.
  4. Queue an order to build a structure.

comment:3 by Freagarach, 4 years ago

Milestone: BacklogWork In Progress
Patch: Phab:D2702

comment:4 by wraitii, 4 years ago

Owner: set to wraitii
Resolution: fixed
Status: newclosed

In 23744:

Fix formations not queueing orders properly.

Formation controllers were not always waiting on their members correctly because the "finishedOrder" check was being reset in the wrong place. This happened particularly with queued orders.

This merges the "finishedOrder" and the "InPosition" logic.

Reported by: elexis

Based on a patch by: Freagarach

Tested by: Freagarach

Reviewed By: Angen

Fixes #3274

Differential Revision: https://code.wildfiregames.com/D2702

comment:5 by wraitii, 4 years ago

Milestone: Work In ProgressAlpha 24
Note: See TracTickets for help on using tickets.