feat-add-files-from-qr (#8)

Reviewed-on: http://catelliub.zapto.org:3000/brontomark/flux/pulls/8
Co-authored-by: Mark M2 Macbook <marco@catelli.it>
Co-committed-by: Mark M2 Macbook <marco@catelli.it>

lib/features/services/blocs/service_files_bloc.dart

@@ -0,0 +1,232 @@
+import 'dart:async';
+import 'dart:io';
+
+import 'package:file_picker/file_picker.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:equatable/equatable.dart';
+import 'package:flux/core/utils/string_extensions.dart';
+import 'package:flux/features/services/data/services_repository.dart';
+import 'package:flux/features/services/models/service_file_model.dart';
+import 'package:flux/features/services/models/service_model.dart';
+import 'package:get_it/get_it.dart';
+
+part 'service_files_events.dart';
+part 'service_files_state.dart';
+
+class ServiceFilesBloc extends Bloc<ServiceFilesEvent, ServiceFilesState> {
+  final _repository = GetIt.I.get<ServicesRepository>();
+  final String? serviceId;
+
+  ServiceFilesBloc({this.serviceId})
+    : super(
+        ServiceFilesState(
+          status: ServiceFilesStatus.initial,
+          serviceId: serviceId,
+        ),
+      ) {
+    on<ServiceSavedEvent>(_onServiceSaved);
+    on<LoadServiceFilesEvent>(_onLoadServiceFiles);
+    on<AddServiceFilesEvent>(_onAddServiceFiles);
+    on<UploadServiceFilesEvent>(_onUploadServiceFiles);
+    on<UploadMultipleServiceFilesEvent>(_onUploadMultipleServiceFiles);
+    on<DeleteServiceFilesEvent>(_onDeleteServiceFiles);
+    on<ToggleServiceFileSelectionEvent>(_onToggleServiceFileSelection);
+    // Se il BLoC nasce con un ID, accendiamo subito lo stream!
+    if (serviceId != null) {
+      add(LoadServiceFilesEvent(serviceId: serviceId));
+    }
+  }
+
+  FutureOr<void> _onServiceSaved(
+    ServiceSavedEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) {
+    // 1. Aggiorniamo l'ID nello stato
+    // 2. PIALLIAMO i file locali: ormai sono partiti per Supabase!
+    // Così la UI si pulisce all'istante e aspetta quelli remoti.
+    emit(
+      state.copyWith(
+        serviceId: event.serviceId,
+        localFiles: [], // <-- LA MAGIA ANTI-DUPLICATI
+      ),
+    );
+
+    // Lanciamo il caricamento
+    add(LoadServiceFilesEvent(serviceId: event.serviceId));
+  }
+
+  FutureOr<void> _onLoadServiceFiles(
+    LoadServiceFilesEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) async {
+    // Usiamo l'ID dell'evento, e se non c'è usiamo quello dello stato
+    final currentId = event.serviceId ?? state.serviceId;
+
+    if (currentId != null) {
+      emit(state.copyWith(status: ServiceFilesStatus.loading));
+
+      await emit.forEach(
+        _repository.getServiceFilesStream(
+          currentId,
+        ), // <-- Usiamo l'ID corretto!
+        onData: (data) => state.copyWith(
+          status: ServiceFilesStatus.success,
+          remoteFiles: data,
+        ),
+        onError: (error, stackTrace) => state.copyWith(
+          status: ServiceFilesStatus.failure,
+          error: error.toString(),
+        ),
+      );
+    }
+  }
+
+  void _onAddServiceFiles(
+    AddServiceFilesEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) async {
+    final currentId = state.serviceId;
+    // BIVIO 1: PRATICA NUOVA (Nessun ID)
+    if (currentId == null) {
+      // Mettiamo i file nel "parcheggio" locale dello State
+      final newLocalFiles = event.files.map((file) {
+        return ServiceFileModel(
+          id: null,
+          serviceId: serviceId ?? '',
+          name: file.name.fileNameWithoutExtension(),
+          extension: file.name.fileExtension(),
+          storagePath: '',
+          fileSize: file.size,
+          localBytes: file.bytes,
+        );
+      }).toList();
+      final List<ServiceFileModel> updatedLocalFiles = [
+        ...state.localFiles,
+        ...newLocalFiles,
+      ];
+      emit(
+        state.copyWith(
+          localFiles: updatedLocalFiles,
+          status: ServiceFilesStatus.success,
+        ),
+      );
+      return;
+    }
+
+    // BIVIO 2: PRATICA ESISTENTE (Abbiamo l'ID)
+    emit(state.copyWith(status: ServiceFilesStatus.uploading));
+    try {
+      // Logica identica a quella che abbiamo fatto per i clienti
+      for (var file in event.files) {
+        await _repository.uploadAndRegisterServiceFile(
+          serviceId: serviceId!,
+          pickedFile: file,
+        );
+      }
+      emit(state.copyWith(status: ServiceFilesStatus.success));
+    } catch (e) {
+      emit(
+        state.copyWith(status: ServiceFilesStatus.failure, error: e.toString()),
+      );
+    }
+  }
+
+  FutureOr<void> _onUploadServiceFiles(
+    UploadServiceFilesEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) async {
+    if (event.pickedFiles == null && event.photos == null) return;
+    if (event.pickedFiles!.isEmpty && event.photos!.isEmpty) return;
+
+    // BIVIO 2: PRATICA ESISTENTE (Abbiamo l'ID
+    emit(state.copyWith(status: ServiceFilesStatus.uploading));
+    try {
+      // Logica identica a quella che abbiamo fatto per i clienti
+      if (event.pickedFiles != null && event.pickedFiles!.isNotEmpty) {
+        for (var file in event.pickedFiles!) {
+          await _repository.uploadAndRegisterServiceFile(
+            serviceId: state.serviceId!,
+            pickedFile: file,
+          );
+        }
+      }
+      emit(state.copyWith(status: ServiceFilesStatus.success));
+    } catch (e) {
+      emit(
+        state.copyWith(status: ServiceFilesStatus.failure, error: e.toString()),
+      );
+    }
+  }
+
+  FutureOr<void> _onUploadMultipleServiceFiles(
+    UploadMultipleServiceFilesEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) async {
+    if (event.files.isEmpty) {
+      emit(
+        state.copyWith(
+          status: ServiceFilesStatus.failure,
+          error: "Nessun file selezionato",
+        ),
+      );
+      return;
+    }
+    emit(state.copyWith(status: ServiceFilesStatus.uploading, error: null));
+    try {
+      // 2. Creiamo una lista di "Promesse" (Futures) per il repository
+      final List<Future<void>> uploadTasks = [];
+      for (var file in event.files) {
+        // Aggiungiamo il task alla lista, ma NON usiamo await qui dentro!
+        uploadTasks.add(
+          _repository.uploadAndRegisterServiceFile(
+            serviceId: state.serviceId!,
+            pickedFile: file,
+          ),
+        );
+      }
+      // 3. ESECUZIONE PARALLELA!
+      // Aspettiamo che tutti i file siano caricati contemporaneamente.
+      await Future.wait(uploadTasks);
+      // 4. GRAN FINALE: Tutto caricato, emettiamo il success!
+      emit(state.copyWith(status: ServiceFilesStatus.success));
+    } catch (e) {
+      // Se anche un solo file fallisce, catturiamo l'errore
+      emit(
+        state.copyWith(
+          status: ServiceFilesStatus.failure,
+          error: "Errore durante l'upload multiplo: $e",
+        ),
+      );
+    }
+  }
+
+  FutureOr<void> _onDeleteServiceFiles(
+    DeleteServiceFilesEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) async {
+    emit(state.copyWith(status: ServiceFilesStatus.loading));
+    try {
+      await _repository.deleteServiceFiles(state.selectedFiles);
+      emit(
+        state.copyWith(status: ServiceFilesStatus.success, selectedFiles: []),
+      );
+    } catch (e) {
+      emit(
+        state.copyWith(status: ServiceFilesStatus.failure, error: e.toString()),
+      );
+    }
+  }
+
+  FutureOr<void> _onToggleServiceFileSelection(
+    ToggleServiceFileSelectionEvent event,
+    Emitter<ServiceFilesState> emit,
+  ) {
+    List<ServiceFileModel> selectedFiles = List.from(state.selectedFiles);
+    if (selectedFiles.contains(event.file)) {
+      selectedFiles.remove(event.file);
+    } else {
+      selectedFiles.add(event.file);
+    }
+    emit(state.copyWith(selectedFiles: selectedFiles));
+  }
+}

lib/features/services/blocs/service_files_events.dart

@@ -0,0 +1,56 @@
+part of 'service_files_bloc.dart';
+
+abstract class ServiceFilesEvent extends Equatable {
+  const ServiceFilesEvent();
+
+  @override
+  List<Object?> get props => [];
+}
+
+class ServiceSavedEvent extends ServiceFilesEvent {
+  final String serviceId;
+  const ServiceSavedEvent(this.serviceId);
+
+  @override
+  List<Object?> get props => [serviceId];
+}
+
+class LoadServiceFilesEvent extends ServiceFilesEvent {
+  final String? serviceId;
+  final ServiceModel? service;
+  const LoadServiceFilesEvent({this.serviceId, this.service});
+
+  @override
+  List<Object?> get props => [serviceId, service];
+}
+
+class AddServiceFilesEvent extends ServiceFilesEvent {
+  final List<PlatformFile> files;
+  const AddServiceFilesEvent(this.files);
+
+  @override
+  List<Object?> get props => [files];
+}
+
+class UploadServiceFilesEvent extends ServiceFilesEvent {
+  final List<PlatformFile>? pickedFiles;
+  final List<File>? photos;
+  const UploadServiceFilesEvent({this.pickedFiles, this.photos});
+
+  @override
+  List<Object?> get props => [pickedFiles, photos];
+}
+
+class UploadMultipleServiceFilesEvent extends ServiceFilesEvent {
+  final List<PlatformFile> files;
+  const UploadMultipleServiceFilesEvent(this.files);
+  @override
+  List<Object?> get props => [files];
+}
+
+class DeleteServiceFilesEvent extends ServiceFilesEvent {}
+
+class ToggleServiceFileSelectionEvent extends ServiceFilesEvent {
+  final ServiceFileModel file;
+  const ToggleServiceFileSelectionEvent(this.file);
+}

lib/features/services/blocs/service_files_state.dart

@@ -0,0 +1,52 @@
+part of 'service_files_bloc.dart';
+
+enum ServiceFilesStatus { initial, loading, uploading, success, failure }
+
+class ServiceFilesState extends Equatable {
+  const ServiceFilesState({
+    this.serviceId,
+    required this.status,
+    this.error,
+    this.localFiles = const [],
+    this.remoteFiles = const [],
+    this.selectedFiles = const [],
+  });
+
+  final String? serviceId;
+  final ServiceFilesStatus status;
+  final String? error;
+  final List<ServiceFileModel> localFiles;
+  final List<ServiceFileModel> remoteFiles;
+
+  final List<ServiceFileModel> selectedFiles;
+
+  @override
+  List<Object?> get props => [
+    serviceId,
+    status,
+    error,
+    localFiles,
+    remoteFiles,
+    selectedFiles,
+  ];
+
+  List<ServiceFileModel> get allFiles => [...remoteFiles, ...localFiles];
+
+  ServiceFilesState copyWith({
+    String? serviceId,
+    ServiceFilesStatus? status,
+    String? error,
+    List<ServiceFileModel>? localFiles,
+    List<ServiceFileModel>? remoteFiles,
+    List<ServiceFileModel>? selectedFiles,
+  }) {
+    return ServiceFilesState(
+      serviceId: serviceId ?? this.serviceId,
+      status: status ?? this.status,
+      error: error,
+      localFiles: localFiles ?? this.localFiles,
+      remoteFiles: remoteFiles ?? this.remoteFiles,
+      selectedFiles: selectedFiles ?? this.selectedFiles,
+    );
+  }
+}

lib/features/services/blocs/services_cubit.dart

@@ -12,6 +12,7 @@ import 'package:flux/features/services/models/service_file_model.dart';
 import 'package:flux/features/services/models/service_model.dart';
 import 'package:get_it/get_it.dart';
 import 'package:collection/collection.dart';
+import 'package:permission_handler/permission_handler.dart';
 part 'services_state.dart';
 
 class ServicesCubit extends Cubit<ServicesState> {
@@ -202,19 +203,31 @@ class ServicesCubit extends Cubit<ServicesState> {
 
   // --- PERSISTENZA ---
 
-  Future<void> saveCurrentService({required bool isBozza}) async {
+  Future<void> saveCurrentService({
+    required bool isBozza,
+    bool shouldPop = true,
+    List<ServiceFileModel>? files,
+  }) async {
     if (state.currentService == null) return;
 
     emit(state.copyWith(status: ServicesStatus.saving, errorMessage: null));
     try {
       // 1. Aggiorniamo il flag bozza in base a quale pulsante ha premuto l'utente
-      final serviceToSave = state.currentService!.copyWith(isBozza: isBozza);
+      final serviceToSave = state.currentService!.copyWith(
+        isBozza: isBozza,
+        files: files,
+      );
 
       // 2. Salvataggio corazzato
-      await _repository.saveFullService(serviceToSave);
+      final updatedService = await _repository.saveFullService(serviceToSave);
 
       // 3. Reset e ricaricamento
-      emit(state.copyWith(status: ServicesStatus.saved, currentService: null));
+      emit(
+        state.copyWith(
+          status: shouldPop ? ServicesStatus.saved : ServicesStatus.savedNoPop,
+          currentService: shouldPop ? null : updatedService,
+        ),
+      );
       await loadServices(refresh: true);
     } catch (e) {
       emit(
@@ -235,7 +248,7 @@ class ServicesCubit extends Cubit<ServicesState> {
         serviceId: state.currentService?.id ?? '',
         name: file.name.fileNameWithoutExtension(),
         extension: file.name.fileExtension(),
-        url: '',
+        storagePath: '',
         fileSize: file.size,
         localBytes: file.bytes,
         createdAt: DateTime.now(),
@@ -273,49 +286,62 @@ class ServicesCubit extends Cubit<ServicesState> {
     );
   }
 
-  void saveAndCopyFileToCustomer(ServiceFileModel file) async {
+  void saveAndCopyFileToCustomer(List<ServiceFileModel> selectedFiles) async {
     final currentService = state.currentService;
+
+    // 1. Check di sicurezza: se non c'è il cliente, non sappiamo dove copiare
     if (currentService == null || currentService.customerId == null) {
-      // Magari mostra un errore: non posso copiare al cliente se non c'è un cliente!
+      emit(
+        state.copyWith(
+          status: ServicesStatus.failure,
+          errorMessage:
+              "Impossibile copiare: nessun cliente associato alla pratica.",
+        ),
+      );
       return;
     }
 
     emit(state.copyWith(status: ServicesStatus.loading));
 
     try {
-      // 1. Salviamo la pratica (Bozza o definitiva che sia)
-      // Questo assicura che il file sia stato caricato su Storage e censito su DB
-      await saveCurrentService(isBozza: currentService.isBozza);
+      // 2. SALVATAGGIO CORAZZATO
+      // Chiamiamo il repo e otteniamo la pratica con TUTTI i file ora dotati di ID e storagePath
+      final updatedService = await _repository.saveFullService(currentService);
 
-      // 2. Recuperiamo il file "aggiornato"
-      // Dopo il saveCurrentService, il file che prima era "locale" ora ha un URL.
-      // Lo cerchiamo nella lista aggiornata per nome o estensione.
-      final savedFile = state.currentService!.files.firstWhere(
-        (f) => f.name == file.name && f.extension == file.extension,
-        orElse: () => file,
-      );
+      // 3. COPIA RELAZIONALE
+      // Per ogni file che l'utente ha selezionato nella UI, cerchiamo la sua versione
+      // "ufficiale" (quella con lo storagePath) nel modello appena tornato dal DB.
+      for (var selectedFile in selectedFiles) {
+        // Cerchiamo il match nel modello aggiornato
+        final persistedFile = updatedService.files.firstWhere(
+          (f) =>
+              f.name == selectedFile.name &&
+              f.extension == selectedFile.extension,
+          orElse: () => throw Exception(
+            "File ${selectedFile.name} non trovato dopo il salvataggio.",
+          ),
+        );
 
-      if (savedFile.url.isEmpty) {
-        throw Exception(
-          "Errore: URL del file non trovato dopo il salvataggio.",
+        // Creiamo il link nel database del cliente
+        await _repository.copyFileToCustomer(
+          file: persistedFile,
+          customerId: currentService.customerId!,
         );
       }
 
-      // 3. Chiamiamo il repository per la copia fisica nel database del cliente
-      // Passiamo l'URL del file e l'ID del cliente
-      await _repository.copyFileToCustomer(
-        file: savedFile,
-        customerId: currentService.customerId!,
+      // 4. AGGIORNAMENTO STATO
+      // Aggiorniamo il Cubit con il servizio salvato così la UI mostra i file come "Remoti"
+      emit(
+        state.copyWith(
+          status: ServicesStatus.success,
+          currentService: updatedService,
+        ),
       );
-
-      // 4. Feedback all'utente
-      // Potresti emettere un successo o mostrare un toast
-      emit(state.copyWith(status: ServicesStatus.success));
     } catch (e) {
       emit(
         state.copyWith(
           status: ServicesStatus.failure,
-          errorMessage: "Errore durante la copia del file: $e",
+          errorMessage: "Errore durante il salvataggio e copia: $e",
         ),
       );
     }

lib/features/services/blocs/services_state.dart

@@ -1,6 +1,15 @@
 part of 'services_cubit.dart';
 
-enum ServicesStatus { initial, loading, ready, saving, saved, success, failure }
+enum ServicesStatus {
+  initial,
+  loading,
+  ready,
+  saving,
+  saved,
+  savedNoPop,
+  success,
+  failure,
+}
 
 class ServicesState extends Equatable {
   final ServicesStatus status;
@@ -10,6 +19,7 @@ class ServicesState extends Equatable {
   final String query;
   final DateTimeRange? dateRange;
   final bool hasReachedMax;
+  final bool isSavingDraft;
 
   const ServicesState({
     required this.status,
@@ -19,6 +29,7 @@ class ServicesState extends Equatable {
     this.query = '',
     this.dateRange,
     this.hasReachedMax = false,
+    this.isSavingDraft = false,
   });
 
   ServicesState copyWith({
@@ -29,6 +40,7 @@ class ServicesState extends Equatable {
     String? query,
     DateTimeRange? dateRange,
     bool? hasReachedMax,
+    bool? isSavingDraft,
   }) {
     return ServicesState(
       status: status ?? this.status,
@@ -38,6 +50,7 @@ class ServicesState extends Equatable {
       query: query ?? this.query,
       dateRange: dateRange ?? this.dateRange,
       hasReachedMax: hasReachedMax ?? this.hasReachedMax,
+      isSavingDraft: isSavingDraft ?? this.isSavingDraft,
     );
   }
 
@@ -50,5 +63,6 @@ class ServicesState extends Equatable {
     query,
     dateRange,
     hasReachedMax,
+    isSavingDraft,
   ];
 }