Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3887 closed defect (fixed)

[PATCH] Put auras back into structree tooltips.

Reported by: s0600204 Owned by: s0600204
Priority: Should Have Milestone: Alpha 21
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

With the recent relocation of auras from being embedded in XML to separate JSON files, said auras have stopped showing up in the structree.

The attached patch resolves that.

Attachments (3)

put_auras_back_in_structree.patch (2.9 KB ) - added by s0600204 8 years ago.
putAurasBack_v2.patch (3.3 KB ) - added by s0600204 8 years ago.
Comment corrected, patch taken from more sensible root
putAurasBack_v3.patch (3.3 KB ) - added by s0600204 8 years ago.
Yes, I'm fairly sure I did. However... fixed.

Download all attachments as: .zip

Change History (8)

by s0600204, 8 years ago

by s0600204, 8 years ago

Attachment: putAurasBack_v2.patch added

Comment corrected, patch taken from more sensible root

comment:1 by sanderd17, 8 years ago

Keywords: review removed

Did you test your patch? I get the following error:

ERROR: JavaScript error: gui/structree/helper.js line 13
ReferenceError: template is not defined
  loadTemplate@gui/structree/helper.js:13:1
  loadUnit@gui/structree/load.js:42:6
  selectCiv@gui/structree/structree.js:81:5
  __eventhandler38 (selectionchange)@civSelection selectionchange:0:1
  init@gui/structree/structree.js:38:3
  __eventhandler8 (press)@menuStrucTreeButton press:2:8
ERROR: GUI page 'page_structree.xml': Failed to call init() function

by s0600204, 8 years ago

Attachment: putAurasBack_v3.patch added

Yes, I'm fairly sure I did. However... fixed.

comment:2 by s0600204, 8 years ago

Keywords: review added

comment:3 by sanderd17, 8 years ago

Resolution: fixed
Status: newclosed

In 18010:

Add auras back to structure tree. Patch by bb. Fixes #3887

comment:4 by sanderd17, 8 years ago

Keywords: review removed

comment:5 by sanderd17, 8 years ago

s0600204, sorry for he wrong commit message ... ofc you made the patch.

Note: See TracTickets for help on using tickets.