summaryrefslogtreecommitdiff
path: root/rt/share/html/Admin/CustomFields
diff options
context:
space:
mode:
authormark <mark>2010-10-05 17:32:24 +0000
committermark <mark>2010-10-05 17:32:24 +0000
commit6889413a55f87e4b7f6b6637773f87a0de52b3b4 (patch)
treea2da6f7a1cf2964ab7c5986b51f7f63744e8d0cb /rt/share/html/Admin/CustomFields
parente94bccb7296ad6c884005b6beb24a1c2716314a2 (diff)
prevent broken custom field values, RT#9959
Diffstat (limited to 'rt/share/html/Admin/CustomFields')
-rw-r--r--rt/share/html/Admin/CustomFields/Modify.html10
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);