Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#3093 closed enhancement (fixed)

[PATCH] Builders do not deposit after finishing building a dropsite

Reported by: Kalle Richter Owned by: maveric
Priority: Should Have Milestone: Alpha 19
Component: UI & Simulation Keywords:
Cc: Patch:

Description (last modified by leper)

Builders carrying some resource type that can be deposited at the constructed dropsite should deposit their resources.

Attachments (3)

3093.patch (969 bytes ) - added by maveric 9 years ago.
3093.2.patch (1.4 KB ) - added by maveric 9 years ago.
3093.3.patch (1.4 KB ) - added by maveric 9 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 by leper, 9 years ago

Component: Core engineUI & Simulation
Description: modified (diff)
Keywords: simple added
Summary: Builders don't unload their good after finishing building a storage buildingBuilders do not deposit after finishing building a dropsite

comment:2 by maveric, 9 years ago

Owner: set to maveric

comment:3 by maveric, 9 years ago

Status: newassigned

by maveric, 9 years ago

Attachment: 3093.patch added

comment:4 by maveric, 9 years ago

Keywords: patch review added
Summary: Builders do not deposit after finishing building a dropsite[PATCH] Builders do not deposit after finishing building a dropsite

comment:5 by sanderd17, 9 years ago

The patch does not work when you queue an order. Steps to reproduce:

  • Make sure your units are carrying some resources
  • Let them build a dropsite, and queue gathering a resource (not the nearest resource, so you clearly see the queued order gets executed).
  • When the units have build the dropsite, they'll walk towards the queued target, without first dropping the resources

I think you could use "PushOrderFront" instead of just "PushOrder", to bypass the queue. But testing this behaviour is needed.

Last edited 9 years ago by sanderd17 (previous) (diff)

comment:6 by maveric, 9 years ago

Thanks for reply. There is no point of usage PushOrderFront instead of PushOrder in 2690 line because order queue is empty anyway.

by maveric, 9 years ago

Attachment: 3093.2.patch added

comment:7 by maveric, 9 years ago

Now it works when there are queued orders.

by maveric, 9 years ago

Attachment: 3093.3.patch added

comment:8 by maveric, 9 years ago

Changed 'if' statement according to coding conventions.

comment:9 by sanderd17, 9 years ago

Resolution: fixed
Status: assignedclosed

In 16581:

Drop off resources when building dropsite. Patch by maveric. Fixes #3093

comment:10 by sanderd17, 9 years ago

Keywords: simple patch review removed
Milestone: BacklogAlpha 19
Note: See TracTickets for help on using tickets.