Fix for bug #3138

ActiveSync/NGMimeMessage+ActiveSync.m

@@ -1,6 +1,6 @@
 /*
 
-Copyright (c) 2014, Inverse inc.
+Copyright (c) 2015, Inverse inc.
 All rights reserved.
 
 Redistribution and use in source and binary forms, with or without
@@ -40,35 +40,39 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 @implementation NGMimeMessage (ActiveSync)
 
-- (NSArray *) allRecipients
+- (void) _addRecipients: (NSEnumerator *) enumerator
+                toArray: (NSMutableArray *) recipients
 {
-  NSEnumerator *enumerator, *addressList;
-  NSMutableArray *recipients;
   NGMailAddressParser *parser;
+  NSEnumerator *addressList;
   NGMailAddress *address;
   NSString *s;
 
+  while ((s = [enumerator nextObject]))
+    {
+      parser = [NGMailAddressParser mailAddressParserWithString: s];
+      addressList = [[parser parseAddressList] objectEnumerator];
+      
+      while ((address = [addressList nextObject]))
+        [recipients addObject: [address address]];
+    }
+}
+
+- (NSArray *) allRecipients
+{
+  NSMutableArray *recipients;
+  NSEnumerator *enumerator;
+
   recipients = [NSMutableArray array];
 
-  enumerator = [[self headersForKey: @"to"] objectEnumerator];
+  [self _addRecipients: [[self headersForKey: @"to"] objectEnumerator]
-  while ((s = [enumerator nextObject]))
+               toArray: recipients];
-    {
-      parser = [NGMailAddressParser mailAddressParserWithString: s];
-      addressList = [[parser parseAddressList] objectEnumerator];
-      
-      while ((address = [addressList nextObject]))
-        [recipients addObject: [address address]];
-    }
 
-  enumerator = [[self headersForKey: @"cc"] objectEnumerator];
+  [self _addRecipients: [[self headersForKey: @"cc"] objectEnumerator]
-  while ((s = [enumerator nextObject]))
+               toArray: recipients];
-    {
+
-      parser = [NGMailAddressParser mailAddressParserWithString: s];
+  [self _addRecipients: [[self headersForKey: @"bcc"] objectEnumerator]
-      addressList = [[parser parseAddressList] objectEnumerator];
+               toArray: recipients];
-      
-      while ((address = [addressList nextObject]))
-        [recipients addObject: [address address]];
-    }
 
   return recipients;
 }

ActiveSync/SOGoActiveSyncDispatcher.m

@@ -2273,7 +2273,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
   
   error = [self _sendMail: data
                recipients: [message allRecipients]
-                saveInSentItems:  ([(id)[theDocumentElement getElementsByTagName: @"SaveInSentItems"] count] ? YES : NO)];
+                saveInSentItems: ([(id)[theDocumentElement getElementsByTagName: @"SaveInSentItems"] count] ? YES : NO)];
 
   if (error)
     {

NEWS

@@ -16,6 +16,7 @@ Bug fixes
  - reduced telephone number to a single value in JSON response of contacts list
  - fixed freebusy data when 'busy off hours' is enabled and period starts during the weekend
  - fixed fetching of freebusy data from the Web interface
+ - fixed EAS handling of Bcc in emails (#3138)
 
 2.2.16 (2015-02-12)
 -------------------