X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FCVar.pm;h=0264d8ab09ceadb707dc80006d6f7b081e6fccfd;hb=6f1b36cfb8b26f6c8540ba5f36e48b9188d9470a;hp=b15f82be2e6e4ebcb0f08fd82278df45a7bbc687;hpb=873a9a839d7c51f1dfbfb3a36bd4d7b152a1e818;p=kivitendo-erp.git diff --git a/SL/CVar.pm b/SL/CVar.pm index b15f82be2..0264d8ab0 100644 --- a/SL/CVar.pm +++ b/SL/CVar.pm @@ -55,6 +55,10 @@ SQL } $self->_unpack_flags($config); + + my $cvar_config = SL::DB::CustomVariableConfig->new(id => $config->{id})->load; + @{$config->{'partsgroups'}} = map {$_->id} @{$cvar_config->partsgroups}; + } $::form->{CVAR_CONFIGS}->{$params{module}} = $configs; } @@ -132,6 +136,8 @@ sub get_custom_variables { $act_var = $h_var->fetchrow_hashref(); $valid = $self->get_custom_variables_validity(config_id => $cvar->{id}, trans_id => $params{trans_id}); + } else { + $valid = !$cvar->{flag_defaults_to_invalid}; } if ($act_var) { @@ -145,7 +151,7 @@ sub get_custom_variables { : $act_var->{text_value}; $cvar->{valid} = $valid; } else { - $cvar->{valid} = 1; + $cvar->{valid} = $valid // 1; if ($cvar->{type} eq 'date') { if ($cvar->{default_value} eq 'NOW') { @@ -277,13 +283,20 @@ sub render_inputs { my $myconfig = \%main::myconfig; my $form = $main::form; - my %options = ( name_prefix => "$params{name_prefix}", - name_postfix => "$params{name_postfix}", - hide_non_editable => $params{hide_non_editable}, + my %options = ( name_prefix => "$params{name_prefix}", + name_postfix => "$params{name_postfix}", + hide_non_editable => $params{hide_non_editable}, show_disabled_message => $params{show_disabled_message}, ); + # should this cvar be filtered by partsgroups? foreach my $var (@{ $params{variables} }) { + if ($var->{flag_partsgroup_filter}) { + if (!$params{partsgroup_id} || (!grep {$params{partsgroup_id} == $_} @{ $var->{partsgroups} })) { + $var->{partsgroup_filtered} = 1; + } + } + $var->{HTML_CODE} = $form->parse_html_template('amcvar/render_inputs', { var => $var, %options }); $var->{VALID_BOX} = $form->parse_html_template('amcvar/render_checkboxes', { var => $var, %options }); }