diff options
author | Danny Dierickx <danny.dierickx@gmail.com> | 2016-11-08 14:31:52 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-11-08 14:31:52 +0100 |
commit | a16e8ee98e6b18f96da1faf86ad82cc20000856c (patch) | |
tree | 1a2be6553a9a55bb23faf835dfd99320b58e8e80 | |
parent | 3c60fba26b1fa51ec302d05dc6e76236d2f54ba4 (diff) | |
download | bugzilla-a16e8ee98e6b18f96da1faf86ad82cc20000856c.tar.gz bugzilla-a16e8ee98e6b18f96da1faf86ad82cc20000856c.tar.xz |
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.
-rw-r--r-- | Bugzilla/Bug.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 46b16ffd6..f039ca759 100644 --- 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]); } |