Fixed the duplicated contacts issue

Author: AlaaElattar

app/lib/screens/wallets/contacts.dart

@@ -43,11 +43,8 @@ class _ContactsScreenState extends State<ContactsScreen> {
   }
 
   _onAddContact(PkidContact contact) async {
-    final exists = myPkidContacts.any((c) => c.address == contact.address);
-    if (!exists) {
-      myPkidContacts.add(contact);
-      setState(() {});
-    }
+    myPkidContacts.add(contact);
+    setState(() {});
   }
 
   _openAddContactOverlay() {
@@ -61,6 +58,7 @@ class _ContactsScreenState extends State<ContactsScreen> {
               onAddContact: _onAddContact,
               chainType: widget.chainType,
               contacts: [...myPkidContacts, ...myWalletContacts],
+              getAllContacts: () => [...myPkidContacts, ...myWalletContacts],
             ));
   }
 
@@ -146,14 +144,14 @@ class _ContactsScreenState extends State<ContactsScreen> {
                                 c.address != widget.currentWalletAddress &&
                                 c.type == widget.chainType)
                             .toList(),
+                        chainType: widget.chainType,
                         onSelectToAddress: widget.onSelectToAddress),
                     ContactsWidget(
-                      contacts: myPkidContacts
-                          .where((c) => c.type == widget.chainType)
-                          .toList(),
+                      contacts: myPkidContacts,
                       onSelectToAddress: widget.onSelectToAddress,
                       onDeleteContact: _onDeleteContact,
                       onEditContact: _openEditContactOverlay,
+                      chainType: widget.chainType,
                       canEditAndDelete: true,
                     ),
                   ],

app/lib/widgets/wallets/add_edit_contact.dart

@@ -17,6 +17,7 @@ class AddEditContact extends StatefulWidget {
     this.name = '',
     this.address = '',
     this.onEditContact,
+    this.getAllContacts,
   });
 
   final void Function(PkidContact addedContact)? onAddContact;
@@ -27,6 +28,7 @@ class AddEditContact extends StatefulWidget {
   final String address;
   final void Function(String oldName, String newName, String newAddress)?
       onEditContact;
+  final List<PkidContact> Function()? getAllContacts;
 
   @override
   State<StatefulWidget> createState() {
@@ -81,27 +83,34 @@ class _AddEditContactState extends State<AddEditContact> {
 
   bool _validateAddress(String contactAddress, {bool edit = false}) {
     addressError = null;
+    final allContacts = widget.getAllContacts != null
+        ? widget.getAllContacts!()
+        : widget.contacts;
 
     if (contactAddress.isEmpty) {
       addressError = "Address can't be empty";
       return false;
     }
-    // Check for duplicates across all chain types
-    final contacts = widget.contacts.where((c) => c.address == contactAddress);
-    if (contacts.isNotEmpty) {
+
+    final isDuplicate = allContacts.any((c) => c.address == contactAddress);
+
+    if (isDuplicate) {
       addressError = 'Address is used in another contact';
       return false;
     }
+
     if (_selectedChainType == ChainType.TFChain &&
         contactAddress.length != 48) {
       addressError = 'Address length should be 48 characters';
       return false;
     }
+
     if (_selectedChainType == ChainType.Stellar &&
         !isValidStellarAddress(contactAddress)) {
       addressError = 'Invaild Stellar address';
       return false;
     }
+
     return true;
   }
 
@@ -122,10 +131,12 @@ class _AddEditContactState extends State<AddEditContact> {
           Icons.error, DialogType.Error);
       return;
     }
-    if (chainType == widget.chainType) {
-      widget.onAddContact!(PkidContact(
-          name: contactName, address: contactAddress, type: chainType));
-    }
+    widget.onAddContact?.call(PkidContact(
+      name: contactName,
+      address: contactAddress,
+      type: chainType,
+    ));
+
     if (!context.mounted) return;
     Navigator.pop(context);
   }

app/lib/widgets/wallets/contacts_widget.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart';
 import 'package:threebotlogin/models/contact.dart';
+import 'package:threebotlogin/models/wallet.dart';
 import 'package:threebotlogin/widgets/wallets/contact_card.dart';
 
 class ContactsWidget extends StatelessWidget {
@@ -9,19 +10,21 @@ class ContactsWidget extends StatelessWidget {
     required this.onSelectToAddress,
     this.onDeleteContact,
     this.onEditContact,
+    required this.chainType,
     this.canEditAndDelete = false,
   });
-
   final List<PkidContact> contacts;
   final void Function(String address) onSelectToAddress;
   final bool canEditAndDelete;
   final void Function(String name)? onDeleteContact;
   final void Function(String oldName, String oldAddress)? onEditContact;
+  final ChainType chainType;
 
   @override
   Widget build(BuildContext context) {
+     final filteredContacts = contacts.where((c) => c.type == chainType).toList();
     Widget content;
-    if (contacts.isEmpty) {
+    if (filteredContacts.isEmpty) {
       content = Center(
         child: Text(
           'No contacts yet.',
@@ -33,7 +36,7 @@ class ContactsWidget extends StatelessWidget {
       );
     } else {
       content = ListView(children: [
-        for (final contact in contacts)
+        for (final contact in filteredContacts)
           InkWell(
               onTap: () {
                 onSelectToAddress(contact.address);