SAK-48326 WC sakai-picture-changer dont attach all of the cropping un…

…til the user wants to change their profile image (#12325)
sakaiproject · Feb 12, 2024 · 4a592f4 · 4a592f4
1 parent 7e48a1c
commit 4a592f4
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/...mponents/tool/src/main/frontend/packages/sakai-picture-changer/src/SakaiPictureChanger.js b/...mponents/tool/src/main/frontend/packages/sakai-picture-changer/src/SakaiPictureChanger.js
@@ -22,7 +22,6 @@ export class SakaiPictureChanger extends SakaiElement {
     super();
 
     this.loadTranslations("sakai-picture-changer").then(r => this._i18n = r);
-    this._loadExisting();
   }
 
   _attachCropper() {
@@ -48,7 +47,13 @@ export class SakaiPictureChanger extends SakaiElement {
   }
 
   firstUpdated() {
-    this._loadExisting();
+    // Add an event listener for the Bootstrap 'show.bs.modal' event
+    const modal = this.renderRoot.querySelector("#profile-image-upload");
+    if (modal) {
+      modal.addEventListener("show.bs.modal", () => {
+        this._loadExisting();
+      });
+    }
   }
 
   _filePicked(e) {
@@ -123,6 +128,7 @@ export class SakaiPictureChanger extends SakaiElement {
   }
 
   _loadExisting() {
+    if (this._imageUrl) return;
 
     const url = `/direct/profile-image/details?_=${Date.now()}`;
     fetch(url, { credentials: "include" }).then(r => r.json()).then(json => {
@@ -199,7 +205,7 @@ export class SakaiPictureChanger extends SakaiElement {
   }
 
   shouldUpdate() {
-    return this._i18n && this._imageUrl;
+    return this._i18n;
   }
 
   render() {