From e9da98079038830d533b00fa67fe1de34b89c421 Mon Sep 17 00:00:00 2001
From: Allen Shaw <allen@JoineryHQ.com>
Date: Tue, 16 Oct 2018 09:07:34 -0500
Subject: [PATCH] Toward CRM-19751: conditionally change On-Hold criteria to
 select on Advanced Search form.

---
 CRM/Contact/BAO/Query.php                     |  5 +-
 CRM/Contact/Form/Search/Criteria.php          | 10 ++-
 .../Fields/preferred_communication_method.tpl |  8 ++-
 .../CRM/Contact/Form/Search/CriteriaTest.php  | 61 +++++++++++++++++++
 4 files changed, 78 insertions(+), 6 deletions(-)
 create mode 100644 tests/phpunit/CRM/Contact/Form/Search/CriteriaTest.php

diff --git a/CRM/Contact/BAO/Query.php b/CRM/Contact/BAO/Query.php
index 027d8165dfdf..ccbbafb72383 100644
--- a/CRM/Contact/BAO/Query.php
+++ b/CRM/Contact/BAO/Query.php
@@ -1635,8 +1635,9 @@ public static function convertFormValues(&$formValues, $wildcard = 0, $useEquals
         }
       }
       elseif ($id == 'email_on_hold') {
-        if ($formValues['email_on_hold']['on_hold']) {
-          $params[] = array('on_hold', '=', $formValues['email_on_hold']['on_hold'], 0, 0);
+        if ($onHoldValue = CRM_Utils_Array::value('email_on_hold', $formValues)) {
+          $onHoldValue = (array) $onHoldValue;
+          $params[] = array('on_hold', 'IN', $onHoldValue, 0, 0);
         }
       }
       elseif (substr($id, 0, 7) == 'custom_'
diff --git a/CRM/Contact/Form/Search/Criteria.php b/CRM/Contact/Form/Search/Criteria.php
index 845ab7c9a541..012500df3a47 100644
--- a/CRM/Contact/Form/Search/Criteria.php
+++ b/CRM/Contact/Form/Search/Criteria.php
@@ -231,9 +231,13 @@ public static function basic(&$form) {
     $form->addRadio('privacy_toggle', ts('Privacy Options'), $options, array('allowClear' => FALSE));
 
     // preferred communication method
-
-    $onHold[] = $form->createElement('advcheckbox', 'on_hold', NULL, '');
-    $form->addGroup($onHold, 'email_on_hold', ts('Email On Hold'));
+    if (Civi::settings()->get('civimail_multiple_bulk_emails')) {
+      $form->addSelect('email_on_hold',
+        array('entity' => 'email', 'multiple' => 'multiple', 'label' => ts('Email On Hold'), 'options' => CRM_Core_PseudoConstant::emailOnHoldOptions()));
+    }
+    else {
+      $form->add('advcheckbox', 'email_on_hold', ts('Email On Hold'));
+    }
 
     $form->addSelect('preferred_communication_method',
       array('entity' => 'contact', 'multiple' => 'multiple', 'label' => ts('Preferred Communication Method'), 'option_url' => NULL, 'placeholder' => ts('- any -')));
diff --git a/templates/CRM/Contact/Form/Search/Criteria/Fields/preferred_communication_method.tpl b/templates/CRM/Contact/Form/Search/Criteria/Fields/preferred_communication_method.tpl
index 3e94b37b2d42..3edd73cb3b22 100644
--- a/templates/CRM/Contact/Form/Search/Criteria/Fields/preferred_communication_method.tpl
+++ b/templates/CRM/Contact/Form/Search/Criteria/Fields/preferred_communication_method.tpl
@@ -3,7 +3,13 @@
 {$form.preferred_communication_method.html}
 <br/>
 
-{if $form.email_on_hold}
+{if $form.email_on_hold.type == 'select'}
+  <br/>
+  {$form.email_on_hold.label}
+  <br/>
+  {$form.email_on_hold.html}
+  <br/>
+{elseif $form.email_on_hold.type == 'checkbox'}
   <div class="spacer"></div>
   {$form.email_on_hold.html}
   {$form.email_on_hold.label}
diff --git a/tests/phpunit/CRM/Contact/Form/Search/CriteriaTest.php b/tests/phpunit/CRM/Contact/Form/Search/CriteriaTest.php
new file mode 100644
index 000000000000..0091cc249955
--- /dev/null
+++ b/tests/phpunit/CRM/Contact/Form/Search/CriteriaTest.php
@@ -0,0 +1,61 @@
+<?php
+/*
+  +--------------------------------------------------------------------+
+  | CiviCRM version 5                                                  |
+  +--------------------------------------------------------------------+
+  | Copyright CiviCRM LLC (c) 2004-2018                                |
+  +--------------------------------------------------------------------+
+  | This file is a part of CiviCRM.                                    |
+  |                                                                    |
+  | CiviCRM is free software; you can copy, modify, and distribute it  |
+  | under the terms of the GNU Affero General Public License           |
+  | Version 3, 19 November 2007 and the CiviCRM Licensing Exception.   |
+  |                                                                    |
+  | CiviCRM is distributed in the hope that it will be useful, but     |
+  | WITHOUT ANY WARRANTY; without even the implied warranty of         |
+  | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.               |
+  | See the GNU Affero General Public License for more details.        |
+  |                                                                    |
+  | You should have received a copy of the GNU Affero General Public   |
+  | License and the CiviCRM Licensing Exception along                  |
+  | with this program; if not, contact CiviCRM LLC                     |
+  | at info[AT]civicrm[DOT]org. If you have questions about the        |
+  | GNU Affero General Public License or the licensing of CiviCRM,     |
+  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
+  +--------------------------------------------------------------------+
+ */
+
+/**
+ * Test class for CRM_Contact_Form_Search_Criteria
+ *
+ * @package CiviCRM
+ * @group headless
+ */
+class CRM_Contact_Form_Search_CriteraTest extends CiviUnitTestCase {
+
+  /**
+   * Test that the 'multiple bulk email' setting correctly affects the type of
+   * field used for the 'email on hold' criteria in Advanced Search.
+   */
+  public function testAdvancedHSearchObservesMultipleBulkEmailSetting() {
+
+    // If setting is enabled, criteria should be a select element.
+    Civi::settings()->set('civimail_multiple_bulk_emails', 1);
+    $form = new CRM_Contact_Form_Search_Advanced();
+    $form->controller = new CRM_Contact_Controller_Search();
+    $form->preProcess();
+    $form->buildQuickForm();
+    $onHoldElemenClass = (get_class( $form->_elements[$form->_elementIndex['email_on_hold']]));
+    $this->assertEquals('HTML_QuickForm_select', $onHoldElemenClass, 'civimail_multiple_bulk_emails setting = 1, so email_on_hold should be a select element.');
+
+    // If setting is disabled, criteria should be a checkbox.
+    Civi::settings()->set('civimail_multiple_bulk_emails', 0);
+    $form = new CRM_Contact_Form_Search_Advanced();
+    $form->controller = new CRM_Contact_Controller_Search();
+    $form->preProcess();
+    $form->buildQuickForm();
+    $onHoldElemenClass = (get_class( $form->_elements[$form->_elementIndex['email_on_hold']]));
+    $this->assertEquals('HTML_QuickForm_advcheckbox', $onHoldElemenClass, 'civimail_multiple_bulk_emails setting = 0, so email_on_hold should be a checkbox.');
+  }
+
+}