From c4c624667d0bc48e7016d2dd5f518e5bbfb991bd Mon Sep 17 00:00:00 2001
From: MarcoE <m.elefante@integry.it>
Date: Wed, 4 Nov 2026 13:39:31 +0100
Subject: [PATCH 1/2] Permette di eliminare le foto di una scheda anche dopo il
 salvataggio

Finora la foto si poteva togliere solo prima di salvare la scheda.

- La X compare su tutte le foto (non in sola lettura, non mentre la foto e'
  ancora in elaborazione). L'eliminazione avviene al salvataggio: uscendo
  senza salvare le foto restano.
- Una foto gia' inviata si toglie prima dal server (nuovo endpoint
  steup/removeAttachment), poi dal dispositivo e da Scheda.ImageNames. Senza
  rete la X avvisa e non la toglie; se il server non conferma al salvataggio
  la foto resta ovunque e compare un avviso.
- Una foto salvata ma non ancora inviata si elimina dal dispositivo (con la
  rete si prova anche a toglierla dal server, senza bloccare).
- Togliere una foto salvata non la cancellava mai: il flag ToRemove finiva
  su un elemento appena rimosso dall'elenco. Ora le foto tolte stanno in una
  lista a parte fino al salvataggio; ToRemove e' stato eliminato.
- FileManager.GetInspectionFiles: le foto in /toUpload si filtravano per
  nome, ma ImageNames contiene il percorso completo, quindi quelle salvate
  senza rete non ricomparivano mai riaprendo la scheda. Ora si accetta sia il
  percorso sia il nome, e il DTO porta anche i byte del file per l'invio.

Richiede i Servizi con steup/removeAttachment.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 SteUp.Maui/Core/Services/FileManager.cs       |  12 +-
 .../Modal/ModalFormScheda.razor               | 132 ++++++++++++++----
 SteUp.Shared/Core/Dto/AttachedDto.cs          |   1 -
 .../IntegryApi/IIntegrySteupService.cs        |   1 +
 .../Core/Services/IntegrySteupService.cs      |  14 ++
 5 files changed, 130 insertions(+), 30 deletions(-)

diff --git a/SteUp.Maui/Core/Services/FileManager.cs b/SteUp.Maui/Core/Services/FileManager.cs
index 9764249..29bbece 100644
--- a/SteUp.Maui/Core/Services/FileManager.cs
+++ b/SteUp.Maui/Core/Services/FileManager.cs
@@ -16,15 +16,15 @@ public class FileManager(
 
     private async Task<AttachedDto> ConvertToDto(FileInfo file, AttachedDto.TypeAttached type, bool isFromToUpload)
     {
-        var (origUrl, thumbUrl) = await SaveAndCreateThumbAsync(
-            await File.ReadAllBytesAsync(file.FullName),
-            file.Name
-        );
+        var bytes = await File.ReadAllBytesAsync(file.FullName);
+        var (origUrl, thumbUrl) = await SaveAndCreateThumbAsync(bytes, file.Name);
 
         return new AttachedDto
         {
             Name = file.Name,
             Path = file.FullName,
+            // Servono per inviare le foto rimaste in /toUpload (salvate senza rete).
+            FileBytes = bytes,
             TempPath = origUrl,
             ThumbPath = thumbUrl,
             Type = type,
@@ -101,7 +101,9 @@ public class FileManager(
 
         foreach (var file in toUploadFiles)
         {
-            if (!fileNameFilter.Contains(file.Name))
+            // Scheda.ImageNames contiene il percorso completo restituito da SaveInspectionFile:
+            // confrontando solo il nome le foto salvate e non ancora inviate non comparivano mai.
+            if (!fileNameFilter.Contains(file.FullName) && !fileNameFilter.Contains(file.Name))
                 continue;
 
             ct.ThrowIfCancellationRequested();
diff --git a/SteUp.Shared/Components/SingleElements/Modal/ModalFormScheda.razor b/SteUp.Shared/Components/SingleElements/Modal/ModalFormScheda.razor
index f1c9f65..f62103a 100644
--- a/SteUp.Shared/Components/SingleElements/Modal/ModalFormScheda.razor
+++ b/SteUp.Shared/Components/SingleElements/Modal/ModalFormScheda.razor
@@ -171,7 +171,10 @@
                                                 </div>
                                             }
 
-                                            @if (item.p.ToUpload && !ReadOnly)
+                                            @* Rimovibile anche se gia' salvata o inviata. Non mentre e'
+                                               in elaborazione: l'indice della foto serve ancora al
+                                               caricamento in background. *@
+                                            @if (!ReadOnly && !item.p.ThumbPath.IsNullOrEmpty())
                                             {
                                                 <button type="button" class="attach-item__remove"
                                                         aria-label="@($"Rimuovi la foto {item.p.Name}")"
@@ -336,6 +339,9 @@
     // Gli allegati vivono in AttachedList finche' non si salva: li tracciamo con un flag.
     private bool _attachmentsDirty;
     private List<AttachedDto>? AttachedList { get; set; }
+    // Foto gia' salvate tolte dal rilevatore: si eliminano davvero solo al salvataggio,
+    // cosi' uscendo senza salvare restano dove sono.
+    private readonly List<AttachedDto> _removedAttachments = [];
 
     private IDisposable? _backRegistration;
     // Il tasto indietro puo' arrivare mentre il prompt di uscita e' gia' a video.
@@ -506,31 +512,30 @@
 
     private async Task Update(SaveSchedaResponseDto? apiResponse, bool preserveActivityId = false)
     {
+        var ispezione = new Ispezione
+        {
+            CodMdep = CodMdep,
+            Data = Data,
+            Rilevatore = UserSession.User.Username
+        };
+
+        // Prima di toccare l'ActivityId: le foto gia' inviate vanno tolte dall'attivita'
+        // su cui sono state caricate.
+        await RemoveDeletedAttachments(ispezione);
+
         if (!AttachedList.IsNullOrEmpty())
         {
             foreach (var attached in AttachedList!.Where(x => !x.SavedOnAppData))
             {
-                var ispezione = new Ispezione
-                {
-                    CodMdep = CodMdep,
-                    Data = Data,
-                    Rilevatore = UserSession.User.Username
-                };
+                var fileNameAdded = await FileManager.SaveInspectionFile(
+                    ispezione,
+                    attached.FileBytes!,
+                    attached.Name!
+                );
 
-                if (!attached.ToRemove)
-                {
-                    var fileNameAdded = await FileManager.SaveInspectionFile(
-                        ispezione,
-                        attached.FileBytes!,
-                        attached.Name!
-                    );
-
-                    Scheda.ImageNames ??= [];
-                    if (fileNameAdded != null)
-                        Scheda.ImageNames.Add(fileNameAdded);
-                }
-                else
-                    _ = FileManager.RemoveInspectionFile(ispezione, attached.Name!);
+                Scheda.ImageNames ??= [];
+                if (fileNameAdded != null)
+                    Scheda.ImageNames.Add(fileNameAdded);
             }
         }
 
@@ -549,6 +554,70 @@
         }
     }
 
+    /// <summary>
+    /// Elimina le foto salvate che il rilevatore ha tolto dalla scheda: dal server se
+    /// erano gia' state inviate, poi dal dispositivo e dall'elenco della scheda.
+    /// Se il server non conferma, la foto resta (sul server e sul dispositivo) invece
+    /// di sparire solo da qui.
+    /// </summary>
+    private async Task RemoveDeletedAttachments(Ispezione ispezione)
+    {
+        if (_removedAttachments.Count == 0) return;
+
+        var nonEliminate = 0;
+
+        foreach (var attached in _removedAttachments)
+        {
+            if (attached.Name.IsNullOrEmpty()) continue;
+
+            var giaInviata = !attached.ToUpload;
+
+            // Una foto ancora "da inviare" di norma non e' sul server, ma l'invio dalla
+            // card o dalla chiusura ispezione non la sposta fra le inviate: con la rete
+            // si prova comunque a toglierla, senza bloccare l'eliminazione se non c'e'.
+            if (Scheda.ActivityId.IsValorized() && (giaInviata || NetworkService.ConnectionAvailable))
+            {
+                try
+                {
+                    await IntegrySteupService.RemoveFile(Scheda.ActivityId!, attached.Name!);
+                }
+                catch (Exception e)
+                {
+                    if (giaInviata)
+                    {
+                        Logger.LogError(e, "Eliminazione della foto {FileName} dal server non riuscita", attached.Name);
+                        nonEliminate++;
+                        continue;
+                    }
+
+                    Logger.LogWarning(e, "Foto {FileName} non trovata sul server", attached.Name);
+                }
+            }
+
+            try
+            {
+                FileManager.RemoveInspectionFile(ispezione, attached.Name!);
+            }
+            catch (Exception e)
+            {
+                // Il file orfano non fa danni: conta che la foto esca dalla scheda.
+                Logger.LogError(e, "Eliminazione del file {FileName} non riuscita", attached.Name);
+            }
+
+            Scheda.ImageNames?.RemoveAll(x => System.IO.Path.GetFileName(x) == attached.Name);
+        }
+
+        _removedAttachments.Clear();
+
+        if (nonEliminate > 0)
+        {
+            Snackbar.Add(nonEliminate == 1
+                    ? "Una foto già inviata non è stata eliminata: riprova con la connessione"
+                    : $"{nonEliminate} foto già inviate non sono state eliminate: riprova con la connessione",
+                Severity.Warning);
+        }
+    }
+
     private async Task UploadFile(Scheda scheda)
     {
         if (AttachedList.IsNullOrEmpty()) return;
@@ -703,7 +772,8 @@
 
     /// <summary>
     /// Salva il lavoro in corso solo sul dispositivo, marcato come da completare.
-    /// Niente chiamate al server e niente validazione: una bozza può essere incompleta.
+    /// Niente salvataggio sul server e niente validazione: una bozza può essere incompleta.
+    /// Unica eccezione: le foto già inviate tolte dal rilevatore si eliminano anche dal server.
     /// </summary>
     private async Task SaveDraft()
     {
@@ -929,8 +999,22 @@
     {
         if (AttachedList is null || index < 0 || index >= AttachedList.Count) return;
 
-        if (AttachedList[index].SavedOnAppData)
-            AttachedList[index].ToRemove = true;
+        var attached = AttachedList[index];
+
+        // Le foto appena scattate stanno solo in memoria: basta toglierle dall'elenco.
+        if (attached.SavedOnAppData)
+        {
+            // Una foto gia' inviata va tolta anche dal server: senza rete resterebbe
+            // nella scheda inviata pur sparendo dal dispositivo.
+            if (!attached.ToUpload && !NetworkService.ConnectionAvailable)
+            {
+                Snackbar.Clear();
+                Snackbar.Add("Per eliminare una foto già inviata serve la connessione", Severity.Warning);
+                return;
+            }
+
+            _removedAttachments.Add(attached);
+        }
 
         AttachedList.RemoveAt(index);
         _attachmentsDirty = true;
diff --git a/SteUp.Shared/Core/Dto/AttachedDto.cs b/SteUp.Shared/Core/Dto/AttachedDto.cs
index 60a747b..98357b6 100644
--- a/SteUp.Shared/Core/Dto/AttachedDto.cs
+++ b/SteUp.Shared/Core/Dto/AttachedDto.cs
@@ -15,7 +15,6 @@ public class AttachedDto
     public string? ThumbPath { get; set; }
     
     public bool SavedOnAppData { get; set; }
-    public bool ToRemove { get; set; }
     public bool ToUpload { get; set; }
 
     public Stream? FileContent =>
diff --git a/SteUp.Shared/Core/Interface/IntegryApi/IIntegrySteupService.cs b/SteUp.Shared/Core/Interface/IntegryApi/IIntegrySteupService.cs
index 1fcfbbc..c46bc09 100644
--- a/SteUp.Shared/Core/Interface/IntegryApi/IIntegrySteupService.cs
+++ b/SteUp.Shared/Core/Interface/IntegryApi/IIntegrySteupService.cs
@@ -17,6 +17,7 @@ public interface IIntegrySteupService
     Task<SaveSchedaResponseDto?> SaveMultipleSchede(List<SaveRequestDto> request);
     Task CompleteInspection(string activityId);
     Task UploadFile(string activityId, byte[] file, string fileName);
+    Task RemoveFile(string activityId, string fileName);
     
     Task DeleteScheda(string activityId);
 }
\ No newline at end of file
diff --git a/SteUp.Shared/Core/Services/IntegrySteupService.cs b/SteUp.Shared/Core/Services/IntegrySteupService.cs
index 5160af4..30ebacc 100644
--- a/SteUp.Shared/Core/Services/IntegrySteupService.cs
+++ b/SteUp.Shared/Core/Services/IntegrySteupService.cs
@@ -70,6 +70,20 @@ public class IntegrySteupService(IIntegryApiRestClient integryApiRestClient) : I
         content.Dispose();
     }
 
+    /// <summary>
+    /// Toglie un allegato dalla scheda. Il server lo riconosce dal nome con cui
+    /// è stato caricato da <see cref="UploadFile"/>.
+    /// </summary>
+    public Task RemoveFile(string activityId, string fileName) =>
+        integryApiRestClient.AuthorizedGet<object>(
+            $"{BaseRequest}/removeAttachment",
+            new Dictionary<string, object?>
+            {
+                { "activityId", activityId },
+                { "fileName", fileName }
+            }
+        );
+
     public Task DeleteScheda(string activityId) =>
         integryApiRestClient.AuthorizedGet<object>(
             $"{BaseRequest}/deleteScheda",

From e5cf4215f61163180c6ceb7f2e33c71f3ff22254 Mon Sep 17 00:00:00 2001
From: MarcoE <m.elefante@integry.it>
Date: Wed, 4 Nov 2026 15:53:51 +0100
Subject: [PATCH 2/2] -> v1.2.5 (11)

---
 SteUp.Maui/SteUp.Maui.csproj | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/SteUp.Maui/SteUp.Maui.csproj b/SteUp.Maui/SteUp.Maui.csproj
index 99fb7f2..d0a1c80 100644
--- a/SteUp.Maui/SteUp.Maui.csproj
+++ b/SteUp.Maui/SteUp.Maui.csproj
@@ -30,8 +30,8 @@
         <ApplicationId>it.integry.SteUp</ApplicationId>
 
         <!-- Versions -->
-        <ApplicationDisplayVersion>1.2.4</ApplicationDisplayVersion>
-        <ApplicationVersion>10</ApplicationVersion>
+        <ApplicationDisplayVersion>1.2.5</ApplicationDisplayVersion>
+        <ApplicationVersion>11</ApplicationVersion>
 
         <SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'ios'">15.0</SupportedOSPlatformVersion>
         <!--		<SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'maccatalyst'">-->