diff options
author | mark <mark> | 2010-10-05 17:32:24 +0000 |
---|---|---|
committer | mark <mark> | 2010-10-05 17:32:24 +0000 |
commit | 6889413a55f87e4b7f6b6637773f87a0de52b3b4 (patch) | |
tree | a2da6f7a1cf2964ab7c5986b51f7f63744e8d0cb | |
parent | e94bccb7296ad6c884005b6beb24a1c2716314a2 (diff) |
prevent broken custom field values, RT#9959
-rw-r--r-- | rt/share/html/Admin/CustomFields/Modify.html | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/rt/share/html/Admin/CustomFields/Modify.html b/rt/share/html/Admin/CustomFields/Modify.html index d2932d2..fc2c49d 100644 --- a/rt/share/html/Admin/CustomFields/Modify.html +++ b/rt/share/html/Admin/CustomFields/Modify.html @@ -199,6 +199,15 @@ if ( $ARGS{'Update'} && $id ne 'new' ) { push (@results, $msg); } + # Clean up values + foreach my $param (grep /^$paramtag-/, keys(%ARGS)) { + for ($ARGS{$param}) { + s/\r+\n/\n/g; + s/^\s+//; + s/\s+$//; + } + } + # Update any existing values my $values = $CustomFieldObj->ValuesObj; while ( my $value = $values->Next ) { @@ -206,7 +215,6 @@ if ( $ARGS{'Update'} && $id ne 'new' ) { my $param = join("-", $paramtag, $value->Id, $attr); next unless exists $ARGS{$param}; next if ($value->$attr()||'') eq ($ARGS{$param}||''); - my $mutator = "Set$attr"; my ($id, $msg) = $value->$mutator( $ARGS{$param} ); push (@results, $msg); |