Merge pull request #2271 from sul-dlss/unnecessary-semi

Remove unnecessary semi-colins

app/javascript/3d.js

@@ -1,18 +1,18 @@
-"use strict";
+"use strict"
 
-import "controllers";
+import "controllers"
 
-import EmbedThis from "src/modules/embed_this";
-import PopupPanels from "src/modules/popup_panels";
-import ModelViewer from "src/modules/model_viewer";
-import { trackView } from "src/modules/metrics";
+import EmbedThis from "src/modules/embed_this"
+import PopupPanels from "src/modules/popup_panels"
+import ModelViewer from "src/modules/model_viewer"
+import { trackView } from "src/modules/metrics"
 
-import "@google/model-viewer";
+import "@google/model-viewer"
 
 document.addEventListener("DOMContentLoaded", () => {
-  document.getElementById("sul-embed-object").hidden = false;
-  ModelViewer.init();
-  PopupPanels.init();
-  EmbedThis.init();
-  trackView();
-});
+  document.getElementById("sul-embed-object").hidden = false
+  ModelViewer.init()
+  PopupPanels.init()
+  EmbedThis.init()
+  trackView()
+})

app/javascript/controllers/companion_window_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
-import EmbedThis from 'src/modules/embed_this';
+import EmbedThis from 'src/modules/embed_this'
 
 export default class extends Controller {
   // TODO: accessability and transcript should move to a controller just for media.
@@ -9,7 +9,7 @@ export default class extends Controller {
     this.element.hidden = false
     // Open the drawer on load
     this.openLeftDrawer()
-    EmbedThis.init();
+    EmbedThis.init()
   }
 
   toggleLeft() {

app/javascript/controllers/content_list_controller.js

@@ -12,7 +12,7 @@ export default class extends Controller {
   // Show a different media file when the user selects it in the list
   // Sends an event that tells the MediaWrapperController to display the selected media.
   showMedia(evt) {
-    evt.preventDefault();
+    evt.preventDefault()
     const event = new CustomEvent('thumbnail-clicked', { detail: { index: evt.params.index } })
     window.dispatchEvent(event)
 

app/javascript/controllers/file_auth_controller.js

@@ -10,7 +10,7 @@ export default class extends Controller {
     window.addEventListener("message", (event) => {
       this.iframe.remove()
       console.debug("Post message", event.data)
-      if (event.origin !== "https://stacks.stanford.edu" && event.origin !== "https://sul-stacks-stage.stanford.edu") return;
+      if (event.origin !== "https://stacks.stanford.edu" && event.origin !== "https://sul-stacks-stage.stanford.edu") return
       if (event.data.type === "AuthAccessTokenError2") {
         this.displayAccessTokenError(event.data)
       } else {
@@ -225,15 +225,15 @@ export default class extends Controller {
   // This method is triggered by stimulus reflex when the user clicks the login button rendered by `loginNeeded`
   login(evt) {
     this.loginPanelTarget.hidden = true
-    const windowReference = window.open(evt.params.url);
-    let loginStart = Date.now();
+    const windowReference = window.open(evt.params.url)
+    let loginStart = Date.now()
     console.debug("window reference", windowReference)
     let checkWindow = setInterval(() => {
       console.debug("in interval", (Date.now() - loginStart))
       if ((Date.now() - loginStart) < 30000 &&
-        (!windowReference || !windowReference.closed)) return;
+        (!windowReference || !windowReference.closed)) return
 
-      clearInterval(checkWindow);
+      clearInterval(checkWindow)
       // once the window is closed we can initiate the token request
       this.afterLoginWindowClosed(evt.params.messageid)
     }, 500)

app/javascript/controllers/fullscreen_controller.js

@@ -8,10 +8,10 @@ export default class extends Controller {
       this.areaTarget.requestFullscreen().catch((err) => {
         alert(
           `Error attempting to enable fullscreen mode: ${err.message} (${err.name})`,
-        );
-      });
+        )
+      })
     } else {
-      document.exitFullscreen();
+      document.exitFullscreen()
     }
   }
 }

app/javascript/controllers/iiif_metadata_controller.js

@@ -14,9 +14,9 @@ export default class extends Controller {
       const node = new DOMParser().parseFromString(record, "text/html").body.firstChild
       if ('target' in node) {
         // Set all elements owning target to target=_blank
-        node.setAttribute('target', '_blank');
+        node.setAttribute('target', '_blank')
         // Prevent https://www.owasp.org/index.php/Reverse_Tabnabbing
-        node.setAttribute('rel', 'noopener noreferrer');
+        node.setAttribute('rel', 'noopener noreferrer')
         return `<dd>${node.outerHTML}</dd>`
       } else {
         return `<dd>${record}</dd>`

app/javascript/controllers/media_player_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
-import videojs from 'video.js';
+import videojs from 'video.js'
 
 export default class extends Controller {
   initializeVideoJSPlayer() {

app/javascript/controllers/media_tag_controller.js

@@ -31,7 +31,7 @@ export default class extends Controller {
     }
 
     if(typeof(completeCallback) === 'function') {
-      completeCallback(result.authResponse);
+      completeCallback(result.authResponse)
     }
   }
 
@@ -55,17 +55,17 @@ export default class extends Controller {
 
   // Open the login window in a new window and then poll to see if the auth credentials are now active.
   logIn(event) {
-    const windowReference = window.open(event.params.loginService);
-    this.loginStart = Date.now();
+    const windowReference = window.open(event.params.loginService)
+    this.loginStart = Date.now()
     var checkWindow = setInterval(() => {
       if ((Date.now() - this.loginStart) < 30000 &&
-        (!windowReference || !windowReference.closed)) return;
+        (!windowReference || !windowReference.closed)) return
       this.validateMedia((authResponse) => {
         if(authResponse.status === 'success') {
-          clearInterval(checkWindow);
+          clearInterval(checkWindow)
         }
-      });
-      return;
+      })
+      return
     }, 500)
   }
 }

app/javascript/controllers/media_wrapper_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
-import videojs from 'video.js';
+import videojs from 'video.js'
 
 export default class extends Controller {
   static targets = [ ]
@@ -11,7 +11,7 @@ export default class extends Controller {
   toggleVisibility(event) {
     const index =  event.detail.index
     this.element.hidden = this.indexValue !== index
-    this.pauseAllMedia();
+    this.pauseAllMedia()
   }
 
   pauseAllMedia() {

app/javascript/controllers/osd_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
-import OpenSeadragon from 'openseadragon';
+import OpenSeadragon from 'openseadragon'
 
 export default class extends Controller {
   static values = {

app/javascript/controllers/transcript_controller.js

@@ -1,5 +1,5 @@
 import { Controller } from "@hotwired/stimulus"
-import videojs from 'video.js';
+import videojs from 'video.js'
 
 // This is tightly coupled to VideoJS's tracks implementation, because VideoJS removes the tracks from the
 // native player when it initializes.  This depends on the media_tag_controller.js emitting a custom media-loaded

app/javascript/document.js

@@ -1,6 +1,6 @@
-import "controllers";
-import { trackView } from "src/modules/metrics";
+import "controllers"
+import { trackView } from "src/modules/metrics"
 
 document.addEventListener("DOMContentLoaded", () => {
-  trackView();
-});
+  trackView()
+})

app/javascript/file.js

@@ -1,11 +1,11 @@
-import EmbedThis from 'src/modules/embed_this';
-import PopupPanels from 'src/modules/popup_panels';
-import { trackView } from 'src/modules/metrics';
+import EmbedThis from 'src/modules/embed_this'
+import PopupPanels from 'src/modules/popup_panels'
+import { trackView } from 'src/modules/metrics'
 import "file_controllers"
 
 document.addEventListener('DOMContentLoaded', () => {
   document.getElementById('sul-embed-object').hidden = false
-  PopupPanels.init();
-  EmbedThis.init();
-  trackView();
+  PopupPanels.init()
+  EmbedThis.init()
+  trackView()
 })

app/javascript/file_controllers/download_all_controller.js

@@ -12,14 +12,14 @@ import { Controller } from "@hotwired/stimulus"
 
 export default class extends Controller {
   download(event) {
-    const el = event.target;
+    const el = event.target
     if (el.localName == 'button') {
-      el.disabled = true;
-      el.innerText = " Initializing download...";
+      el.disabled = true
+      el.innerText = " Initializing download..."
 
       // replace the anchor tag with the button to prevent further clicks
-      const a = el.parentElement;
-      a.replaceWith(el);
+      const a = el.parentElement
+      a.replaceWith(el)
     }
   }
 }

app/javascript/file_controllers/tree_controller.js

@@ -111,65 +111,65 @@ export default class extends Controller {
       case event.key === 'Enter' && this.isBranch(event.target):
         event.preventDefault()
         this.toggle(event)
-        break;
+        break
       case event.key === 'Enter' && this.isLeaf(event.target):
         // Perform the default action (download the file)
         const downloadElement = this.downloadElement(event.target)
-        if(!downloadElement) break;
+        if(!downloadElement) break
 
         event.preventDefault()
         downloadElement.click()
-        break;
+        break
       case event.key === 'ArrowRight' && this.isClosedBranch(event.target):
         // When a closed node, opens the node
         event.preventDefault()
         this.toggle(event)
-        break;
+        break
       case event.key === 'ArrowRight' && this.isOpenBranch(event.target):
         // When an open node, moves focus to the first child node
         event.preventDefault()
         event.target.nextElementSibling.focus()
-        break;
+        break
       case event.key === 'ArrowLeft' && this.isOpenBranch(event.target):
         // When an open node, closes the node
         event.preventDefault()
         this.toggle(event)
-        break;
+        break
       case event.key === 'ArrowLeft' && event.target.getAttribute('aria-level') > 1:
         // When a closed node, moves focus to the node's parent node
         const parentBranchRowElement = this.parentBranchRowElement(event.target)
-        if(!parentBranchRowElement) break;
+        if(!parentBranchRowElement) break
 
         event.preventDefault()
         parentBranchRowElement.focus()
-        break;
+        break
       case event.key === 'ArrowUp':
         // Move to the previous node without opening or closing
         const previousRowElement = this.previousBranchRowElement(event.target)
-        if(!previousRowElement) break;
+        if(!previousRowElement) break
 
         event.preventDefault()
         previousRowElement.focus()
-        break;
+        break
       case event.key === 'ArrowDown':
         // Move to the next node without opening or closing
         const nextRowElement = this.nextBranchRowElement(event.target)
-        if(!nextRowElement) break;
+        if(!nextRowElement) break
 
         event.preventDefault()
         nextRowElement.focus()
-        break;
+        break
       case event.key === 'Home':
         // Move to the first node without opening or closing
         event.preventDefault()
         this.visibleRows[0].focus()
-        break;
+        break
       case event.key === 'End':
         // Move to the last node without opening or closing
         event.preventDefault()
         const rowElements = this.visibleRows
         rowElements[rowElements.length - 1].focus()
-        break;
+        break
     }
   }
 

app/javascript/geo.js

@@ -1,15 +1,15 @@
-import EmbedThis from "../src/modules/embed_this";
-import PopupPanels from "../src/modules/popup_panels";
-import GeoViewer from "../src/modules/geo_viewer";
-import { trackView } from "../src/modules/metrics";
+import EmbedThis from "../src/modules/embed_this"
+import PopupPanels from "../src/modules/popup_panels"
+import GeoViewer from "../src/modules/geo_viewer"
+import { trackView } from "../src/modules/metrics"
 
-import "leaflet";
-import "../../../vendor/assets/javascripts/Leaflet.Control.Custom";
+import "leaflet"
+import "../../../vendor/assets/javascripts/Leaflet.Control.Custom"
 
 document.addEventListener("DOMContentLoaded", () => {
-  document.getElementById("sul-embed-object").hidden = false;
-  GeoViewer.init();
-  PopupPanels.init();
-  EmbedThis.init();
-  trackView();
-});
+  document.getElementById("sul-embed-object").hidden = false
+  GeoViewer.init()
+  PopupPanels.init()
+  EmbedThis.init()
+  trackView()
+})

app/javascript/media.js

@@ -1,6 +1,6 @@
-import "controllers";
-import { trackView } from "src/modules/metrics";
+import "controllers"
+import { trackView } from "src/modules/metrics"
 
 document.addEventListener("DOMContentLoaded", () => {
-  trackView();
-});
+  trackView()
+})

app/javascript/packs/geo.js

@@ -1,15 +1,15 @@
-import EmbedThis from "../src/modules/embed_this";
-import PopupPanels from "../src/modules/popup_panels";
-import GeoViewer from "../src/modules/geo_viewer";
-import { trackView } from "../src/modules/metrics";
+import EmbedThis from "../src/modules/embed_this"
+import PopupPanels from "../src/modules/popup_panels"
+import GeoViewer from "../src/modules/geo_viewer"
+import { trackView } from "../src/modules/metrics"
 
-import "leaflet";
-import "../../../vendor/assets/javascripts/Leaflet.Control.Custom";
+import "leaflet"
+import "../../../vendor/assets/javascripts/Leaflet.Control.Custom"
 
 document.addEventListener("DOMContentLoaded", () => {
-  document.getElementById("sul-embed-object").hidden = false;
-  GeoViewer.init();
-  PopupPanels.init();
-  EmbedThis.init();
-  trackView();
-});
+  document.getElementById("sul-embed-object").hidden = false
+  GeoViewer.init()
+  PopupPanels.init()
+  EmbedThis.init()
+  trackView()
+})

app/javascript/packs/m3.js

@@ -1,9 +1,9 @@
-import M3Viewer from "../src/modules/m3_viewer.js";
-import { trackView } from "../src/modules/metrics.js";
+import M3Viewer from "../src/modules/m3_viewer.js"
+import { trackView } from "../src/modules/metrics.js"
 
-document.getElementById("sul-embed-object").hidden = false;
-M3Viewer.init();
+document.getElementById("sul-embed-object").hidden = false
+M3Viewer.init()
 
 document.addEventListener("DOMContentLoaded", () => {
-  trackView();
-});
+  trackView()
+})