RSS
To get this branch, use:
bzr branch /bugzilla/3.6

« back to all changes in this revision

Viewing changes to Bugzilla/Field/Choice.pm

  • Committer: Max Kanat-Alexander
  • Date: 2010-07-23 01:35:59 UTC
  • Revision ID: mkanat@bugzilla.org-20100723013559-wxdtrbgpzwyv7vn4
Bug 577054: Field::Choice was denying the deletion of any value if
the field had *any* value-controlling values.
r=LpSolit, a=LpSolit

Show diffs side-by-side

added added

removed removed

202
202
sub _check_if_controller {
203
203
    my $self = shift;
204
204
    my $vis_fields = $self->controls_visibility_of_fields;
205
 
    my $values     = $self->controlled_values;
206
 
    if (@$vis_fields || scalar(keys %$values)) {
 
205
    my $values = $self->controlled_values_array;
 
206
    if (@$vis_fields || @$values) {
207
207
        ThrowUserError('fieldvalue_is_controller',
208
208
            { value => $self, fields => [map($_->name, @$vis_fields)],
209
 
              vals => $values });
 
209
              vals => $self->controlled_values });
210
210
    }
211
211
}
212
212
 
 
213
sub controlled_values_array {
 
214
    my ($self) = @_;
 
215
    my $values = $self->controlled_values;
 
216
    return [map { @{ $values->{$_} } } keys %$values];
 
217
}
213
218
 
214
219
#############
215
220
# Accessors #