Close #3925. Update selectInput's binding to use selectize.js' getVal…

…ue() method when relevant
rstudio · Oct 26, 2023 · 134d2a0 · 134d2a0
1 parent 81bdde6
commit 134d2a0
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 12 deletions.
diff --git a/inst/www/shared/shiny.js b/inst/www/shared/shiny.js
@@ -9253,7 +9253,12 @@
     }, {
       key: "getValue",
       value: function getValue(el) {
-        return (0, import_jquery15.default)(el).val();
+        if (!isSelectize(el)) {
+          return (0, import_jquery15.default)(el).val();
+        } else {
+          var selectize = this._selectize(el);
+          return selectize === null || selectize === void 0 ? void 0 : selectize.getValue();
+        }
       }
     }, {
       key: "setValue",

diff --git a/inst/www/shared/shiny.js.map b/inst/www/shared/shiny.js.map
diff --git a/inst/www/shared/shiny.min.js b/inst/www/shared/shiny.min.js
diff --git a/inst/www/shared/shiny.min.js.map b/inst/www/shared/shiny.min.js.map
diff --git a/srcts/src/bindings/input/selectInput.ts b/srcts/src/bindings/input/selectInput.ts
@@ -3,7 +3,6 @@ import { InputBinding } from "./inputBinding";
 import { $escape, hasDefinedProperty, updateLabel } from "../../utils";
 import { indirectEval } from "../../utils/eval";
 import { shinyBindAll, shinyUnbindAll } from "../../shiny/initedMethods";
-import type { NotUndefined } from "../../utils/extraTypes";
 
 type SelectHTMLElement = HTMLSelectElement & { nonempty: boolean };
 
@@ -62,10 +61,14 @@ class SelectInputBinding extends InputBinding {
   getId(el: SelectHTMLElement): string {
     return InputBinding.prototype.getId.call(this, el) || el.name;
   }
-  getValue(
-    el: HTMLElement
-  ): NotUndefined<ReturnType<JQuery<HTMLElement>["val"]>> {
-    return $(el).val() as NotUndefined<ReturnType<JQuery<HTMLElement>["val"]>>;
+  getValue(el: SelectHTMLElement): any {
+    if (!isSelectize(el)) {
+      return $(el).val();
+    } else {
+      const selectize = this._selectize(el);
+
+      return selectize?.getValue();
+    }
   }
   setValue(el: SelectHTMLElement, value: string): void {
     if (!isSelectize(el)) {