units have different sound for order_repair and order_build at least in templates
issue is that i unit_actions.js is only command repair and so always is used and played order_repair even if target is just foundation and not damaged building
and unitAi does not differentiate these commands too
refs IRClogs yesterday and today, and lobby logs today.
The file contains the same code as currently committed, but uses the class syntax.
The Schema uses the getter syntax, i.e. performs the function if the property is referenced, i.e. .Schema as usual, not .Schema(). It works, has been uncharted territory for me too.
As reported on the lobby yesterday, another feature of the class keyword is that it throws an error if one calls a constructor without new:
TypeError: class constructors must be invoked with |new|
Class keyword discussion with Krinkle on:
Class keyword comments by smiley on lobby on September 1st and 6th
Class keyword discussion with Krinkle, nani, Chakakhan, bb, fpre:
Class keyword disussion with wraitii on:
GUI to be rewritten from procedural code to OOP code in #5387, hence the necessary decision.
To make it consistent the AI, simulation and rmgen would be considered as well.