[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