Add CRM_Contact_BAO_Query::getQillValue fn
authorMonish Deb <deb.monish@gmail.com>
Wed, 30 Oct 2019 12:22:40 +0000 (17:52 +0530)
committerMonish Deb <deb.monish@gmail.com>
Wed, 30 Oct 2019 12:24:08 +0000 (17:54 +0530)
CRM/Case/BAO/Query.php
CRM/Contact/BAO/Query.php

index 55e32bd99e4c7eff3161595ff67e17e1071a1d01..f14466306732670caeb4d7c45fce5ac20173c002 100644 (file)
@@ -291,7 +291,7 @@ class CRM_Case_BAO_Query extends CRM_Core_BAO_Query {
         }
 
         $query->_where[$grouping][] = CRM_Contact_BAO_Query::buildClause("civicrm_case.{$name}", $op, $value, "Integer");
-        $query->_qill[$grouping][] = self::getQillValue('CRM_Case_DAO_Case', $name, $value, $op, $label);
+        $query->_qill[$grouping][] = CRM_Contact_BAO_Query::getQillValue('CRM_Case_DAO_Case', $name, $value, $op, $label);
         $query->_tables['civicrm_case'] = $query->_whereTables['civicrm_case'] = 1;
         return;
 
@@ -332,7 +332,7 @@ class CRM_Case_BAO_Query extends CRM_Core_BAO_Query {
 
       case 'case_subject':
         $query->_where[$grouping][] = CRM_Contact_BAO_Query::buildClause("civicrm_case.subject", $op, $value, 'String');
-        $query->_qill[$grouping][] = self::getQillValue('CRM_Case_DAO_Case', $name, $value, $op, 'Case Subject');
+        $query->_qill[$grouping][] = CRM_Contact_BAO_Query::getQillValue('CRM_Case_DAO_Case', $name, $value, $op, 'Case Subject');
         $query->_tables['civicrm_case'] = $query->_whereTables['civicrm_case'] = 1;
         $query->_tables['civicrm_case_contact'] = $query->_whereTables['civicrm_case_contact'] = 1;
         return;
@@ -588,25 +588,6 @@ case_relation_type.id = case_relationship.relationship_type_id )";
     return $from;
   }
 
-  /**
-   * Get the qill (search description for field) for the specified field.
-   *
-   * @todo this is private because it is the first step towards generalising rather than
-   * the final product IMHO.
-   *
-   * @param string $daoName
-   * @param string $name
-   * @param string $value
-   * @param string|array $op
-   * @param string $label
-   *
-   * @return string
-   */
-  private static function getQillValue($daoName, string $name, $value, $op, string $label) {
-    list($op, $value) = CRM_Contact_BAO_Query::buildQillForFieldValue($daoName, $name, $value, $op);
-    return ts('%1 %2 %3', [1 => $label, 2 => $op, 3 => $value]);
-  }
-
   /**
    * Getter for the qill object.
    *
index bb5665b259d0cb125d4d09262514387c47d6fb0f..d67f250dc038f58c391975b22803d74edab61185 100644 (file)
@@ -6247,6 +6247,22 @@ AND   displayRelType.is_active = 1
     return [CRM_Utils_Array::value($op, $qillOperators, $op), $fieldValue];
   }
 
+  /**
+   * Get the qill (search description for field) for the specified field.
+   *
+   * @param string $daoName
+   * @param string $name
+   * @param string $value
+   * @param string|array $op
+   * @param string $label
+   *
+   * @return string
+   */
+  public static function getQillValue($daoName, string $name, $value, $op, string $label) {
+    list($op, $value) = self::buildQillForFieldValue($daoName, $name, $value, $op);
+    return ts('%1 %2 %3', [1 => $label, 2 => $op, 3 => $value]);
+  }
+
   /**
    * Alter value to reflect wildcard settings.
    *
@@ -6651,7 +6667,7 @@ AND   displayRelType.is_active = 1
         FROM (
       SELECT civicrm_contribution.total_amount, civicrm_contribution.currency
       $from
-      $where  AND civicrm_contribution.cancel_date IS NOT NULL 
+      $where  AND civicrm_contribution.cancel_date IS NOT NULL
       GROUP BY civicrm_contribution.id
     ) as conts
     GROUP BY currency";