Changeset 3697 in subversion

Timestamp:

Jun 2, 2010 6:02:23 AM (3 years ago)

Author:

alec

Message:

• Fix duplicated contacts when copying with group selected (#1486766)

File:

• trunk/roundcubemail/program/steps/addressbook/copy.inc (modified) (3 diffs)

trunk/roundcubemail/program/steps/addressbook/copy.inc

@@ -34 +34 @@
 
   if ($TARGET && $TARGET->ready && !$TARGET->readonly) {
-    if ($target_group && $TARGET->groups)
-      $TARGET->set_group($target_group);
-    
     $arr_cids = explode(',', $cid);
+    $ids = array();
+
     foreach ($arr_cids as $cid) {
       $plugin = $RCMAIL->plugins->exec_hook('create_contact', array(
@@ -46 +45 @@
       $a_record = $plugin['record'];
 
-      if (!$plugin['abort'])
-        if ($TARGET->insert($a_record, true))
-          $success++;
+      if (!$plugin['abort']) {
+        // check if contact exists, if so, we'll need it's ID
+        $result = $TARGET->search('email', $a_record['email'], true, true);
+
+        // insert contact record
+        if (!$result->count) {
+          if ($insert_id = $TARGET->insert($a_record, false)) {
+            $ids[] = $insert_id;
+            $success++;
+          }
+        }
+        else {
+          $record = $result->first();
+          $ids[] = $record['ID'];
+        }
+      }
+    }
+
+    // assign to group
+    if ($target_group && $TARGET->groups && !empty($ids)) {
+      $plugin = $RCMAIL->plugins->exec_hook('group_addmembers', array(
+        'group_id' => $target_group, 'ids' => $ids, 'source' => $target));
+
+      if (!$plugin['abort']) {
+        $TARGET->reset();
+        $TARGET->set_group($target_group);
+
+        if (($maxnum = $RCMAIL->config->get('max_group_members', 0)) && ($TARGET->count()->count + count($plugin['ids']) > $maxnum)) {
+          $OUTPUT->show_message('maxgroupmembersreached', 'warning', array('max' => $maxnum));
+          $OUTPUT->send();
+        }
+
+        if (($cnt = $TARGET->add_to_group($target_group, $plugin['ids'])) && $cnt > $success)
+          $success = $cnt;
+      }
     }
   }
@@ -56 +87 @@
   else
     $OUTPUT->show_message('copysuccess', 'notice', array('nr' => $success));
-    
-  // close connection to second address directory
-  $TARGET->close();
 }