Timetrack fields, "Orig Est."+"Hours Left", accept numbers with commas or point

For non-US locales the bug entry/update page shows values with commas as decimal seperator in the timetracking fields "Orig Est." and "Hours Left". However when saving/updating bugzilla refuses these values. (see bug 1122419)
These 2 changed lines let bugzilla accept/process these "comma-values" as "point-values".
This patch *should* not influence systems with US locale.

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index 46b16ffd6c80be804dd91501995f18bf5a5138b9..f039ca75993931fe4c43ab04ee8ae7ab194f5e9c 100644 (file)
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -2683,7 +2683,7 @@ sub set_dup_id {
                                  extra_data => $self->dup_id });
     }
 }
-sub set_estimated_time { $_[0]->set('estimated_time', $_[1]); }
+sub set_estimated_time { my $et = $_[1]; $et =~ s/,/./g; $_[0]->set('estimated_time', $et); }
 sub _set_everconfirmed { $_[0]->set('everconfirmed', $_[1]); }
 sub set_flags {
     my ($self, $flags, $new_flags) = @_;
@@ -2852,7 +2852,7 @@ sub reset_qa_contact {
     my $comp = $self->component_obj;
     $self->set_qa_contact($comp->default_qa_contact);
 }
-sub set_remaining_time { $_[0]->set('remaining_time', $_[1]); }
+sub set_remaining_time { my $rt = $_[1]; $rt =~ s/,/./g; $_[0]->set('remaining_time', $rt); }
 # Used only when closing a bug or moving between closed states.
 sub _zero_remaining_time { $_[0]->{'remaining_time'} = 0; }
 sub set_reporter_accessible { $_[0]->set('reporter_accessible', $_[1]); }