JS - reset correctly radio buttons

Author: calixteman

src/display/annotation_layer.js

@@ -915,7 +915,7 @@ class CheckboxWidgetAnnotationElement extends WidgetAnnotationElement {
     element.type = "checkbox";
     element.name = this.data.fieldName;
     if (value) {
-      element.setAttribute("checked", true);
+      element.checked = true;
     }
     element.setAttribute("id", id);
 
@@ -1000,9 +1000,8 @@ class RadioButtonWidgetAnnotationElement extends WidgetAnnotationElement {
     element.type = "radio";
     element.name = data.fieldName;
     if (value) {
-      element.setAttribute("checked", true);
+      element.checked = true;
     }
-    element.setAttribute("pdfButtonValue", data.buttonValue);
     element.setAttribute("id", id);
 
     element.addEventListener("change", function (event) {
@@ -1016,19 +1015,16 @@ class RadioButtonWidgetAnnotationElement extends WidgetAnnotationElement {
     });
 
     if (this.enableScripting && this.hasJSActions) {
+      const pdfButtonValue = data.buttonValue;
       element.addEventListener("updatefromsandbox", event => {
         const { detail } = event;
         const actions = {
           value() {
-            const fieldValue = detail.value;
+            const checked = pdfButtonValue === detail.value;
             for (const radio of document.getElementsByName(event.target.name)) {
               const radioId = radio.getAttribute("id");
-              if (fieldValue === radio.getAttribute("pdfButtonValue")) {
-                radio.setAttribute("checked", true);
-                storage.setValue(radioId, { value: true });
-              } else {
-                storage.setValue(radioId, { value: false });
-              }
+              radio.checked = radioId === id && checked;
+              storage.setValue(radioId, { value: radio.checked });
             }
           },
           focus() {

src/scripting_api/field.js

@@ -481,6 +481,9 @@ class RadioButtonField extends Field {
   }
 
   set value(value) {
+    if (value === null) {
+      this._value = "";
+    }
     const i = this.exportValues.indexOf(value);
     if (0 <= i && i < this._radioIds.length) {
       this._id = this._radioIds[i];

test/integration/scripting_spec.js

@@ -174,9 +174,15 @@ describe("Interaction", () => {
     it("must reset all", async () => {
       await Promise.all(
         pages.map(async ([browserName, page]) => {
+          // click on a radio button
+          await page.click("[data-annotation-id='449R']");
+
           // this field has no actions but it must be cleared on reset
           await page.type("#\\34 05R", "employee", { delay: 200 });
 
+          let checked = await page.$eval("#\\34 49R", el => el.checked);
+          expect(checked).toEqual(true);
+
           // click on reset button
           await page.click("[data-annotation-id='402R']");
 
@@ -194,6 +200,9 @@ describe("Interaction", () => {
 
           const sum = await page.$eval("#\\34 27R", el => el.value);
           expect(sum).toEqual("");
+
+          checked = await page.$eval("#\\34 49R", el => el.checked);
+          expect(checked).toEqual(false);
         })
       );
     });