Opened 16 months ago
Closed 15 months ago
#6727 closed defect (fixed)
Rally point relics
Reported by: | Langbart | Owned by: | Freagarach |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 27 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: | Phab:D4933 |
Description
to reproduce
- start a map with some relics
# command line pyrogenesis -autostart-size=128 -autostart="random/mainland" -autostart-victory=capture_the_relic -autostart-reliccount=14
- train a horse in your
CivicCenter
- set the rally point from the
CivicCenter
to a relic - notice the horse will not move to the relic after it has been trained
bisect
Attachments (2)
Change History (6)
by , 16 months ago
by , 16 months ago
Attachment: | broken.gif added |
---|
comment:1 by , 16 months ago
comment:2 by , 16 months ago
possible solution ?
-
binaries/data/mods/public/simulation/components/UnitAI.js
a b UnitAI.prototype.Attack = function(target, allowCapture = this.DEFAULT_CAPTURE, 5502 5502 this.WalkToTarget(target, queued, pushFront); 5503 5503 return; 5504 5504 } 5505 let cmpHealth = Engine.QueryInterface(target, IID_Health); 5506 if(!cmpHealth) 5507 allowCapture = true 5505 5508 let order = { 5506 5509 "target": target, 5507 5510 "force": true,
Version 0, edited 16 months ago by (next)
comment:3 by , 16 months ago
Owner: | set to |
---|---|
Patch: | → Phab:D4933 |
Note:
See TracTickets
for help on using tickets.
Hmm, this one is annoying. Previously it was allowed to capture, so it did. Now the default is to not capture. Since the catafalque has no cmpHealth, the entity cannot attack and will therefore not walk towards it.
I'll have to think about a fix (maybe some check in
unit_actions.js:set-rally-point
).