fixed some code smells

ui/src/components/admin/ConfigView.vue

@@ -161,7 +161,7 @@
 </template>
 
 <script lang="ts">
-import { AdminConfigModel, Credentials, DoorsSaved } from "@/lib/api";
+import { AdminConfigModel, Credentials, DoorSaved } from "@/lib/api";
 import { advent22Store } from "@/plugins/store";
 import { DateTime } from "luxon";
 import { Options, Vue } from "vue-class-component";
@@ -216,7 +216,7 @@ export default class extends Vue {
       config_file: "sed diam nonumy",
     },
   };
-  public doors: DoorsSaved = [];
+  public doors: DoorSaved[] = [];
   public dav_credentials: Credentials = ["", ""];
   public ui_credentials: Credentials = ["", ""];
 
@@ -230,7 +230,7 @@ export default class extends Vue {
   public on_open(ready: () => void, fail: () => void): void {
     Promise.all([
       this.$advent22.api_get<AdminConfigModel>("admin/config_model"),
-      this.$advent22.api_get<DoorsSaved>("admin/doors"),
+      this.$advent22.api_get<DoorSaved[]>("admin/doors"),
     ])
       .then(([admin_config_model, doors]) => {
         this.admin_config_model = admin_config_model;

ui/src/components/admin/DoorMapEditor.vue

@@ -69,7 +69,7 @@
 </template>
 
 <script lang="ts">
-import { DoorsSaved } from "@/lib/api";
+import { DoorSaved } from "@/lib/api";
 import { Door } from "@/lib/door";
 import { Options, Vue } from "vue-class-component";
 
@@ -105,7 +105,7 @@ export default class extends Vue {
   private load_doors(): Promise<void> {
     return new Promise<void>((resolve, reject) => {
       this.$advent22
-        .api_get<DoorsSaved>("admin/doors")
+        .api_get<DoorSaved[]>("admin/doors")
         .then((data) => {
           this.doors.length = 0;
 
@@ -124,7 +124,7 @@ export default class extends Vue {
 
   private save_doors(): Promise<void> {
     return new Promise<void>((resolve, reject) => {
-      const data: DoorsSaved = [];
+      const data: DoorSaved[] = [];
 
       for (const door of this.doors) {
         data.push(door.save());

ui/src/components/calendar/ThouCanvas.vue

@@ -21,15 +21,13 @@ import { advent22Store } from "@/plugins/store";
 import { Options, Vue } from "vue-class-component";
 
 function get_event_thous(event: MouseEvent): Vector2D {
-  if (event.currentTarget === null) {
+  if (!(event.currentTarget instanceof SVGSVGElement)) {
     return new Vector2D();
   }
 
-  const target = event.currentTarget as Element;
-
   return new Vector2D(
-    Math.round((event.offsetX / target.clientWidth) * 1000),
+    Math.round((event.offsetX / event.currentTarget.clientWidth) * 1000),
-    Math.round((event.offsetY / target.clientHeight) * 1000),
+    Math.round((event.offsetY / event.currentTarget.clientHeight) * 1000),
   );
 }
 

ui/src/components/editor/PreviewDoor.vue

@@ -52,7 +52,7 @@ export default class extends Vue {
   }
 
   public on_click(event: MouseEvent) {
-    if (event.target === null || !(event.target instanceof SVGRectElement)) {
+    if (!(event.currentTarget instanceof SVGRectElement)) {
       return;
     }
 

ui/src/lib/api.ts

@@ -54,8 +54,6 @@ export interface DoorSaved {
   y2: number;
 }
 
-export type DoorsSaved = DoorSaved[];
-
 export type Credentials = [username: string, password: string];
 
 export function objForEach<T>(

ui/src/main.ts

@@ -8,11 +8,11 @@ import App from "./App.vue";
 import "@/main.scss";
 
 const app = createApp(App);
+
 app.use(Advent22Plugin);
 app.use(FontAwesomePlugin);
-app.use(createPinia());
 
-const store = advent22Store();
+app.use(createPinia());
-store.init(app.config.globalProperties.$advent22);
+advent22Store().init();
 
 app.mount("#app");

ui/src/plugins/store.ts

@@ -1,6 +1,6 @@
-import { Credentials, DoorsSaved, SiteConfigModel } from "@/lib/api";
+import { Credentials, DoorSaved, SiteConfigModel } from "@/lib/api";
 import { Advent22 } from "@/plugins/advent22";
-import { useLocalStorage } from "@vueuse/core";
+import { RemovableRef, useLocalStorage } from "@vueuse/core";
 import { AxiosBasicCredentials } from "axios";
 import { acceptHMRUpdate, defineStore } from "pinia";
 
@@ -10,12 +10,23 @@ declare global {
   }
 }
 
+type State = {
+  advent22: Advent22;
+  api_creds: RemovableRef<Credentials>;
+  is_touch_device: boolean;
+  is_admin: boolean;
+  site_config: SiteConfigModel;
+  calendar_aspect_ratio: number;
+  user_doors: DoorSaved[];
+  next_door_target: number | null;
+};
+
 export const advent22Store = defineStore({
   id: "advent22",
 
-  state: () => ({
+  state: (): State => ({
-    advent22: {} as Advent22,
+    advent22: new Advent22(),
-    api_creds: useLocalStorage<Credentials>("advent22/auth", ["", ""]),
+    api_creds: useLocalStorage("advent22/auth", ["", ""]),
     is_touch_device:
       window.matchMedia("(any-hover: none)").matches ||
       "ontouchstart" in window ||
@@ -27,10 +38,10 @@ export const advent22Store = defineStore({
       subtitle: "",
       content: "",
       footer: "",
-    } as SiteConfigModel,
+    },
     calendar_aspect_ratio: 1,
-    user_doors: [] as DoorsSaved,
+    user_doors: [],
-    next_door_target: null as number | null,
+    next_door_target: null,
   }),
 
   getters: {
@@ -41,11 +52,10 @@ export const advent22Store = defineStore({
   },
 
   actions: {
-    init(advent22: Advent22): void {
+    init(): void {
-      this.advent22 = advent22;
       this.update_is_admin();
 
-      advent22
+      this.advent22
         .api_get_blob("user/favicon")
         .then((favicon_src) => {
           const link: HTMLLinkElement =
@@ -61,9 +71,9 @@ export const advent22Store = defineStore({
         .catch(() => {});
 
       Promise.all([
-        advent22.api_get<SiteConfigModel>("user/site_config"),
+        this.advent22.api_get<SiteConfigModel>("user/site_config"),
-        advent22.api_get<DoorsSaved>("user/doors"),
+        this.advent22.api_get<DoorSaved[]>("user/doors"),
-        advent22.api_get<number | null>("user/next_door"),
+        this.advent22.api_get<number | null>("user/next_door"),
       ])
         .then(([site_config, user_doors, next_door]) => {
           document.title = site_config.title;
@@ -77,11 +87,11 @@ export const advent22Store = defineStore({
           if (next_door !== null)
             this.next_door_target = Date.now() + next_door;
         })
-        .catch(advent22.alert_user_error);
+        .catch(this.advent22.alert_user_error);
     },
 
     update_is_admin(): Promise<boolean> {
-      return new Promise<boolean>((resolve, reject) => {
+      return new Promise((resolve, reject) => {
         this.advent22
           .api_get<boolean>("admin/is_admin")
           .then((is_admin) => {