Refactoring the JavaScript used to confirm the license granted for a (#…

…1910)

dataset

app/javascript/entrypoints/application.js

@@ -16,6 +16,7 @@ import './vendor/jquery-ui-triggeredAutocomplete';
 
 import PdcUiLoader from './pdc/pdc_ui_loader.es6';
 import WorksWizardPolicy from './works_wizard_policy';
+import WorksWizardReview from './works_wizard_review';
 
 // If using a TypeScript entrypoint file:
 //     <%= vite_typescript_tag 'application' %>
@@ -44,6 +45,7 @@ function ready() {
   loader.run();
 
   WorksWizardPolicy.bind('#agreement');
+  WorksWizardReview.bind('#grant-button');
 }
 
 // Must run the javascript loader on every page even if turbolinks loads it

app/javascript/entrypoints/works_wizard_review.js

@@ -0,0 +1,32 @@
+class WorksWizardReview {
+  static bind(elementSelector) {
+    let built;
+    const root = document.querySelector(elementSelector);
+
+    if (root) {
+      built = new WorksWizardReview(root);
+    }
+
+    return built;
+  }
+
+  static handleClick(event) {
+    const message = "You're about to grant license for this dataset. Are you sure?";
+    const input = window.confirm(message);
+
+    if (input === false) {
+      event.preventDefault();
+    }
+  }
+
+  constructor(root) {
+    this.root = root;
+
+    if (this.root) {
+      const handleClick = WorksWizardReview.handleClick.bind(this);
+      this.root.addEventListener('click', handleClick);
+    }
+  }
+}
+
+export default WorksWizardReview;

app/views/works_wizard/review.html.erb

@@ -18,12 +18,3 @@
   <% end %>
 </div>
 
-<script>
-  $("#grant-button").on("click", (event) => {
-    var message = "You're about to grant license for this dataset. Are you sure?";
-    var input = confirm(message);
-    if(input == false){
-      event.preventDefault();
-    }
-  });
-</script>