From 9fcaae96873677c0dbbce3cefe1d71b926075d2b Mon Sep 17 00:00:00 2001
From: Davide Grilli <davide.grilli@outlook.com>
Date: Sun, 17 May 2026 23:27:04 +0200
Subject: [PATCH] fix(shopping_list): genera ID univoci in ShoppingListProvider
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

add() usava millisecondsSinceEpoch come ID: chiamate ravvicinate nello
stesso millisecondo producevano ID identici, causando toggle e remove
incorretti su più elementi. Introdotto _idSeq come contatore progressivo,
allineato all'approccio già usato in addAll().

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../shopping_list/providers/shopping_list_provider.dart     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/features/shopping_list/providers/shopping_list_provider.dart b/lib/features/shopping_list/providers/shopping_list_provider.dart
index d100f40..79fef9e 100644
--- a/lib/features/shopping_list/providers/shopping_list_provider.dart
+++ b/lib/features/shopping_list/providers/shopping_list_provider.dart
@@ -6,6 +6,7 @@ import '../../../core/constants/app_constants.dart';
 
 class ShoppingListProvider extends ChangeNotifier {
   List<ShoppingItem> _items = [];
+  int _idSeq = 0;
 
   List<ShoppingItem> get items => _items;
   List<ShoppingItem> get pendingItems =>
@@ -39,7 +40,7 @@ class ShoppingListProvider extends ChangeNotifier {
     _items = [
       ..._items,
       ShoppingItem(
-        id: DateTime.now().millisecondsSinceEpoch.toString(),
+        id: '${DateTime.now().millisecondsSinceEpoch}_${_idSeq++}',
         name: t,
       ),
     ];
@@ -61,11 +62,10 @@ class ShoppingListProvider extends ChangeNotifier {
     }
 
     final toAdd = <ShoppingItem>[];
-    var i = 0;
     for (final entry in counts.entries) {
       if (existing.contains(entry.key)) continue;
       toAdd.add(ShoppingItem(
-        id: '${DateTime.now().millisecondsSinceEpoch}_${i++}',
+        id: '${DateTime.now().millisecondsSinceEpoch}_${_idSeq++}',
         name: canonical[entry.key]!,
         quantity: entry.value,
       ));