Evita articoli persi o doppi con griglia e scanner insieme

Nella scheda, scegliendo alcuni articoli dalla griglia e sparandone altri,
non tutti i barcode arrivavano al salvataggio.

- La selezione dalla griglia modificava Scheda.Articoli da un thread del
  pool, mentre le scansioni lo fanno sul thread della UI: le due modifiche
  potevano sovrapporsi e far perdere articoli. Ora passa da InvokeAsync.
- La griglia non scartava i barcode gia' sparati, e lo scanner controllava
  solo il codice letto, non quello restituito dal server (che puo' essere
  il barcode principale, lo stesso della griglia). Un doppione rompe il
  @key della lista e il salvataggio locale (indice unico SchedaId+Barcode).
  Ora entrambi i percorsi aggiungono solo barcode non ancora presenti.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

SteUp.Data/LocalDb/EntityServices/IspezioniService.cs

@@ -155,14 +155,29 @@ public class IspezioniService(AppDbContext db) : IIspezioniService
         await db.SaveChangesAsync();
     }
 
-    public Task<List<Scheda>> GetAllSchedeOfIspezioneAsync(string codMdep, DateTime data, string rilevatore) =>
-        db.Schede
+    public async Task<List<Scheda>> GetAllSchedeOfIspezioneAsync(string codMdep, DateTime data, string rilevatore)
+    {
+        // Le versioni precedenti dell'app hanno lasciato schede non bozza senza
+        // reparto o motivo: risultano inviabili ma il server le rifiuta, e
+        // bloccano la conclusione dell'ispezione. Tornano bozze, cosi' il
+        // rilevatore le completa. Quelle gia' inviate non si toccano.
+        await db.Schede
+            .Where(x => x.CodMdep == codMdep &&
+                        x.Data == data &&
+                        x.Rilevatore == rilevatore &&
+                        !x.Bozza &&
+                        x.ActivityId == null &&
+                        (string.IsNullOrEmpty(x.CodJfas) || string.IsNullOrEmpty(x.ActivityTypeId)))
+            .ExecuteUpdateAsync(s => s.SetProperty(x => x.Bozza, true));
+
+        return await db.Schede
             .AsNoTracking()
             .Include(s => s.Articoli)
             .Where(x => x.CodMdep == codMdep &&
                         x.Data == data &&
                         x.Rilevatore == rilevatore)
             .ToListAsync();
+    }
 
     public Task<Scheda?> GetSchedaAsync(int schedaId) =>
         db.Schede

SteUp.Maui/Core/Services/AttachedService.cs

@@ -8,12 +8,21 @@ namespace SteUp.Maui.Core.Services;
 
 public class AttachedService(ILogger<FileManager> logger) : IAttachedService
 {
-    public async Task<AttachedDto?> SelectImageFromCamera()
+    public bool IsCameraAvailable => MediaPicker.Default.IsCaptureSupported;
+
+    public async Task<bool> RequestCameraPermissions()
     {
         var cameraPerm = await Permissions.RequestAsync<Permissions.Camera>();
         var storagePerm = await Permissions.RequestAsync<Permissions.StorageWrite>();
 
-        if (cameraPerm != PermissionStatus.Granted || storagePerm != PermissionStatus.Granted)
+        return cameraPerm == PermissionStatus.Granted && storagePerm == PermissionStatus.Granted;
+    }
+
+    public void OpenAppSettings() => AppInfo.Current.ShowSettingsUI();
+
+    public async Task<AttachedDto?> SelectImageFromCamera()
+    {
+        if (!await RequestCameraPermissions())
             return null;
 
         FileResult? result;
@@ -28,7 +37,10 @@ public class AttachedService(ILogger<FileManager> logger) : IAttachedService
             logger.LogError(ex, ex.Message);
             Console.WriteLine($"Errore cattura foto: {ex.Message}");
             SentrySdk.CaptureException(ex);
-            return null;
+
+            // Rilancia: restituire null qui era indistinguibile da un annullamento,
+            // e il rilevatore toccava "Aggiungi una foto" senza vedere nulla.
+            throw;
         }
 
         return result is null ? null : await UtilityFile.ConvertToDto(result, AttachedDto.TypeAttached.Image);

SteUp.Maui/Platforms/Android/AndroidManifest.xml

@@ -25,4 +25,11 @@
     <uses-permission android:name="android.permission.READ_MEDIA_IMAGES" />
     <uses-permission android:name="android.permission.READ_MEDIA_VIDEO" />
     <uses-permission android:name="android.permission.READ_MEDIA_AUDIO" />
+
+    <!-- Android 11+: rende visibile l'app fotocamera di sistema a MediaPicker -->
+    <queries>
+        <intent>
+            <action android:name="android.media.action.IMAGE_CAPTURE" />
+        </intent>
+    </queries>
 </manifest>

SteUp.Maui/SteUp.Maui.csproj

@@ -30,8 +30,8 @@
         <ApplicationId>it.integry.SteUp</ApplicationId>
 
         <!-- Versions -->
-        <ApplicationDisplayVersion>1.2.1</ApplicationDisplayVersion>
-        <ApplicationVersion>7</ApplicationVersion>
+        <ApplicationDisplayVersion>1.2.3</ApplicationDisplayVersion>
+        <ApplicationVersion>9</ApplicationVersion>
 
         <SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'ios'">15.0</SupportedOSPlatformVersion>
         <!--		<SupportedOSPlatformVersion Condition="$([MSBuild]::GetTargetPlatformIdentifier('$(TargetFramework)')) == 'maccatalyst'">-->
@@ -116,12 +116,12 @@
     <ItemGroup>
         <PackageReference Include="CommunityToolkit.Maui" Version="14.1.0" />
         <PackageReference Include="CommunityToolkit.Mvvm" Version="8.4.2" />
-        <PackageReference Include="IntegryApiClient.MAUI" Version="2.0.1" />
-        <PackageReference Include="Microsoft.Extensions.Logging" Version="10.0.7" />
+        <PackageReference Include="IntegryApiClient.MAUI" Version="2.6.0" />
+        <PackageReference Include="Microsoft.Extensions.Logging" Version="10.0.8" />
         <PackageReference Include="Microsoft.Maui.Controls" Version="10.0.51" />
         <PackageReference Include="Microsoft.Maui.Controls.Compatibility" Version="10.0.51" />
         <PackageReference Include="Microsoft.AspNetCore.Components.WebView.Maui" Version="10.0.51" />
-        <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="10.0.7" />
+        <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="10.0.8" />
         <PackageReference Include="Sentry.Maui" Version="6.4.1" />
         <PackageReference Include="SkiaSharp" Version="3.119.2" />
     </ItemGroup>

SteUp.Shared/Components/Pages/IspezionePage.razor

@@ -157,6 +157,7 @@
             @foreach (var group in SchedeGrouped)
             {
                 var reparto = group.Key;
+                var senzaReparto = ReferenceEquals(reparto, SenzaReparto);
                 var chiuso = IsCollapsed(reparto);
                 var daInviare = group.Value.CountDaInviare();
 
@@ -172,7 +173,9 @@
                         </button>
 
                         <div class="reparto__actions">
-                            @if (!IsCompleted)
+                            @* Il gruppo "Senza reparto" raccoglie solo bozze da
+                               sistemare: non ha un "+" ne' un invio suo. *@
+                            @if (!IsCompleted && !senzaReparto)
                             {
                                 @* Da qui la scheda nasce gia' con il suo reparto: e'
                                    l'unica differenza con il "+" della barra. *@
@@ -326,13 +329,15 @@
     // schermata, non del documento.
     private readonly HashSet<string> _collapsed = new(StringComparer.OrdinalIgnoreCase);
 
-    private bool IsCollapsed(JtbFasiDto reparto) =>
-        reparto.CodJfas is not null && _collapsed.Contains(reparto.CodJfas);
+    // Il gruppo "Senza reparto" non ha codice: la chiave vuota e' la sua.
+    private static string CollapseKey(JtbFasiDto reparto) => reparto.CodJfas ?? string.Empty;
+
+    private bool IsCollapsed(JtbFasiDto reparto) => _collapsed.Contains(CollapseKey(reparto));
 
     private void Toggle(JtbFasiDto reparto)
     {
-        if (reparto.CodJfas is null) return;
-        if (!_collapsed.Remove(reparto.CodJfas)) _collapsed.Add(reparto.CodJfas);
+        var key = CollapseKey(reparto);
+        if (!_collapsed.Remove(key)) _collapsed.Add(key);
     }
 
     private static string Conteggi(List<Scheda> schede)
@@ -542,15 +547,22 @@
             StateHasChanged();
         });
 
+    // Una bozza si puo' salvare anche senza reparto: senza questo gruppo
+    // sparirebbe dall'elenco pur restando sul dispositivo e contata fra le bozze.
+    private static readonly JtbFasiDto SenzaReparto = new() { Descrizione = "Senza reparto" };
+
     private void GroupSchede()
     {
-        SchedeGrouped = SchedeList
-            .Where(s => s.Reparto != null)
-            .GroupBy(s => s.CodJfas)
-            .ToDictionary(
-                g => g.First().Reparto!,
-                g => g.ToList()
-            );
+        var grouped = new Dictionary<JtbFasiDto, List<Scheda>>();
+
+        // In testa: sono schede a meta' che chiedono di essere completate.
+        var senzaReparto = SchedeList.FindAll(s => s.Reparto == null);
+        if (senzaReparto.Count > 0) grouped[SenzaReparto] = senzaReparto;
+
+        foreach (var g in SchedeList.Where(s => s.Reparto != null).GroupBy(s => s.CodJfas))
+            grouped[g.First().Reparto!] = g.ToList();
+
+        SchedeGrouped = grouped;
     }
 
     private void OnSchedaModified(Scheda obj)

SteUp.Shared/Components/SingleElements/Card/SchedaCard.razor

@@ -16,7 +16,7 @@
 <div class="list-card" aria-label="@SemanticLabel">
     <div class="list-card__head">
         <div class="list-card__ident">
-            <div class="t-ident">@Scheda.DescrizioneReparto</div>
+            <div class="t-ident">@(Scheda.DescrizioneReparto ?? "Senza reparto")</div>
             <h3 class="list-card__title">@Titolo</h3>
         </div>
     </div>
@@ -125,7 +125,7 @@
     };
 
     private string SemanticLabel =>
-        $"Scheda {Titolo}, reparto {Scheda.DescrizioneReparto}, {SyncState.ToBadge().Label}, {ArticoliLabel}, {FotoLabel}";
+        $"Scheda {Titolo}, reparto {Scheda.DescrizioneReparto ?? "non indicato"},{SyncState.ToBadge().Label}, {ArticoliLabel}, {FotoLabel}";
 
     private async Task SetBusy(bool value)
     {

SteUp.Shared/Components/SingleElements/Modal/ModalAddAttached.razor

@@ -40,7 +40,32 @@
 
     private async Task OnCamera()
     {
-        var selectImageFromCamera = await AttachedService.SelectImageFromCamera();
+        if (!await AttachedService.RequestCameraPermissions())
+        {
+            Snackbar.Clear();
+            Snackbar.Add("Per scattare le foto serve il permesso della fotocamera", Severity.Warning, config =>
+            {
+                config.Action = "Impostazioni";
+                config.OnClick = _ =>
+                {
+                    AttachedService.OpenAppSettings();
+                    return Task.CompletedTask;
+                };
+            });
+            return;
+        }
+
+        AttachedDto? selectImageFromCamera;
+        try
+        {
+            selectImageFromCamera = await AttachedService.SelectImageFromCamera();
+        }
+        catch (Exception)
+        {
+            Snackbar.Clear();
+            Snackbar.Add("Non è stato possibile scattare la foto, riprova", Severity.Error);
+            return;
+        }
 
         if (selectImageFromCamera != null)
         {

SteUp.Shared/Components/SingleElements/Modal/ModalFormScheda.razor

@@ -754,12 +754,60 @@
 
     #region File
 
+    // Dopo un rifiuto definitivo Android non ripropone la richiesta: il
+    // permesso si riattiva solo dalle impostazioni, quindi le apriamo noi.
+    private void ShowCameraPermissionDenied()
+    {
+        Snackbar.Clear();
+        Snackbar.Add("Per scattare le foto serve il permesso della fotocamera", Severity.Warning, config =>
+        {
+            config.Action = "Impostazioni";
+            config.OnClick = _ =>
+            {
+                AttachedService.OpenAppSettings();
+                return Task.CompletedTask;
+            };
+        });
+    }
+
     private async Task OpenAddAttached()
     {
-        var selectImageFromCamera = await AttachedService.SelectImageFromCamera();
-        if (selectImageFromCamera == null) return;
+        List<AttachedDto>? attachedList;
 
-        var attachedList = new List<AttachedDto> { selectImageFromCamera };
+        if (AttachedService.IsCameraAvailable)
+        {
+            if (!await AttachedService.RequestCameraPermissions())
+            {
+                ShowCameraPermissionDenied();
+                return;
+            }
+
+            AttachedDto? selectImageFromCamera;
+            try
+            {
+                selectImageFromCamera = await AttachedService.SelectImageFromCamera();
+            }
+            catch (Exception)
+            {
+                // Gia' registrato e inviato a Sentry dal servizio
+                Snackbar.Clear();
+                Snackbar.Add("Non è stato possibile scattare la foto, riprova", Severity.Error);
+                return;
+            }
+
+            if (selectImageFromCamera == null) return;
+            attachedList = [selectImageFromCamera];
+        }
+        else
+        {
+            // Senza fotocamera "Aggiungi una foto" non avrebbe altra strada:
+            // si passa alla galleria invece di lasciare il rilevatore fermo.
+            Snackbar.Clear();
+            Snackbar.Add("Fotocamera non disponibile: scegli la foto dalla galleria", Severity.Warning);
+
+            attachedList = await AttachedService.SelectImageFromGallery();
+            if (attachedList is null || attachedList.Count == 0) return;
+        }
 
         OnLoading = true;
         await InvokeAsync(StateHasChanged);
@@ -848,29 +896,32 @@
 
             var modal = await WithChildDialog(() => Dialog.OpenSelectArt(articoli));
 
+            // Gli articoli si aggiungono sul thread della UI, come fanno le scansioni:
+            // toccare Scheda.Articoli da qui mentre arriva una lettura (o mentre la
+            // lista viene disegnata) puo' far perdere articoli.
             await InvokeAsync(() =>
             {
                 OnLoading = false;
                 TextLoading = null;
 
+                if (modal is { Canceled: false, Data: List<ArticoliInGrigliaDto> articoliSelezionati })
+                {
+                    // Quelli gia' sparati non si aggiungono di nuovo.
+                    Scheda.Articoli.InsertRange(0, articoliSelezionati
+                        .DistinctBy(x => x.Barcode)
+                        .Where(x => !ContainsBarcode(x.Barcode))
+                        .Select(x => new SchedaArticolo
+                        {
+                            Barcode = x.Barcode,
+                            Descrizione = x.Descrizione
+                        })
+                    );
+
+                    RecalcDirty();
+                }
+
                 StateHasChanged();
             });
-
-            if (modal is { Canceled: false, Data: List<ArticoliInGrigliaDto> articoliSelezionati })
-            {
-                Scheda.Articoli.InsertRange(0, articoliSelezionati.ConvertAll(x => new SchedaArticolo
-                    {
-                        Barcode = x.Barcode,
-                        Descrizione = x.Descrizione
-                    })
-                );
-
-                await InvokeAsync(() =>
-                {
-                    RecalcDirty();
-                    StateHasChanged();
-                });
-            }
         });
     }
 
@@ -936,7 +987,7 @@
         try
         {
             if (value is null) return;
-            if (Scheda.Articoli.Any(x => x.Barcode.Equals(value))) return;
+            if (ContainsBarcode(value)) return;
 
             await InvokeAsync(() =>
             {
@@ -951,6 +1002,11 @@
                 OnLoading = false;
                 StateHasChanged();
 
+                // Il controllo va ripetuto sul barcode restituito: il server puo' rispondere
+                // con un barcode diverso da quello letto (lo stesso gia' scelto dalla griglia),
+                // e due letture ravvicinate superano entrambe il controllo iniziale.
+                if (art != null && ContainsBarcode(art.Barcode)) return;
+
                 if (art != null)
                 {
                     Scheda.Articoli.Insert(0, new SchedaArticolo
@@ -984,6 +1040,8 @@
 
     private void OnErrorScan(string? value) => OnError(new Exception(value), value);
 
+    private bool ContainsBarcode(string barcode) => Scheda.Articoli.Any(x => x.Barcode.Equals(barcode));
+
     #endregion
 
     private void OnError(Exception? e, string? errorMessage)

SteUp.Shared/Core/Interface/System/IAttachedService.cs

@@ -4,6 +4,14 @@ namespace SteUp.Shared.Core.Interface.System;
 
 public interface IAttachedService
 {
+    bool IsCameraAvailable { get; }
+
+    /// <summary>Chiede i permessi per scattare; false se negati.</summary>
+    Task<bool> RequestCameraPermissions();
+
+    void OpenAppSettings();
+
+    /// <summary>Null se l'utente annulla; lancia se lo scatto fallisce.</summary>
     Task<AttachedDto?> SelectImageFromCamera();
     Task<List<AttachedDto>?> SelectImageFromGallery();
 }

SteUp.Shared/SteUp.Shared.csproj

@@ -11,13 +11,13 @@
     </ItemGroup>
 
     <ItemGroup>
-        <PackageReference Include="IntegryApiClient.Core" Version="2.0.1" />
+        <PackageReference Include="IntegryApiClient.Core" Version="2.6.0" />
         <PackageReference Include="Microsoft.AspNetCore.Components.Web" Version="10.0.7" />
         <PackageReference Include="CommunityToolkit.Mvvm" Version="8.4.2" />
         <PackageReference Include="MudBlazor" Version="9.0.0" />
         <PackageReference Include="CodeBeam.MudBlazor.Extensions" Version="9.0.1" />
         <PackageReference Include="Sentry" Version="6.4.1" />
-        <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.17.0" />
+        <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.18.0" />
         <PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="10.0.7" />
         <PackageReference Include="Microsoft.Maui.Essentials" Version="10.0.51" />
     </ItemGroup>

SteUp.Web/SteUp.Web.csproj

@@ -7,7 +7,7 @@
     </PropertyGroup>
 
     <ItemGroup>
-        <PackageReference Include="IntegryApiClient.Blazor" Version="2.0.1" />
+        <PackageReference Include="IntegryApiClient.Blazor" Version="2.6.0" />
         <PackageReference Include="Microsoft.AspNetCore.Components.Authorization" Version="10.0.7" />
         <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly" Version="10.0.7" />
         <PackageReference Include="Microsoft.AspNetCore.Components.WebAssembly.DevServer" Version="10.0.7" />