Drop all eventHub usage.

dannon · Nov 15, 2023 · 433252c · 433252c
1 parent 2bfec6c
commit 433252c
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 33 deletions.
diff --git a/client/src/components/Panels/Common/ToolSection.vue b/client/src/components/Panels/Common/ToolSection.vue
@@ -15,7 +15,7 @@ const emit = defineEmits<{
     (e: "onOperation", tool: any, evt: Event): void;
 }>();
 
-const eventHub = useEventBus<string>("open-tool-section");
+const eventBus = useEventBus<string>("open-tool-section");
 
 const props = defineProps({
     category: {
@@ -134,11 +134,11 @@ watch(
 );
 
 onMounted(() => {
-    eventHub.on(openToolSection);
+    eventBus.on(openToolSection);
 });
 
 onUnmounted(() => {
-    eventHub.off(openToolSection);
+    eventBus.off(openToolSection);
 });
 
 function openToolSection(sectionId: string) {

diff --git a/client/src/components/plugins/eventHub.js b/client/src/components/plugins/eventHub.js
diff --git a/client/src/components/plugins/index.js b/client/src/components/plugins/index.js
@@ -1,4 +1,3 @@
-export { eventHubMixin, eventHubPlugin } from "./eventHub";
 export { iconMixin, iconPlugin } from "./icons";
 export { localizationPlugin } from "./localization";
 export { vueRxShortcutPlugin, vueRxShortcuts } from "./vueRxShortcuts";

diff --git a/client/src/utils/mountVueComponent.js b/client/src/utils/mountVueComponent.js
@@ -3,7 +3,7 @@
 // the same plugins and events.
 
 import BootstrapVue from "bootstrap-vue";
-import { eventHubPlugin, iconPlugin, localizationPlugin, vueRxShortcutPlugin } from "components/plugins";
+import { iconPlugin, localizationPlugin, vueRxShortcutPlugin } from "components/plugins";
 import Vue from "vue";
 import Vuex from "vuex";
 
@@ -14,11 +14,6 @@ Vue.use(Vuex);
 // Bootstrap components
 Vue.use(BootstrapVue);
 
-// Add a global event bus. We could just use root but I don't think that will
-// work right when we have more than one root, which we often will until the
-// application has been completely converted to Vue.
-Vue.use(eventHubPlugin);
-
 // localization filters and directives
 Vue.use(localizationPlugin);
 

diff --git a/client/tests/jest/helpers.js b/client/tests/jest/helpers.js
@@ -3,7 +3,6 @@
  */
 import { createLocalVue, shallowMount } from "@vue/test-utils";
 import BootstrapVue from "bootstrap-vue";
-import { eventHubPlugin } from "components/plugins/eventHub";
 import { iconPlugin } from "components/plugins/icons";
 import { localizationPlugin } from "components/plugins/localization";
 import { vueRxShortcutPlugin } from "components/plugins/vueRxShortcuts";
@@ -190,7 +189,6 @@ export function getLocalVue(instrumentLocalization = false) {
     const l = instrumentLocalization ? testLocalize : _l;
     localVue.use(localizationPlugin, l);
     localVue.use(vueRxShortcutPlugin);
-    localVue.use(eventHubPlugin);
     localVue.use(iconPlugin);
     localVue.directive("b-tooltip", mockedDirective);
     localVue.directive("b-popover", mockedDirective);