Bug 785729 - Webservice Bug.fields (_legal_field_values) should return isactive for...

r=dkl, a=LpSolit
- Missed some changed from earlier checkin

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index c787aed5cf2cbff40afe0cbba72804d75bea7234..75c3c2f61cc95a4973fafdb46bb5a0f3c2a94d39 100644 (file)
--- 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