From b1db52d2b6b2ffad097490074cc0ff2457239d37 Mon Sep 17 00:00:00 2001
From: Kenneth SOARES <kensoares@gmail.com>
Date: Mon, 17 Feb 2025 17:51:42 +0100
Subject: [PATCH] clean typescript

---
 core/static/bundled/user/pictures-index.ts | 75 +++++++---------------
 core/templates/core/user_pictures.jinja    |  8 +--
 sas/templates/sas/album.jinja              |  3 -
 3 files changed, 24 insertions(+), 62 deletions(-)

diff --git a/core/static/bundled/user/pictures-index.ts b/core/static/bundled/user/pictures-index.ts
index 34140323..64af694d 100644
--- a/core/static/bundled/user/pictures-index.ts
+++ b/core/static/bundled/user/pictures-index.ts
@@ -1,43 +1,11 @@
 import { paginated } from "#core:utils/api";
 import { HttpReader, ZipWriter } from "@zip.js/zip.js";
 import { showSaveFilePicker } from "native-file-system-adapter";
-import { picturesFetchPictures } from "#openapi";
-
-/**
- * @typedef UserProfile
- * @property {number} id
- * @property {string} first_name
- * @property {string} last_name
- * @property {string} nick_name
- * @property {string} display_name
- * @property {string} profile_url
- * @property {string} profile_pict
- */
-/**
- * @typedef Picture
- * @property {number} id
- * @property {string} name
- * @property {number} size
- * @property {string} date
- * @property {UserProfile} owner
- * @property {string} full_size_url
- * @property {string} compressed_url
- * @property {string} thumb_url
- * @property {string} album
- * @property {boolean} is_moderated
- * @property {boolean} asked_for_removal
- */
-
-/**
- * @typedef PicturePageConfig
- * @property {number} userId Id of the user to get the pictures from (optional if albumId defined)
- * @property {number} albumId Id of the album to get the pictures from (optinal if userId defined)
- **/
-
-/**
- * Load user picture page with a nice download bar
- * @param {PicturePageConfig} config
- **/
+import {
+  type PicturesFetchPicturesData,
+  type PictureSchema,
+  picturesFetchPictures,
+} from "#openapi";
 
 interface PagePictureConfig {
   userId?: number;
@@ -48,26 +16,29 @@ document.addEventListener("alpine:init", () => {
   Alpine.data("user_pictures", (config: PagePictureConfig) => ({
     isDownloading: false,
     loading: true,
-    pictures: [],
-    albums: {},
+    pictures: [] as PictureSchema[],
+    albums: {} as Record<string, PictureSchema[]>,
 
     async init() {
-      const query: Record<string, number> = {};
+      const query: PicturesFetchPicturesData["query"] = {};
 
       if (config.userId) {
-        query.user_identified = config.userId;
+        query.users_identified = [config.userId];
       } else {
         query.album_id = config.albumId;
       }
       this.pictures = await paginated(picturesFetchPictures, { query: query });
 
-      this.albums = this.pictures.reduce((acc, picture) => {
-        if (!acc[picture.album]) {
-          acc[picture.album] = [];
-        }
-        acc[picture.album].push(picture);
-        return acc;
-      }, {});
+      this.albums = this.pictures.reduce(
+        (acc: Record<string, PictureSchema[]>, picture: PictureSchema) => {
+          if (!acc[picture.album]) {
+            acc[picture.album] = [];
+          }
+          acc[picture.album].push(picture);
+          return acc;
+        },
+        {},
+      );
       this.loading = false;
     },
 
@@ -77,8 +48,9 @@ document.addEventListener("alpine:init", () => {
       bar.value = 0;
       bar.max = this.pictures.length;
 
-      const incrementProgressBar = () => {
+      const incrementProgressBar = (_total: number): undefined => {
         bar.value++;
+        return undefined;
       };
 
       const fileHandle = await showSaveFilePicker({
@@ -88,14 +60,13 @@ document.addEventListener("alpine:init", () => {
           { extension: "zip" },
           true,
         ),
-        types: {},
         excludeAcceptAllOption: false,
       });
       const zipWriter = new ZipWriter(await fileHandle.createWritable());
 
       await Promise.all(
-        this.pictures.map((p) => {
-          const imgName = `${p.album}/IMG_${p.date.replaceAll(/[:\-]/g, "_")}${p.name.slice(p.name.lastIndexOf("."))}`;
+        this.pictures.map((p: PictureSchema) => {
+          const imgName = `${p.album}/IMG_${p.date.replace(/[:\-]/g, "_")}${p.name.slice(p.name.lastIndexOf("."))}`;
           return zipWriter.add(imgName, new HttpReader(p.full_size_url), {
             level: 9,
             lastModDate: new Date(p.date),
diff --git a/core/templates/core/user_pictures.jinja b/core/templates/core/user_pictures.jinja
index e5fd36ce..c51d2f0e 100644
--- a/core/templates/core/user_pictures.jinja
+++ b/core/templates/core/user_pictures.jinja
@@ -14,7 +14,7 @@
 {% endblock %}
 
 {% block content %}
-  <main x-data="user_pictures">
+  <main x-data="user_pictures({ userId: {{ object.id }} })">
     {% if user.id == object.id %}
       {{ download_button() }}
     {% endif %}
@@ -49,11 +49,5 @@
 {% endblock content %}
 
 {% block script %}
-
   {{ super() }}
-  <script>
-    window.addEventListener("DOMContentLoaded", () => {
-      loadPicturePage({ userId: {{ object.id }} });
-    })
-  </script>
 {% endblock script %}
diff --git a/sas/templates/sas/album.jinja b/sas/templates/sas/album.jinja
index f1fc565d..07c0836b 100644
--- a/sas/templates/sas/album.jinja
+++ b/sas/templates/sas/album.jinja
@@ -122,9 +122,6 @@
         albumId: {{ album.id }},
         maxPageSize: {{ settings.SITH_SAS_IMAGES_PER_PAGE }},
       });
-
-      loadPicturePage({ albumId: {{ album.id }} })
-
     });
 
     // Todo: migrate to alpine.js if we have some time