Fix phet-io validation issues, see #589

phetsims · Jul 22, 2020 · bb5ec31 · bb5ec31
1 parent 91c9114
commit bb5ec31
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/js/model/CircuitConstructionKitModel.js b/js/model/CircuitConstructionKitModel.js
@@ -100,7 +100,8 @@ class CircuitConstructionKitModel {
 
     // True if the simulation is playing, controlled by the TimeControlNode
     this.isPlayingProperty = new BooleanProperty( true, {
-      tandem: tandem.createTandem( 'isPlayingProperty' )
+      tandem: tandem.createTandem( 'isPlayingProperty' ),
+      phetioFeatured: true
     } );
 
     // @public {Property.<InteractionMode>} - whether the user is in the CircuitConstructionKitModel.InteractionMode.EXPLORE or CircuitConstructionKitModel.InteractionMode.TEST mode
@@ -135,7 +136,7 @@ class CircuitConstructionKitModel {
           this.isValueDepictionEnabledProperty.value = false;
         }
       };
-       this.circuit.vertexGroup.elementCreatedEmitter.addListener( pause );
+      this.circuit.vertexGroup.elementCreatedEmitter.addListener( pause );
       this.circuit.vertexGroup.elementDisposedEmitter.addListener( pause );
       this.circuit.componentEditedEmitter.addListener( pause );
       this.circuit.circuitElements.lengthProperty.link( pause );

diff --git a/js/model/VoltageSource.js b/js/model/VoltageSource.js
@@ -31,7 +31,7 @@ class VoltageSource extends FixedCircuitElement {
       isFlammable: true,
       numberOfDecimalPlaces: 1,
       voltagePropertyOptions: {
-        tandem: tandem.createTandem( 'voltageProperty' )
+        tandem: tandem.createTandem( 'voltageSourceProperty' ) // TODO: Collides with Battery.voltageProperty
       }
     }, options );
     super( startVertex, endVertex, length, tandem, options );

diff --git a/js/view/CCKCScreenView.js b/js/view/CCKCScreenView.js
@@ -290,6 +290,7 @@ class CCKCScreenView extends ScreenView {
     zoomControlPanel.setScaleMagnitude( 0.8 );
 
     const timeControlNode = new TimeControlNode( model.isPlayingProperty, {
+      tandem: tandem.createTandem( 'timeControlNode' ),
       playPauseStepButtonOptions: {
         stepForwardButtonOptions: {
           listener: () => model.stepSingleStep()