From: Frank Becker <Frank@Frank-Becker.de>
Date: Fri, 31 Aug 2012 21:17:31 +0000 (-0400)
Subject: Bug 785729 - Webservice Bug.fields (_legal_field_values) should return isactive for... 
X-Git-Tag: bugzilla-4.4rc1~80
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=566b03f4af70a22212ff75d9ffe7fa111056333b;p=thirdparty%2Fbugzilla.git

Bug 785729 - Webservice Bug.fields (_legal_field_values) should return isactive for versions, components and milestones
r=dkl, a=LpSolit
- Missed some changed from earlier checkin
---

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index c787aed5cf..75c3c2f61c 100644
--- a/Bugzilla/WebService/Bug.pm
+++ b/Bugzilla/WebService/Bug.pm
@@ -167,7 +167,7 @@ sub _legal_field_values {
                     sort_key => $self->type('int', $sortkey),
                     sortkey  => $self->type('int', $sortkey), # deprecated
                     visibility_values => [$self->type('string', $product_name)],
-                    is_active         => $self->type('boolean', $value->is_active()),
+                    is_active         => $self->type('boolean', $value->is_active),
                 });
             }
         }
@@ -1193,8 +1193,9 @@ and C<visibility_values> will reflect which product(s) this value appears in.
 
 =item C<is_active>
 
-C<boolean> True when the version, target_milestone or component is active,
-false when the element is retired.
+C<boolean> This value is defined only for certain product specific fields
+such as version, target_milestone or component. When true, the value is active,
+otherwise the value is not active.
 
 =item C<description>
 
@@ -1252,6 +1253,8 @@ You specified an invalid field name or id.
 
 =item C<sortkey> was renamed to C<sort_key> in Bugzilla B<4.2>.
 
+=item C<is_active> return key for C<values> was added in Bugzilla B<4.4>.
+
 =back
 
 =back