davide/biteplan
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(shopping_list): genera ID univoci in ShoppingListProvider

Browse Source 
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>
This commit is contained in:
Davide Grilli
2026-05-17 23:27:04 +02:00
parent 49e7bcdbfa
commit 9fcaae9687
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/features/shopping_list/providers/shopping_list_provider.dart
+3 -3
View File
@@ -6,6 +6,7 @@ import '../../../core/constants/app_constants.dart';
class ShoppingListProvider extends ChangeNotifier { class ShoppingListProvider extends ChangeNotifier {
List<ShoppingItem> _items = []; List<ShoppingItem> _items = [];
int _idSeq = 0;
List<ShoppingItem> get items => _items; List<ShoppingItem> get items => _items;
List<ShoppingItem> get pendingItems => List<ShoppingItem> get pendingItems =>
@@ -39,7 +40,7 @@ class ShoppingListProvider extends ChangeNotifier {
_items = [ _items = [
..._items, ..._items,
ShoppingItem( ShoppingItem(
id: DateTime.now().millisecondsSinceEpoch.toString(), id: '${DateTime.now().millisecondsSinceEpoch}_${_idSeq++}',
name: t, name: t,
), ),
]; ];
@@ -61,11 +62,10 @@ class ShoppingListProvider extends ChangeNotifier {
} }
final toAdd = <ShoppingItem>[]; final toAdd = <ShoppingItem>[];
var i = 0;
for (final entry in counts.entries) { for (final entry in counts.entries) {
if (existing.contains(entry.key)) continue; if (existing.contains(entry.key)) continue;
toAdd.add(ShoppingItem( toAdd.add(ShoppingItem(
id: '${DateTime.now().millisecondsSinceEpoch}_${i++}', id: '${DateTime.now().millisecondsSinceEpoch}_${_idSeq++}',
name: canonical[entry.key]!, name: canonical[entry.key]!,
quantity: entry.value, quantity: entry.value,
)); ));