[Rt-commit] r16256 - in rt/branches/3.999-DANGEROUS: lib/RT/Action

[sunnavy at bestpractical.com]

Author: sunnavy
Date: Thu Oct  9 03:55:14 2008
New Revision: 16256

Modified:
   rt/branches/3.999-DANGEROUS/   (props changed)
   rt/branches/3.999-DANGEROUS/lib/RT/Action/CreateTicketLinks.pm
   rt/branches/3.999-DANGEROUS/lib/RT/Action/DeleteTicketLinks.pm

Log:
 r17084 at sunnavys-mb:  sunnavy | 2008-10-09 15:53:36 +0800
 updated messages keeper to be an array


Modified: rt/branches/3.999-DANGEROUS/lib/RT/Action/CreateTicketLinks.pm
==============================================================================
--- rt/branches/3.999-DANGEROUS/lib/RT/Action/CreateTicketLinks.pm	(original)
+++ rt/branches/3.999-DANGEROUS/lib/RT/Action/CreateTicketLinks.pm	Thu Oct  9 03:55:14 2008
@@ -27,7 +27,6 @@
 
 =cut
 
-
 sub take_action {
     my $self = shift;
     my %map  = (
@@ -51,23 +50,24 @@
                     target => $value,
                     type   => $type,
                 );
-                $self->result->content('detailed_messages')->{$field} .= $msg;
+                push @{ $self->result->content('detailed_messages')->{$field} },
+                  $msg;
             }
-            for my $value ( split /\s+/, $self->argument_value( $map{$field} ) ) {
+            for my $value ( split /\s+/, $self->argument_value( $map{$field} ) )
+            {
                 next unless $value;
                 my ( $val, $msg ) = $ticket->add_link(
                     base => $value,
                     type => $type,
                 );
-                $self->result->content('detailed_messages')->{ $map{field} } .=
-                  "$msg.";
+                push @{ $self->result->content('detailed_messages')->{ $map{$field} } }, $msg;
             }
         }
 
         # now we handle merge_into stuff
         if ( my $merge_into = $self->argument_value('merge_into') ) {
             $merge_into =~ s/\s+//g;
-            my ( $val, $msg ) = $ticket->merge_into( $merge_into );
+            my ( $val, $msg ) = $ticket->merge_into($merge_into);
             $self->result->content('detailed_messages')->{'merge_into'} = $msg;
         }
     }

Modified: rt/branches/3.999-DANGEROUS/lib/RT/Action/DeleteTicketLinks.pm
==============================================================================
--- rt/branches/3.999-DANGEROUS/lib/RT/Action/DeleteTicketLinks.pm	(original)
+++ rt/branches/3.999-DANGEROUS/lib/RT/Action/DeleteTicketLinks.pm	Thu Oct  9 03:55:14 2008
@@ -29,8 +29,7 @@
                         target => $value,
                         type   => $type,
                     );
-                    $self->result->content('detailed_messages')->{$field} .=
-                      $msg;
+                    push @{ $self->result->content('detailed_messages')->{$field} }, $msg;
                 }
             }
             if ( my $v = $self->argument_value( $map{$field} ) ) {
@@ -39,7 +38,7 @@
                         base => $value,
                         type => $type,
                     );
-                    $self->result->content('detailed_messages')->{ $map{field} } .= "$msg.";
+                    push @{ $self->result->content('detailed_messages')->{ $map{$field} } }, $msg;
                 }
             }
         }