HomeWildfire Games

[gameplay-a24] differentiate African and Indian war elephants because of their…

Description

[gameplay-a24] differentiate African and Indian war elephants because of their sizes.

maur, pers, sele champion elephants +10% resource costs, training time, health, and attack damage;
cart, kush, ptol champion elephants −10% resource costs, training time, health, and attack damage.

Patch by: @Nescio
Accepted by: @Feldfeld, @borg-

Differential Revision: https://code.wildfiregames.com/D2861

Event Timeline

minohaka raised a concern with this commit.Jul 5 2020, 8:23 PM
minohaka added a subscriber: minohaka.

Was playing the Mauryas an got these (interestinglog.html):

ERROR: RelaxNGValidator: Validation error: (null):0: Extra element food in interleave

ERROR: RelaxNGValidator: Validation error: units/maur_champion_elephant:1: Element Resources failed to validate content

ERROR: RelaxNGValidator: Validation error: (null):0: Extra element food in interleave

ERROR: RelaxNGValidator: Validation error: units/maur_champion_elephant:1: Element Loot failed to validate content

ERROR: RelaxNGValidator: Validation failed for '(null)'

ERROR: Failed to validate entity template 'units/maur_champion_elephant'

ERROR: JavaScript error: gui/session/session.js line 222 Error: deepfreeze requires exactly one object as an argument. GetTemplateData@gui/session/session.js:222:1 g_SelectionPanels.Training.setupButton@gui/session/selection_panels.js:933:18 setupUnitPanel@gui/session/unit_commands.js:95:8 updateUnitCommands@gui/session/unit_commands.js:147:4 updateSelectionDetails@gui/session/selection_details.js:499:2 updateGUIObjects@gui/session/session.js:663:2 onTick@gui/session/session.js:598:3 __eventhandler50 (Tick)@session Tick:0:1

ERROR: Errors executing script event "Tick"

ERROR: JavaScript error: gui/session/session.js line 222 Error: deepfreeze requires exactly one object as an argument. GetTemplateData@gui/session/session.js:222:1 g_SelectionPanels.Training.setupButton@gui/session/selection_panels.js:933:18 setupUnitPanel@gui/session/unit_commands.js:95:8 updateUnitCommands@gui/session/unit_commands.js:147:4 updateSelectionDetails@gui/session/selection_details.js:499:2 updateGUIObjects@gui/session/session.js:663:2 onSimulationUpdate@gui/session/session.js:634:2 __eventhandler52 (SimulationUpdate)@session SimulationUpdate:0:1

ERROR: Errors executing script event "SimulationUpdate"
This commit now has outstanding concerns.Jul 5 2020, 8:23 PM
Stan added a comment.Jul 5 2020, 8:27 PM

Thanks for the report!

Stan added a comment.Jul 5 2020, 8:42 PM

Looks like I need mul_round instead of mul

Stan requested verification of this commit.Jul 9 2020, 11:24 AM
This commit now requires verification by auditors.Jul 9 2020, 11:24 AM
Stan removed an auditor: minohaka.Dec 5 2020, 5:26 PM
This commit no longer requires audit.Dec 5 2020, 5:26 PM