From 30e71606a03a30a0eaf93d57834327f9136b0cd2 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Sat, 06 Jun 2015 03:26:02 -0400
Subject: [PATCH] Fix Compose action in addressbook for results from multiple addressbooks (#1490413)
---
CHANGELOG | 1 +
program/steps/addressbook/mailto.inc | 25 ++++++++++++++-----------
2 files changed, 15 insertions(+), 11 deletions(-)
diff --git a/CHANGELOG b/CHANGELOG
index 3b5b5a2..9a62f4e 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,7 @@
- Fix "Importing..." message does not hide on error (#1490422)
- Fix SQL error on logout when using session_storage=php (#1490421)
- Update to jQuery 2.1.4 (#1490406)
+- Fix Compose action in addressbook for results from multiple addressbooks (#1490413)
RELEASE 1.1.2
-------------
diff --git a/program/steps/addressbook/mailto.inc b/program/steps/addressbook/mailto.inc
index f5ff20b..43109c9 100644
--- a/program/steps/addressbook/mailto.inc
+++ b/program/steps/addressbook/mailto.inc
@@ -19,9 +19,9 @@
+-----------------------------------------------------------------------+
*/
-$cids = rcmail_get_cids();
-$mailto = array();
-$recipients = null;
+$cids = rcmail_get_cids();
+$mailto = array();
+$sources = array();
foreach ($cids as $source => $cid) {
$CONTACTS = $RCMAIL->get_address_book($source);
@@ -29,15 +29,15 @@
if ($CONTACTS->ready) {
$CONTACTS->set_page(1);
$CONTACTS->set_pagesize(count($cid) + 2); // +2 to skip counting query
- $recipients = $CONTACTS->search($CONTACTS->primary_key, $cid, 0, true, true, 'email');
+ $sources[] = $CONTACTS->search($CONTACTS->primary_key, $cid, 0, true, true, 'email');
}
}
if (!empty($_REQUEST['_gid']) && isset($_REQUEST['_source'])) {
- $source = rcube_utils::get_input_value('_source', rcube_utils::INPUT_GPC);
+ $source = rcube_utils::get_input_value('_source', rcube_utils::INPUT_GPC);
$CONTACTS = $RCMAIL->get_address_book($source);
- $group_id = rcube_utils::get_input_value('_gid', rcube_utils::INPUT_GPC);
+ $group_id = rcube_utils::get_input_value('_gid', rcube_utils::INPUT_GPC);
$group_data = $CONTACTS->get_group($group_id);
// group has an email address assigned: use that
@@ -48,20 +48,23 @@
$CONTACTS->set_group($group_id);
$CONTACTS->set_page(1);
$CONTACTS->set_pagesize(200); // limit somehow
- $recipients = $CONTACTS->list_records();
+ $sources[] = $CONTACTS->list_records();
}
}
-if ($recipients) {
- while (is_object($recipients) && ($rec = $recipients->iterate())) {
+foreach ($sources as $source) {
+ while (is_object($source) && ($rec = $source->iterate())) {
$emails = $CONTACTS->get_col_values('email', $rec, true);
- $mailto[] = format_email_recipient($emails[0], $rec['name']);
+
+ if (!empty($emails)) {
+ $mailto[] = format_email_recipient($emails[0], $rec['name']);
+ }
}
}
if (!empty($mailto)) {
$mailto_str = join(', ', $mailto);
- $mailto_id = substr(md5($mailto_str), 0, 16);
+ $mailto_id = substr(md5($mailto_str), 0, 16);
$_SESSION['mailto'][$mailto_id] = urlencode($mailto_str);
$OUTPUT->command('open_compose_step', array('_mailto' => $mailto_id));
}
--
Gitblit v1.9.1