Opened 11 years ago

Closed 11 years ago

#1754 closed defect (fixed)

Aegis Bot JavaScript warning when slaughtering AI-controlled animals.

Reported by: michael Owned by: wraitii
Priority: Must Have Milestone: Alpha 13
Component: UI & Simulation Keywords:
Cc: wraitii Patch:

Description (last modified by michael)

Happens on Median Oasis 4 and maybe Deccan Plateau (both newer maps).

WARNING: JavaScript warning: simulation/ai/common-api-v2/entity.js line 279 reference to undefined property this._entity.hitpoints

WARNING: JavaScript warning: simulation/ai/common-api-v2/entity.js line 62 reference to undefined property this._template.Health

ERROR: JavaScript error: simulation/ai/common-api-v2/entity.js line 62 TypeError: this._template.Health is undefined ()@simulation/ai/common-api-v2/entity.js:62 ()@simulation/ai/common-api-v2/entity.js:280 ()@simulation/ai/common-api-v2/entity.js:282 ([object Object],"1923",[object Object])@simulation/ai/qbot-wc/economy.js:238 ([object Object])@simulation/ai/common-api-v2/entitycollection.js:79 ([object Object])@simulation/ai/qbot-wc/economy.js:238 ([object Object],[object Object],[object Array])@simulation/ai/qbot-wc/economy.js:754 ()@simulation/ai/qbot-wc/qbot.js:132 ([object Object])@simulation/ai/common-api-v2/base.js:147 @:0

Attachments (2)

mainlog.html​ (69.2 KB ) - added by michael 11 years ago.
interestinglog.html​ (37.6 KB ) - added by michael 11 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 by michael, 11 years ago

Description: modified (diff)

comment:2 by historic_bruno, 11 years ago

Cc: wraitii added

by michael, 11 years ago

Attachment: mainlog.html​ added

by michael, 11 years ago

Attachment: interestinglog.html​ added

comment:3 by michael, 11 years ago

Description: modified (diff)

comment:4 by michael, 11 years ago

Possible problem could be with killed sheep belonging to the player?

comment:5 by Kieran P, 11 years ago

Milestone: Backlog → Alpha 12
Priority: Should Have → Release Blocker

comment:6 by historic_bruno, 11 years ago

Milestone: Alpha 12 → Alpha 13
Priority: Release Blocker → Must Have

This is worked around now, which is good enough for A12.

comment:7 by wraitii, 11 years ago

FYI, this is fixed in the API v3.

in reply to:  7 comment:8 by michael, 11 years ago

Replying to wraitii:

FYI, this is fixed in the API v3.

Will API v3 be committed soon?

comment:9 by michael, 11 years ago

Summary: Aegis Bot JavaScript warning on Median Oasis 4. → Aegis Bot JavaScript warning when slaughtering AI-controlled animals.

in reply to:  7 comment:10 by leper, 11 years ago

Replying to wraitii:

FYI, this is fixed in the API v3.

Could you backport the (probably small) fix to v2?

comment:11 by leper, 11 years ago

In 13005:

Work around the sheep bug on Acropolis 7. Refs #1754, #1793.

comment:12 by wraitii, 11 years ago

Committing the API 3 is basically only a matter of reviewing. Quantumstate started, I dunno where he got.

Leper, I'll try tomorrow, it's indeed a simple fix.

comment:13 by wraitii, 11 years ago

Owner: set to wraitii
Resolution: → fixed
Status: new → closed

In 13012:

Backport from API v3 to API v2 that should fix a bug happening with sheeps. fixes #1754

Note: See TracTickets for help on using tickets.