summaryrefslogtreecommitdiffstats
path: root/Bugzilla/WebService
diff options
context:
space:
mode:
authorFrédéric Buclin <LpSolit@gmail.com>2012-11-26 20:20:29 +0100
committerFrédéric Buclin <LpSolit@gmail.com>2012-11-26 20:20:29 +0100
commit46e5d925026ccde28118bfcf28061c75f188b066 (patch)
treed9d95ff2e945ba67f1b8368cc33d24e38cd5dc6f /Bugzilla/WebService
parentccf2d3e5e476ce72cb95894b68358950bfa1d24b (diff)
downloadbugzilla-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.pm11
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".