diff options
| author | Frédéric Buclin <LpSolit@gmail.com> | 2012-11-26 20:20:29 +0100 |
|---|---|---|
| committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-11-26 20:20:29 +0100 |
| commit | 46e5d925026ccde28118bfcf28061c75f188b066 (patch) | |
| tree | d9d95ff2e945ba67f1b8368cc33d24e38cd5dc6f /Bugzilla/WebService | |
| parent | ccf2d3e5e476ce72cb95894b68358950bfa1d24b (diff) | |
| download | bugzilla-46e5d925026ccde28118bfcf28061c75f188b066.tar.gz bugzilla-46e5d925026ccde28118bfcf28061c75f188b066.tar.xz | |
Bug 718289: The deadline field should be visible by non-members of the timetracking group
r=dkl a=LpSolit
Diffstat (limited to 'Bugzilla/WebService')
| -rw-r--r-- | Bugzilla/WebService/Bug.pm | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm index 7a29d325f..f4cb53600 100644 --- a/Bugzilla/WebService/Bug.pm +++ b/Bugzilla/WebService/Bug.pm @@ -418,7 +418,7 @@ sub search { delete $params->{WHERE}; unless (Bugzilla->user->is_timetracker) { - delete $params->{$_} foreach qw(estimated_time remaining_time deadline); + delete $params->{$_} foreach TIMETRACKING_FIELDS; } # Do special search types for certain fields. @@ -851,6 +851,9 @@ sub _bug_to_hash { classification => $self->type('string', $bug->classification), component => $self->type('string', $bug->component), creation_time => $self->type('dateTime', $bug->creation_ts), + # No need to format $bug->deadline specially, because Bugzilla::Bug + # already does it for us. + deadline => $self->type('string', $bug->deadline), id => $self->type('int', $bug->bug_id), is_confirmed => $self->type('boolean', $bug->everconfirmed), last_change_time => $self->type('dateTime', $bug->delta_ts), @@ -943,9 +946,6 @@ sub _bug_to_hash { if (Bugzilla->user->is_timetracker) { $item{'estimated_time'} = $self->type('double', $bug->estimated_time); $item{'remaining_time'} = $self->type('double', $bug->remaining_time); - # No need to format $bug->deadline specially, because Bugzilla::Bug - # already does it for us. - $item{'deadline'} = $self->type('string', $bug->deadline); $item{'actual_time'} = $self->type('double', $bug->actual_time); } @@ -1782,9 +1782,6 @@ C<string> The login name of the person who filed this bug (the reporter). C<string> The day that this bug is due to be completed, in the format C<YYYY-MM-DD>. -If you are not in the time-tracking group, this field will not be included -in the return value. - =item C<depends_on> C<array> of C<int>s. The ids of bugs that this bug "depends on". |