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.

Timetrack fields, "Orig Est."+"Hours Left", accept numbers with commas or point
a16e8ee9 · Danny Dierickx · GitHub · 3c60fba2 · a16e8ee9
Commit a16e8ee9 authored Nov 08, 2016 by Danny Dierickx Committed by GitHub Nov 08, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

Bug.pm Bugzilla/Bug.pm +2 -2

No files found.
--- 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]); }