Opened 6 years ago

Last modified 20 months ago

#4951 new defect

Long range capturing

Reported by: temple Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch: Phab:D2709

Description

If you attack as a formation with a formation that can attack as formation (currently phalanx, skirmish, syntagma, testudo, or wedge), and then while attacking switch to capturing, units will capture from where they stand rather than first moving to the proper range.

In the replay I show how the bug happens with the wedge formation but not close order.

Attachments (1)

commands.txt (12.6 KB ) - added by temple 6 years ago.

Download all attachments as: .zip

Change History (9)

by temple, 6 years ago

Attachment: commands.txt added

comment:1 by temple, 6 years ago

For the moment it might be best to disable attacking as formation, Phab:D1220.

comment:2 by temple, 6 years ago

Formation attack uses the formation controller's position to determine if units are within range, and that's not necessarily correct. For example, a unit could be trapped between buildings but if it's still considered part of a formation it can attack with them, even if they're on the other side of the map. It also doesn't take into account the attack type so it'll return the same range whether they're attacking or capturing. (The code hasn't been updated in four years.)

comment:3 by temple, 6 years ago

In 21042:

Disable attack as formation

Differential Revision: https://code.wildfiregames.com/D1220
Reviewed by: wraitii
Refs: #4951

comment:4 by Imarok, 5 years ago

Component: UI & SimulationSimulation

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

comment:5 by Silier, 4 years ago

Last edited 4 years ago by Silier (previous) (diff)

comment:6 by Silier, 4 years ago

Milestone: BacklogWork In Progress
Owner: set to Silier
Patch: Phab:D2709

comment:7 by Freagarach, 3 years ago

Since r25235, they should only be able to attack once from too far away, since cmpAttack now checks for the range when actually attacking.

comment:8 by Stan, 20 months ago

Milestone: Work In ProgressBacklog
Owner: Silier removed

Backlogging due to lack of progress and Silier retiring.

Note: See TracTickets for help on using tickets.