[Rt-commit] r12749 - in rt/branches/3.8-TESTING: . lib/RT/Interface sbin share/html/NoAuth/css/web2

[sartak at bestpractical.com]

Author: sartak
Date: Thu May 29 13:22:41 2008
New Revision: 12749

Modified:
   rt/branches/3.8-TESTING/   (props changed)
   rt/branches/3.8-TESTING/lib/RT/Interface/Web.pm
   rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm
   rt/branches/3.8-TESTING/sbin/rt-test-dependencies.in
   rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css

Log:


Modified: rt/branches/3.8-TESTING/lib/RT/Interface/Web.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Interface/Web.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Interface/Web.pm	Thu May 29 13:22:41 2008
@@ -952,7 +952,6 @@
       Queue
     );
 
-    my @results;
 
     if ( $ARGSRef->{'Queue'} and ( $ARGSRef->{'Queue'} !~ /^(\d+)$/ ) ) {
         my $tempqueue = RT::Queue->new($RT::SystemUser);
@@ -962,21 +961,12 @@
         }
     }
 
-    # On queue change, change queue for reminders too
-    if ( $ARGSRef->{'Queue'} and ( $TicketObj->QueueObj->Id != $ARGSRef->{'Queue'} ) ) {
-        my $reminder_collection = $TicketObj->Reminders->Collection;
-        while ( my $reminder = $reminder_collection->Next ) {
-            my ($val, $msg) = $reminder->SetQueue($ARGSRef->{'Queue'});
-            push @results, $msg if ( !$val ); 
-        }
-    }
-
 
     # Status isn't a field that can be set to a null value.
     # RT core complains if you try
     delete $ARGSRef->{'Status'} unless $ARGSRef->{'Status'};
     
-    push @results, UpdateRecordObject(
+    my @results = UpdateRecordObject(
         AttributesRef => \@attribs,
         Object        => $TicketObj,
         ARGSRef       => $ARGSRef,

Modified: rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm
==============================================================================
--- rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm	(original)
+++ rt/branches/3.8-TESTING/lib/RT/Ticket_Overlay.pm	Thu May 29 13:22:41 2008
@@ -1703,6 +1703,12 @@
         $RT::Logger->error("Couldn't set owner on queue change: $msg") unless $status;
     }
 
+    # On queue change, change queue for reminders too
+    my $reminder_collection = $self->Reminders->Collection;
+    while ( my $reminder = $reminder_collection->Next ) {
+        return ( 0, $self->loc("Queue change failed for reminder [_1]: [_2]", $reminder->Id ,$msg) ) unless $val;
+    }
+
     return ( $self->_Set( Field => 'Queue', Value => $NewQueueObj->Id() ) );
 }
 

Modified: rt/branches/3.8-TESTING/sbin/rt-test-dependencies.in
==============================================================================
--- rt/branches/3.8-TESTING/sbin/rt-test-dependencies.in	(original)
+++ rt/branches/3.8-TESTING/sbin/rt-test-dependencies.in	Thu May 29 13:22:41 2008
@@ -87,7 +87,7 @@
     'with-CLI' => 1,
     'with-MAILGATE' => 1, 
     'with-DEV' => @RT_DEVEL_MODE@, 
-    'with-STANDALONE' => 0,
+    'with-STANDALONE' => 1,
     'with-GPG' => @RT_GPG@,
     'with-ICAL' => 1,
     'with-SMTP' => 1,
@@ -253,7 +253,7 @@
 HTTP::Server::Simple 0.07
 HTTP::Server::Simple::Mason 0.09
 Net::Server
-Net::Server::PreForking
+Net::Server::PreFork
 .
 
 $deps{'MAILGATE'} = [ text_to_hash( << '.') ];

Modified: rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css
==============================================================================
--- rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css	(original)
+++ rt/branches/3.8-TESTING/share/html/NoAuth/css/web2/ticket.css	Thu May 29 13:22:41 2008
@@ -122,7 +122,7 @@
 
 
 
-div#ticket-history td.content {
+div#ticket-history>td.content {
  position: relative;
  padding-right: 1em;
  font-size: 1.1em;
@@ -156,7 +156,7 @@
   color: #666;
 }
 
-div#ticket-history .messagebody {
+div#ticket-history>.messagebody {
  font-size: 1.1em;
  padding-left: 1em;
  margin-top: 0.5em;