feat: upgrade fields to use new JSO hooks (google#5077)

* Upgrade field angle to use new serialization

* Upgrade field checkbox to use new serialization

* Upgrade field colour to use new serialization

* Upgrade field dropdown to use new serialization

* Upgrade serializable label field to use new serialization

* Upgrade field multiline input to use new serialization

* Upgrade field number to use new serialization

* Upgrade field text input to use new serialization

* Upgrade variable field to use new serialization

* Fix type casts

* Feedback from PR

* Switch to use getValue()

core/field_angle.js

@@ -241,8 +241,6 @@ Blockly.FieldAngle.prototype.configure_ = function(config) {
   }
 };
 
-
-
 /**
  * Create the block UI for this field.
  * @package
@@ -256,6 +254,26 @@ Blockly.FieldAngle.prototype.initView = function() {
   this.textElement_.appendChild(this.symbol_);
 };
 
+/**
+ * Saves this field's value.
+ * @return {number} The angle value held by this field.
+ * @override
+ * @package
+ */
+Blockly.FieldAngle.prototype.saveState = function() {
+  return /** @type {number} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the angle field.
+ * @override
+ * @package
+ */
+Blockly.FieldAngle.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * Updates the graph when the field rerenders.
  * @protected

core/field_checkbox.js

@@ -102,6 +102,26 @@ FieldCheckbox.prototype.configure_ = function(config) {
   }
 };
 
+/**
+ * Saves this field's value.
+ * @return {boolean} The boolean value held by this field.
+ * @override
+ * @package
+ */
+FieldCheckbox.prototype.saveState = function() {
+  return /** @type {boolean} */ (this.getValueBoolean());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the checkbox field.
+ * @override
+ * @package
+ */
+FieldCheckbox.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * Create the block UI for this checkbox.
  * @package

core/field_colour.js

@@ -188,6 +188,26 @@ FieldColour.prototype.initView = function() {
   }
 };
 
+/**
+ * Saves this field's value.
+ * @return {string} The colour value held by this field.
+ * @override
+ * @package
+ */
+FieldColour.prototype.saveState = function() {
+  return /** @type {string} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the colour field.
+ * @override
+ * @package
+ */
+FieldColour.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * @override
  */

core/field_dropdown.js

@@ -170,6 +170,29 @@ FieldDropdown.prototype.fromXml = function(fieldElement) {
   this.setValue(fieldElement.textContent);
 };
 
+/**
+ * Saves this field's value.
+ * @return {string} The dropdown value held by this field.
+ * @override
+ * @package
+ */
+FieldDropdown.prototype.saveState = function() {
+  return /** @type {string} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the dropdown field.
+ * @override
+ * @package
+ */
+FieldDropdown.prototype.loadState = function(state) {
+  if (this.isOptionListDynamic()) {
+    this.getOptions(false);
+  }
+  this.setValue(state);
+};
+
 /**
  * Serializable fields are saved by the XML renderer, non-serializable fields
  * are not. Editable fields should also be serializable.

core/field_label_serializable.js

@@ -68,6 +68,26 @@ FieldLabelSerializable.prototype.EDITABLE = false;
  */
 FieldLabelSerializable.prototype.SERIALIZABLE = true;
 
+/**
+ * Saves this field's value.
+ * @return {string} The text value held by this field.
+ * @override
+ * @package
+ */
+FieldLabelSerializable.prototype.saveState = function() {
+  return /** @type {string} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the label field.
+ * @override
+ * @package
+ */
+FieldLabelSerializable.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 fieldRegistry.register('field_label_serializable', FieldLabelSerializable);
 
 exports = FieldLabelSerializable;

core/field_multilineinput.js

@@ -122,6 +122,26 @@ FieldMultilineInput.prototype.fromXml = function(fieldElement) {
   this.setValue(fieldElement.textContent.replace(/
/g, '\n'));
 };
 
+/**
+ * Saves this field's value.
+ * @return {string} The text value held by this field.
+ * @override
+ * @package
+ */
+FieldMultilineInput.prototype.saveState = function() {
+  return /** @type {string} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the multiline input field.
+ * @override
+ * @package
+ */
+FieldMultilineInput.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * Create the block UI for this field.
  * @package

core/field_number.js

@@ -118,6 +118,26 @@ FieldNumber.prototype.configure_ = function(config) {
   this.setPrecisionInternal_(config['precision']);
 };
 
+/**
+ * Saves this field's value.
+ * @return {number} The number value held by this field.
+ * @override
+ * @package
+ */
+FieldNumber.prototype.saveState = function() {
+  return /** @type {number} */ (this.getValue());
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the nuber field.
+ * @override
+ * @package
+ */
+FieldNumber.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * Set the maximum, minimum and precision constraints on this field.
  * Any of these properties may be undefined or NaN to be disabled.

core/field_textinput.js

@@ -182,6 +182,26 @@ FieldTextInput.prototype.initView = function() {
   this.createTextElement_();
 };
 
+/**
+ * Saves this field's value.
+ * @return {*} The text value held by this field.
+ * @override
+ * @package
+ */
+FieldTextInput.prototype.saveState = function() {
+  return this.getValue();
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} state The state to apply to the text input field.
+ * @override
+ * @package
+ */
+FieldTextInput.prototype.loadState = function(state) {
+  this.setValue(state);
+};
+
 /**
  * Ensure that the input value casts to a valid string.
  * @param {*=} opt_newValue The input value.

core/field_variable.js

@@ -197,6 +197,28 @@ FieldVariable.prototype.toXml = function(fieldElement) {
   return fieldElement;
 };
 
+/**
+ * Saves this field's value.
+ * @return {string} The id of the variable referenced by this field.
+ * @override
+ * @package
+ */
+FieldVariable.prototype.saveState = function() {
+  // Make sure the variable is initialized.
+  this.initModel();
+  return this.variable_.getId();
+};
+
+/**
+ * Sets the field's value based on the given state.
+ * @param {*} id The id of the variable to assign to this variable field.
+ * @override
+ * @package
+ */
+FieldVariable.prototype.loadState = function(id) {
+  this.setValue(id);
+};
+
 /**
  * Attach this field to a block.
  * @param {!Block} block The block containing this field.

tests/mocha/field_angle_test.js

@@ -6,7 +6,7 @@
 
 goog.module('Blockly.test.fieldAngle');
 
-const {createTestBlock, sharedTestSetup, sharedTestTeardown} = goog.require('Blockly.test.helpers');
+const {createTestBlock, defineRowBlock, sharedTestSetup, sharedTestTeardown, workspaceTeardown} = goog.require('Blockly.test.helpers');
 
 
 suite('Angle Fields', function() {
@@ -315,4 +315,35 @@ suite('Angle Fields', function() {
       });
     });
   });
+
+  suite('Serialization', function() {
+    setup(function() {
+      this.workspace = new Blockly.Workspace();
+      defineRowBlock();
+
+      this.assertValue = (value) => {
+        const block = this.workspace.newBlock('row_block');
+        const field = new Blockly.FieldAngle(value);
+        block.getInput('INPUT').appendField(field, 'ANGLE');
+        const jso = Blockly.serialization.blocks.save(block);
+        chai.assert.deepEqual(jso['fields'], {'ANGLE': value});
+      };
+    });
+
+    teardown(function() {
+      workspaceTeardown.call(this, this.workspace);
+    });
+
+    test('Simple', function() {
+      this.assertValue(90);
+    });
+
+    test('Max precision', function() {
+      this.assertValue(1.000000000000001);
+    });
+
+    test('Smallest number', function() {
+      this.assertValue(5e-324);
+    });
+  });
 });

tests/mocha/field_checkbox_test.js

@@ -6,7 +6,7 @@
 
 goog.module('Blockly.test.fieldCheckbox');
 
-const {sharedTestSetup, sharedTestTeardown} = goog.require('Blockly.test.helpers');
+const {defineRowBlock, sharedTestSetup, sharedTestTeardown, workspaceTeardown} = goog.require('Blockly.test.helpers');
 
 
 suite('Checkbox Fields', function() {
@@ -209,4 +209,31 @@ suite('Checkbox Fields', function() {
       });
     });
   });
+
+  suite('Serialization', function() {
+    setup(function() {
+      this.workspace = new Blockly.Workspace();
+      defineRowBlock();
+
+      this.assertValue = (value) => {
+        const block = this.workspace.newBlock('row_block');
+        const field = new Blockly.FieldCheckbox(value);
+        block.getInput('INPUT').appendField(field, 'CHECK');
+        const jso = Blockly.serialization.blocks.save(block);
+        chai.assert.deepEqual(jso['fields'], {'CHECK': value});
+      };
+    });
+
+    teardown(function() {
+      workspaceTeardown.call(this, this.workspace);
+    });
+
+    test('True', function() {
+      this.assertValue(true);
+    });
+
+    test('False', function() {
+      this.assertValue(false);
+    });
+  });
 });

tests/mocha/field_colour_test.js

@@ -6,7 +6,7 @@
 
 goog.module('Blockly.test.fieldColour');
 
-const {createTestBlock, sharedTestSetup, sharedTestTeardown} = goog.require('Blockly.test.helpers');
+const {createTestBlock, defineRowBlock, sharedTestSetup, sharedTestTeardown, workspaceTeardown} = goog.require('Blockly.test.helpers');
 
 
 suite('Colour Fields', function() {
@@ -282,4 +282,31 @@ suite('Colour Fields', function() {
       });
     });
   });
+
+  suite('Serialization', function() {
+    setup(function() {
+      this.workspace = new Blockly.Workspace();
+      defineRowBlock();
+
+      this.assertValue = (value) => {
+        const block = this.workspace.newBlock('row_block');
+        const field = new Blockly.FieldColour(value);
+        block.getInput('INPUT').appendField(field, 'COLOUR');
+        const jso = Blockly.serialization.blocks.save(block);
+        chai.assert.deepEqual(jso['fields'], {'COLOUR': value});
+      };
+    });
+
+    teardown(function() {
+      workspaceTeardown.call(this, this.workspace);
+    });
+
+    test('Three char', function() {
+      this.assertValue('#001122');
+    });
+
+    test('Six char', function() {
+      this.assertValue('#012345');
+    });
+  });
 });