(js) Fix autocompletion of LDAP-based groups

Fixes #3673
Alinto · May 19, 2016 · cc28357 · cc28357
1 parent fdc8291
commit cc28357
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 9 deletions.
diff --git a/NEWS b/NEWS
@@ -4,6 +4,7 @@
 Bug fixes
  - [web] fixed creation of chip on blur (sgTransformOnBlur directive)
  - [web] fixed composition of new messages from Contacts module
+ - [web] fixed autocompletion of LDAP-based groups (#3673)
 
 3.1.0 (2016-05-18)
 ------------------

diff --git a/UI/WebServerResources/js/Contacts/AddressBookController.js b/UI/WebServerResources/js/Contacts/AddressBookController.js
@@ -171,31 +171,32 @@
       var promises = [], recipients = [];
 
       _.forEach(selectedCards, function(card) {
-        if (card.c_component == 'vcard' && card.c_mail.length) {
-          recipients.push(card.c_cn + ' <' + card.c_mail + '>');
-        }
-        else if (card.$isList()) {
+        if (card.$isList({expandable: true})) {
           // If the list's members were already fetch, use them
           if (angular.isDefined(card.refs) && card.refs.length) {
             _.forEach(card.refs, function(ref) {
               if (ref.email.length)
-                recipients.push(ref.c_cn + ' <' + ref.email + '>');
+                recipients.push(ref.$shortFormat());
             });
           }
           else {
             promises.push(vm.selectedFolder.$getCard(card.id).then(function(card) {
               return card.$futureCardData.then(function(data) {
                 _.forEach(data.refs, function(ref) {
                   if (ref.email.length)
-                    recipients.push(ref.c_cn + ' <' + ref.email + '>');
+                    recipients.push(ref.$shortFormat());
                 });
               });
             }));
           }
         }
+        else if (card.c_mail.length) {
+          recipients.push(card.$shortFormat());
+        }
       });
 
       $q.all(promises).then(function() {
+        recipients = _.uniq(recipients);
         if (recipients.length)
           vm.newMessage($event, recipients);
       });

diff --git a/UI/WebServerResources/js/Contacts/Card.service.js b/UI/WebServerResources/js/Contacts/Card.service.js
@@ -285,6 +285,9 @@
       else if (this.emails && this.emails.length) {
         email = this.emails[0].value;
       }
+      else if (this.c_mail && this.c_mail.length) {
+        email = this.c_mail[0];
+      }
       else {
         email = '';
       }
@@ -311,8 +314,10 @@
     return this.c_component == 'vcard';
   };
 
-  Card.prototype.$isList = function() {
-    return this.c_component == 'vlist';
+  Card.prototype.$isList = function(options) {
+    // isGroup attribute means it's a group of a LDAP source (not expandable on the client-side)
+    var condition = (!options || !options.expandable || options.expandable && !this.isgroup);
+    return this.c_component == 'vlist' && condition;
   };
 
   Card.prototype.$addOrgUnit = function(orgUnit) {

diff --git a/UI/WebServerResources/js/Mailer/MessageEditorController.js b/UI/WebServerResources/js/Mailer/MessageEditorController.js
@@ -209,7 +209,7 @@
 
       recipients = vm.message.editable[field];
 
-      if (contact.$isList()) {
+      if (contact.$isList({expandable: true})) {
         // If the list's members were already fetch, use them
         if (angular.isDefined(contact.refs) && contact.refs.length) {
           _.forEach(contact.refs, function(ref) {