optimization and bug fix - sensors shouldn't interrogate the model if…

… not enabled, see #191
phetsims · Jan 24, 2019 · 99af46a · 99af46a
1 parent 8df95dc
commit 99af46a
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/js/common/model/TemperatureAndColorSensor.js b/js/common/model/TemperatureAndColorSensor.js
@@ -44,12 +44,17 @@ define( function( require ) {
 
     /**
      * @public
-     * @param {number} dt
      */
-    step: function( dt ) {
-      var temperatureAndColor = this.model.getTemperatureAndColorAtLocation( this.positionProperty.value );
-      this.sensedTemperatureProperty.set( temperatureAndColor.temperature );
-      this.sensedElementColorProperty.set( temperatureAndColor.color );
+    step: function() {
+      if ( this.activeProperty.value ) {
+        var temperatureAndColor = this.model.getTemperatureAndColorAtLocation( this.positionProperty.value );
+        this.sensedTemperatureProperty.set( temperatureAndColor.temperature );
+        this.sensedElementColorProperty.set( temperatureAndColor.color );
+      }
+      else {
+        this.sensedTemperatureProperty.set( EFACConstants.ROOM_TEMPERATURE );
+        this.sensedElementColorProperty.set( EFACConstants.TEMPERATURE_SENSOR_INACTIVE_COLOR );
+      }
     },
 
     /**