Page MenuHomeWildfire Games

constructed → ready (a field)
ClosedPublic

Authored by Stan on May 27 2018, 7:42 PM.

Details

Reviewers
bb
Gallaecio
Summary

Reported by kingu on Transifex.

Test Plan

It only affects a user-visible string.

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 6168
Build 10253: Vulcan BuildJenkins

Event Timeline

Gallaecio created this revision.May 27 2018, 7:42 PM
Vulcan added a subscriber: Vulcan.May 27 2018, 7:44 PM

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

Linter detected issues:
Executing section Default...
Executing section Source...
Executing section JS...
|    | [NORMAL] ESLintBear (curly):
|    | Unnecessary { after 'if' condition.
|----|    | /mnt/data/jenkins-phabricator/workspace/differential/binaries/data/mods/public/maps/tutorials/starting_economy_walkthrough.js
|    |++++| /mnt/data/jenkins-phabricator/workspace/differential/binaries/data/mods/public/maps/tutorials/starting_economy_walkthrough.js
| 430| 430| 		"OnPlayerCommand": function(msg)
| 431| 431| 		{
| 432| 432| 			if (msg.cmd.type == "gather" && msg.cmd.target)
| 433|    |-			{
|    | 433|+			
| 434| 434| 				if (TriggerHelper.GetResourceType(msg.cmd.target).generic == "stone")
| 435| 435| 					this.stone = true;
| 436| 436| 				else if (TriggerHelper.GetResourceType(msg.cmd.target).generic == "metal")
| 437| 437| 					this.metal = true;
| 438|    |-			}
|    | 438|+			
| 439| 439| 			if (this.IsDone())
| 440| 440| 				this.NextGoal();
| 441| 441| 		},

Link to build: https://jenkins.wildfiregames.com/job/differential/586/display/redirect

bb accepted this revision.Jun 1 2018, 10:58 PM
bb added a subscriber: bb.

Technically speaking of code and animations, the field is just as "constructed" as any other building, but probably sounds better otherwise

This revision is now accepted and ready to land.Jun 1 2018, 10:58 PM
Stan commandeered this revision.Dec 29 2018, 5:22 PM
Stan added a reviewer: Gallaecio.

Fixed by rP21982

Stan closed this revision.Dec 29 2018, 5:23 PM