[Rt-commit] r6791 - in rtir/branches/2.1-EXPERIMENTAL: .

ruz at bestpractical.com ruz at bestpractical.com
Mon Jan 22 15:58:23 EST 2007


Author: ruz
Date: Mon Jan 22 15:58:21 2007
New Revision: 6791

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

Log:
 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.1-EXPERIMENTAL/lib/RT/Action/RTIR_SetConstituency.pm
==============================================================================
--- rtir/branches/2.1-EXPERIMENTAL/lib/RT/Action/RTIR_SetConstituency.pm	(original)
+++ rtir/branches/2.1-EXPERIMENTAL/lib/RT/Action/RTIR_SetConstituency.pm	Mon Jan 22 15:58:21 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 );


More information about the Rt-commit mailing list