From a45f9b7bf58475ccc812e819f159638403c00419 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Mon, 01 Jul 2013 04:22:14 -0400 Subject: [PATCH] Contacts drag-n-drop default action is to move contacts (#1488751) Added possibility to choose to move or copy contacts from drag-n-drop menu (#1488751) Use consistent naming: 'moveto' -> 'move' --- program/steps/addressbook/copy.inc | 137 +++++++++++++++++++++++++++++++++------------ 1 files changed, 99 insertions(+), 38 deletions(-) diff --git a/program/steps/addressbook/copy.inc b/program/steps/addressbook/copy.inc index 2d3a91a..82917e2 100644 --- a/program/steps/addressbook/copy.inc +++ b/program/steps/addressbook/copy.inc @@ -4,9 +4,12 @@ +-----------------------------------------------------------------------+ | program/steps/addressbook/copy.inc | | | - | This file is part of the RoundCube Webmail client | - | Copyright (C) 2007, RoundCube Dev. - Switzerland | - | Licensed under the GNU GPL | + | This file is part of the Roundcube Webmail client | + | Copyright (C) 2007, The Roundcube Dev Team | + | | + | Licensed under the GNU General Public License version 3 or | + | any later version with exceptions for skins & plugins. | + | See the README file for a full license statement. | | | | PURPOSE: | | Copy a contact record from one direcotry to another | @@ -14,53 +17,111 @@ +-----------------------------------------------------------------------+ | Author: Thomas Bruederli <roundcube@gmail.com> | +-----------------------------------------------------------------------+ - - $Id: copy.inc 471 2007-02-09 21:25:50Z thomasb $ - */ // only process ajax requests if (!$OUTPUT->ajax_call) return; -$cid = get_input_value('_cid', RCUBE_INPUT_POST); -$target = get_input_value('_to', RCUBE_INPUT_POST); + +$cids = rcmail_get_cids(); +$target = get_input_value('_to', RCUBE_INPUT_POST); $target_group = get_input_value('_togid', RCUBE_INPUT_POST); -if ($cid && preg_match('/^[a-z0-9\-_=]+(,[a-z0-9\-_=]+)*$/i', $cid) && strlen($target) && $target !== $source) +$success = 0; +$errormsg = 'copyerror'; +$maxnum = $RCMAIL->config->get('max_group_members', 0); + +foreach ($cids as $source => $cid) { - $success = 0; - $TARGET = $RCMAIL->get_address_book($target); - - if ($TARGET && $TARGET->ready && !$TARGET->readonly) { - if ($target_group && $TARGET->groups) - $TARGET->set_group($target_group); - - $arr_cids = explode(',', $cid); - foreach ($arr_cids as $cid) { - $plugin = $RCMAIL->plugins->exec_hook('create_contact', array( - 'record' => $CONTACTS->get_record($cid, true), - 'source' => $target, - 'group' => $target_group, - )); - $a_record = $plugin['record']; - - if (!$plugin['abort']) - if ($TARGET->insert($a_record, true)) - $success++; + // Something wrong, target not specified + if (!strlen($target)) { + break; } - } - if ($success == 0) - $OUTPUT->show_message('copyerror', 'error'); - else - $OUTPUT->show_message('copysuccess', 'notice', array('nr' => $success)); - - // close connection to second address directory - $TARGET->close(); + // It maight happen when copying records from search result + // Do nothing, go to next source + if ((string)$target == (string)$source) { + continue; + } + + $CONTACTS = $RCMAIL->get_address_book($source); + $TARGET = $RCMAIL->get_address_book($target); + + if (!$TARGET || !$TARGET->ready || $TARGET->readonly) { + break; + } + + $ids = array(); + + foreach ($cid as $cid) { + $a_record = $CONTACTS->get_record($cid, true); + + // avoid copying groups + if ($a_record['_type'] == 'group') + continue; + + // Check if contact exists, if so, we'll need it's ID + // Note: Some addressbooks allows empty email address field + if (!empty($a_record['email'])) + $result = $TARGET->search('email', $a_record['email'], 1, true, true); + else if (!empty($a_record['name'])) + $result = $TARGET->search('name', $a_record['name'], 1, true, true); + else + $result = new rcube_result_set(); + + // insert contact record + if (!$result->count) { + $plugin = $RCMAIL->plugins->exec_hook('contact_create', array( + 'record' => $a_record, 'source' => $target, 'group' => $target_group)); + + if (!$plugin['abort']) { + if ($insert_id = $TARGET->insert($plugin['record'], false)) { + $ids[] = $insert_id; + $success++; + } + } + else if ($plugin['result']) { + $ids = array_merge($ids, $plugin['result']); + $success++; + } + } + else { + $record = $result->first(); + $ids[] = $record['ID']; + $errormsg = empty($a_record['email']) ? 'contactnameexists' : 'contactexists'; + } + } + + // 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 && ($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; + } + else if ($plugin['result']) { + $success = $plugin['result']; + } + + $errormsg = $plugin['message'] ? $plugin['message'] : 'copyerror'; + } } + +if (!$success) + $OUTPUT->show_message($errormsg, 'error'); +else + $OUTPUT->show_message('copysuccess', 'notice', array('nr' => $success)); // send response $OUTPUT->send(); - -?> -- Gitblit v1.9.1