[Rt-commit] r6801 - in rtir/branches/2.2-TESTING: lib/RT/Action

[ruz at bestpractical.com]

Author: ruz
Date: Tue Jan 23 19:12:33 2007
New Revision: 6801

Modified:
   rtir/branches/2.2-TESTING/   (props changed)
   rtir/branches/2.2-TESTING/lib/RT/Action/RTIR_SetConstituency.pm

Log:
 r1855 at cubic-pc (orig r6791):  ruz | 2007-01-22 23:58:21 +0300
  r1853 at cubic-pc:  cubic | 2007-01-22 12:24:28 +0300
  * CF != 'xxx' autmatically adds 'OR CF IS NULL' in RT's code,
    so don't do this twice
 


Modified: rtir/branches/2.2-TESTING/lib/RT/Action/RTIR_SetConstituency.pm
==============================================================================
--- rtir/branches/2.2-TESTING/lib/RT/Action/RTIR_SetConstituency.pm	(original)
+++ rtir/branches/2.2-TESTING/lib/RT/Action/RTIR_SetConstituency.pm	Tue Jan 23 19:12:33 2007
@@ -41,9 +41,9 @@
         ." AND ( MemberOf = ". $ticket->Id ." OR HasMember = ". $ticket->Id ." )";
 
     if ( $constituency ) {
-        $query .= " AND ( CF.{_RTIR_Constituency} != '$constituency' OR CF.{_RTIR_Constituency} IS NULL )";
+        $query .= " AND CF.{_RTIR_Constituency} != '$constituency'";
     } else {
-        $query .= " AND ( CF.{_RTIR_Constituency} IS NOT NULL )";
+        $query .= " AND CF.{_RTIR_Constituency} IS NOT NULL";
     }
 
     my $tickets = RT::Tickets->new( $RT::SystemUser );