issue linagora#180 fixed duplicate addressbooks showing up when an adressbook is publicly shared and delegated to a user

Author: rezk2ll

src/linagora.esn.contact/app/sidebar/sidebar.controller.js

@@ -150,7 +150,23 @@ function ContactSidebarController(
     var categories = contactAddressbookDisplayService.categorizeDisplayShells(self.displayShells);
 
     self.userAddressbooks = categories.userAddressbooks;
-    self.sharedAddressbooks = categories.sharedAddressbooks;
+    self.sharedAddressbooks = _getUniqueSharedAddressbooks(categories.sharedAddressbooks);
     self.virtualAddressbooks = categories.virtualAddressbooks;
   }
+
+  function _getUniqueSharedAddressbooks(addressbooks) {
+    if (!addressbooks) return;
+
+    const uniqueAddressbookList = addressbooks.reduce((acc, current) => {
+      const href = current.shell.source.href;
+
+      if (!acc[href]) {
+        acc[href] = current;
+      }
+
+      return acc;
+    }, {});
+
+    return Object.values(uniqueAddressbookList);
+  }
 }

src/linagora.esn.contact/app/sidebar/sidebar.controller.spec.js

@@ -46,13 +46,11 @@ describe('The ContactSidebarController controller', function() {
       userAPI = _userAPI_;
       userUtils = _userUtils_;
       CONTACT_ADDRESSBOOK_EVENTS = _CONTACT_ADDRESSBOOK_EVENTS_;
-
-      contactAddressbookDisplayService.categorizeDisplayShells = function() {
-        return {
+      contactAddressbookDisplayService.categorizeDisplayShells = sinon.stub()
+        .returns({
           userAddressbooks: [],
           externalAddressbooks: []
-        };
-      };
+        });
     });
   });
 
@@ -260,6 +258,40 @@ describe('The ContactSidebarController controller', function() {
     ]);
   });
 
+  it('should avoid listing duplicate entries when an adressbook is shared an delegated to the same user', () => {
+    contactAddressbookService.listAddressbooks = sinon.stub().returns($q.when([]));
+    contactAddressbookDisplayService.categorizeDisplayShells.returns({
+      userAddressbooks: [
+        {
+          name: 'bookA',
+          shell: {
+            source: { bookId: 'user0', href: '0' }
+          }
+        }
+      ],
+      sharedAddressbooks: [
+        {
+          name: 'bookB',
+          shell: {
+            source: { bookId: 'user1', href: '1' }
+          }
+        },
+        {
+          name: 'bookC',
+          shell: {
+            source: { bookId: 'user1', href: '1' }
+          }
+        }
+      ]
+    });
+
+    const controller = initController();
+
+    $rootScope.$digest();
+
+    expect(controller.sharedAddressbooks.length).to.equal(1);
+  });
+
   describe('On updated address book event', function() {
     it('should update an address book when updated address book event is fired', function() {
       var addressbooks = [