diff options
| author | Mark Wells <mark@freeside.biz> | 2013-08-15 00:08:24 -0700 |
|---|---|---|
| committer | Mark Wells <mark@freeside.biz> | 2013-08-15 00:08:24 -0700 |
| commit | 4771c3a939c45be1cc2814a5d4c2696cb55212f0 (patch) | |
| tree | d5108cd50381b294074ccd1a93d23e135ee64dc6 /httemplate/edit/bulk-part_pkg.html | |
| parent | ec34946605aefd8455b4d7a8bd197d8eabe7ce3c (diff) | |
fix some UI issues with the report class bulk edit, #24070
Diffstat (limited to 'httemplate/edit/bulk-part_pkg.html')
| -rw-r--r-- | httemplate/edit/bulk-part_pkg.html | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/httemplate/edit/bulk-part_pkg.html b/httemplate/edit/bulk-part_pkg.html index a1c6f0c9b..1773c9a8b 100644 --- a/httemplate/edit/bulk-part_pkg.html +++ b/httemplate/edit/bulk-part_pkg.html @@ -22,15 +22,12 @@ The following packages will be changed:<BR> % foreach my $num (sort keys %report_class) { <TR CLASS="row<%$row % 2%>"> <TD> -% if ( defined $initial_state{$num} ) { - <& /elements/checkbox.html, - field => 'report_option_'.$num, - value => 1, - curr_value => $initial_state{$num} - &> -% } else { +% if ( $initial_state{$num} == -1 ) { % # needs to be a tristate so that you can say "don't change it" <& /elements/checkbox-tristate.html, field => 'report_option_'.$num &> +% } else { +%# for visual consistency + <INPUT TYPE="checkbox" CLASS="partial" NAME="report_option_<%$num%>" VALUE="1" <% $initial_state{$num} ? 'CHECKED':'' %>><LABEL /> % } </TD> <TD><% $report_class{$num}->name %></TD> @@ -64,11 +61,12 @@ foreach my $num (keys %report_class) { } } if ( $yes and $no ) { - $initial_state{$num} = undef; + $initial_state{$num} = -1; } elsif ( $yes ) { $initial_state{$num} = 1; } elsif ( $no ) { - $initial_state{$num} = 0; + $initial_state{$num} = ''; } # else, uh, you didn't provide any pkgparts } +warn Dumper \%initial_state; </%init> |