From 7e5026dfc5131b70fceb9aeccadbdbd04daa0244 Mon Sep 17 00:00:00 2001
From: Calixte Denizet <calixte.denizet@gmail.com>
Date: Tue, 8 Sep 2020 19:07:10 +0200
Subject: [PATCH] Don't try to save something for a button which is neither a
 checkbox nor a radio

---
 src/core/annotation.js       |  3 ++-
 test/unit/annotation_spec.js | 23 +++++++++++++++++++++++
 2 files changed, 25 insertions(+), 1 deletion(-)

diff --git a/src/core/annotation.js b/src/core/annotation.js
index 6263608fc..c837ab379 100644
--- a/src/core/annotation.js
+++ b/src/core/annotation.js
@@ -1505,7 +1505,8 @@ class ButtonWidgetAnnotation extends WidgetAnnotation {
       return this._saveRadioButton(evaluator, task, annotationStorage);
     }
 
-    return super.save(evaluator, task, annotationStorage);
+    // Nothing to save
+    return null;
   }
 
   async _saveCheckbox(evaluator, task, annotationStorage) {
diff --git a/test/unit/annotation_spec.js b/test/unit/annotation_spec.js
index d7fef39bf..4643d79a3 100644
--- a/test/unit/annotation_spec.js
+++ b/test/unit/annotation_spec.js
@@ -2317,6 +2317,29 @@ describe("annotation", function () {
           done();
         }, done.fail);
     });
+
+    it("should save nothing", function (done) {
+      const buttonWidgetRef = Ref.get(124, 0);
+      const xref = new XRefMock([
+        { ref: buttonWidgetRef, data: buttonWidgetDict },
+      ]);
+      const task = new WorkerTask("test save");
+
+      AnnotationFactory.create(
+        xref,
+        buttonWidgetRef,
+        pdfManagerMock,
+        idFactoryMock
+      )
+        .then(annotation => {
+          return annotation.save(partialEvaluator, task, {});
+        })
+        .then(data => {
+          expect(data).toEqual(null);
+          done();
+        })
+        .catch(done.fail);
+    });
   });
 
   describe("ChoiceWidgetAnnotation", function () {