[Rt-commit] r9099 - in rt/branches/3.6-RELEASE: html/REST/1.0/Forms/ticket

sunnavy at bestpractical.com sunnavy at bestpractical.com
Thu Sep 20 02:05:49 EDT 2007


Author: sunnavy
Date: Thu Sep 20 02:05:48 2007
New Revision: 9099

Modified:
   rt/branches/3.6-RELEASE/html/REST/1.0/Forms/ticket/default
   rt/branches/3.6-RELEASE/lib/t/regression/26command_line.t

Log:
A multi-valued custom field should end up with multiple rows in
ObjectCustomFieldValues

Modified: rt/branches/3.6-RELEASE/html/REST/1.0/Forms/ticket/default
==============================================================================
--- rt/branches/3.6-RELEASE/html/REST/1.0/Forms/ticket/default	(original)
+++ rt/branches/3.6-RELEASE/html/REST/1.0/Forms/ticket/default	Thu Sep 20 02:05:48 2007
@@ -317,9 +317,11 @@
                 while (my $v = $vals->Next()) {
                     $v->Delete();
                 }
-                ($n, $s) = $ticket->AddCustomFieldValue(
-                             Field => $cf, Value => $val );
-                $s =~ s/^# // if defined $s;
+                for ( split /,/, $val ) {
+                    ($n, $s) = $ticket->AddCustomFieldValue(
+                             Field => $cf, Value => $_ );
+                    $s =~ s/^# // if defined $s;
+                }
             }
         }
         elsif ($key ne 'id' && $key ne 'type' && $key ne 'creator') {

Modified: rt/branches/3.6-RELEASE/lib/t/regression/26command_line.t
==============================================================================
--- rt/branches/3.6-RELEASE/lib/t/regression/26command_line.t	(original)
+++ rt/branches/3.6-RELEASE/lib/t/regression/26command_line.t	Thu Sep 20 02:05:48 2007
@@ -173,11 +173,11 @@
 expect_send("edit ticket/$ticket_id set CF-MultipleCF$$=1,2,3 ", 'Changing CF...');
 expect_like(qr/Ticket $ticket_id updated/, 'Changed multiple cf');
 expect_send("show ticket/$ticket_id -f CF-MultipleCF$$", 'Checking new value');
-expect_like(qr/CF-MultipleCF$$: 1,2,3/i, 'Verified multiple cf change');
+expect_like(qr/CF-MultipleCF$$: 1,\s*2,\s*3/i, 'Verified multiple cf change');
 expect_send("edit ticket/$ticket_id set CF-MultipleCF$$=a,b,c ", 'Changing CF...');
 expect_like(qr/Ticket $ticket_id updated/, 'Changed multiple cf');
 expect_send("show ticket/$ticket_id -f CF-MultipleCF$$", 'Checking new value');
-expect_like(qr/CF-MultipleCF$$: a,b,c/i, 'Verified change');
+expect_like(qr/CF-MultipleCF$$: a,\s*b,\s*c/i, 'Verified change');
 expect_send("edit ticket/$ticket_id del CF-MultipleCF$$=a ", 'Changing CF...');
 expect_like(qr/Ticket $ticket_id updated/, 'del multiple cf');
 expect_send("show ticket/$ticket_id -f CF-MultipleCF$$", 'Checking new value');


More information about the Rt-commit mailing list