Page MenuHomeWildfire Games

Fix attack/armor techs not applying after D2092/rP22754.
ClosedPublic

Authored by wraitii on Aug 25 2019, 7:50 AM.

Details

Summary

After D2092/rP22754, attack/armor techs were not properly applied in the simulation. It was found that two lines were incorrect and those are fixed in this diff.

Test Plan

Verify its correctness.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/484/display/redirect

wraitii retitled this revision from Fix techs not applying after D2092/rP22754. to Fix attack/armor techs not applying after D2092/rP22754..Aug 25 2019, 8:03 AM
wraitii edited the summary of this revision. (Show Details)
wraitii commandeered this revision.Aug 25 2019, 7:24 PM
wraitii edited reviewers, added: Freagarach; removed: wraitii.

Cheers, I'll commit something very slightly modified.

wraitii updated this revision to Diff 9507.Aug 25 2019, 7:25 PM
wraitii added a subscriber: Nescio.

Also fix capture techs/auras as reported by @Nescio

Freagarach accepted this revision.Aug 25 2019, 7:27 PM
This revision is now accepted and ready to land.Aug 25 2019, 7:27 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/490/display/redirect

Thanks for committing this!