Merge branch 'fixes/fix-bijou-and-pacemaker-photo' into staging

gestion/lib/Controller/PageController.php

@@ -2791,10 +2791,10 @@ class PageController extends Controller {
 				return false;
 			}
 			$folder = FileExportHelpers::GetBijouxOfDefuntFolder($devisOfDefunt["client_nom"],$bijoux["defunt_nom"],$currentConfig->path);
+			$appAdminStorage = $this->rootFolder->getUserFolder(BddConstant::DEFAULT_ADMIN_APP_ID_NEXTCLOUD);
 			try {
 				$this->storage->newFolder($folder);
 				$this->adminStorage->newFolder($folder);
-				$appAdminStorage = $this->rootFolder->getUserFolder(BddConstant::DEFAULT_ADMIN_APP_ID_NEXTCLOUD);
 				$appAdminStorage->newFolder($folder);
             } 
             catch(\OCP\Files\NotPermittedException $e) {
@@ -2830,10 +2830,10 @@ class PageController extends Controller {
 				return null;
 			}
 			$folder = FileExportHelpers::GetPacemakerPhotoFolderOfDefunt($devisOfDefunt["client_nom"],$devisOfDefunt["defunt_nom"],$currentConfig->path);
+			$appAdminStorage = $this->rootFolder->getUserFolder(BddConstant::DEFAULT_ADMIN_APP_ID_NEXTCLOUD);
 			try {
 				$this->storage->newFolder($folder);
 				$this->adminStorage->newFolder($folder);
-				$appAdminStorage = $this->rootFolder->getUserFolder(BddConstant::DEFAULT_ADMIN_APP_ID_NEXTCLOUD);
 				$appAdminStorage->newFolder($folder);
             } 
             catch(\OCP\Files\NotPermittedException $e) {

gestion/src/js/listener/invoiceListener.js

@@ -69,9 +69,10 @@ $('body').on('click', '#payInvoice', function () {
             let datatable = new DataTable('.tabledt');
             Facture.loadFactureDT(datatable);
             showSuccess(t('gestion', "Factures payés avec succès"));
-            return;
         }
+        else{
             showError(t('gestion', "Erreur dans le paiement des factures"));
+        }
     }).fail(function (response, code) {
         $('#facturePaymentModal').hide();
         showError(t('gestion', "Erreur dans le paiement des factures"));
@@ -105,9 +106,10 @@ $('body').on('click', '#exportMultipleFactureToPdf', function () {
         var fileName = response;
         if(fileName != null){
             showSuccess('Sauvegardé dans' + fileName);
-            return;
         }
+        else{
             showError(t('gestion', "Les données pour sauvegarde sont vides"));
+        }
     }).fail(function (response, code) {
         showError(t('gestion', "Erreur dans la génération de facture multiple"));
     }).always(function () {

gestion/src/js/modules/ajaxRequest.mjs

@@ -815,12 +815,13 @@ export function exportDevisToPdf(devisId) {
     }).done(function (response) {
         if(response == null) {
             showMessage('Le devis n\'existe pas');
-            return;
         }
+        else{
             var fileNames = JSON.parse(response);
             fileNames.forEach(fileName => {
                 showSuccess('Sauvegardé dans' + fileName);
             });
+        }
     }).fail(function (response, code) {
         showMessage(t('gestion', 'Erreur dans l\'export du devis en PDF'));
     }).always(function () {
@@ -848,9 +849,10 @@ export function exportClientDevisByMonthAndYearToPdf(clientId,year,month) {
     }).done(function (response) {
         if(response != null) {
             showSuccess('Sauvegardé dans' + response);
-            return;
         }
+        else{
             showError(t('gestion', "Les données pour sauvegarde sont vides"));
+        }
     }).fail(function (response, code) {
         showError(t('gestion', "Erreur dans la génération de devis multiple"));
     }).always(function () {
@@ -880,9 +882,10 @@ export function setBijouxPhoto(bijouxId,file) {
             if(response != null) {
                 showSuccess('Sauvegardé dans' + response);
                 getBijouxById();
-                return;
             }
+            else{
                 showError(t('gestion', "Erreur dans l'assignation de photo sur ce bijou"));
+            }
         }).fail(function (response, code) {
             showError(t('gestion', "Erreur dans la génération de devis multiple"));
         }).always(function () {
@@ -891,6 +894,7 @@ export function setBijouxPhoto(bijouxId,file) {
     }
     else{
         showError(t('gestion', "Veuillez choisir une image"))
+        hideLoader();
     }
 };
 
@@ -916,9 +920,10 @@ export function setDefuntPacemakerPhoto(defuntId,file) {
                 showSuccess('Sauvegardé dans' + response);
                 let pacemakerPhotoLabel =  document.getElementById("pacemakerPhotoLabel");
                 pacemakerPhotoLabel.innerHTML = file.name;
-                return;
             }
+            else{
                 showError(t('gestion', "Le défunt n'appartient à aucun devis"));
+            }
         }).fail(function (response, code) {
             showError(t('gestion', "Le défunt n'appartient à aucun devis"));
         }).always(function () {
@@ -926,6 +931,7 @@ export function setDefuntPacemakerPhoto(defuntId,file) {
         });
     }
     else{
-        showError(t('gestion', "Veuillez choisir une photo"))
+        showError(t('gestion', "Veuillez choisir une photo"));
+        hideLoader();
     }
 };