CRM-18356: Fix incorrect logic Activity Report has when excluding NULL / NOT NULL...

----------------------------------------
* CRM-18356: Activity Reports show null Target Name fields incorrectly for some Activities
  https://issues.civicrm.org/jira/browse/CRM-18356

diff --git a/CRM/Report/Form/Activity.php b/CRM/Report/Form/Activity.php
index 7a9372a15e022871d412c143f45277b750d9e974..67c4269f71967403084c2ad767029d51d7697f2b 100644 (file)
--- a/CRM/Report/Form/Activity.php
+++ b/CRM/Report/Form/Activity.php
@@ -580,7 +580,7 @@ class CRM_Report_Form_Activity extends CRM_Report_Form {
           }
           else {
             $op = CRM_Utils_Array::value("{$fieldName}_op", $this->_params);
-            if ($op && ($op != 'nnll' || $op != 'nll')) {
+            if ($op && ($op != 'nnll' && $op != 'nll')) {
               $clause = $this->whereClause($field,
                 $op,
                 CRM_Utils_Array::value("{$fieldName}_value", $this->_params),