View | Details | Raw Unified | Return to bug 35190
Collapse All | Expand All

(-)a/admin/additional-fields.pl (-19 / +22 lines)
Lines 49-69 my @messages; Link Here
49
49
50
if ( $op eq 'add' ) {
50
if ( $op eq 'add' ) {
51
    my $name = $input->param('name') // q{};
51
    my $name = $input->param('name') // q{};
52
    my $authorised_value_category = $input->param('authorised_value_category') // q{};
52
    my $authorised_value_category = $input->param('authorised_value_category');
53
    my $marcfield = $input->param('marcfield') // q{};
53
    my $marcfield = $input->param('marcfield') // q{};
54
    my $marcfield_mode = $input->param('marcfield_mode') // 'get';
54
    my $marcfield_mode = $input->param('marcfield_mode') // 'get';
55
    my $searchable = $input->param('searchable') ? 1 : 0;
55
    my $searchable = $input->param('searchable') ? 1 : 0;
56
    if ( $field_id and $name ) {
56
    if ( $field_id and $name ) {
57
        my $updated = 0;
57
        my $updated    = 0;
58
        my $set_fields = {
59
            name           => $name,
60
            marcfield      => $marcfield,
61
            marcfield_mode => $marcfield_mode,
62
            searchable     => $searchable,
63
        };
64
        $set_fields->{authorised_value_category} = $authorised_value_category if $authorised_value_category;
65
58
        eval {
66
        eval {
59
            my $af = Koha::AdditionalFields->find($field_id);
67
            my $af = Koha::AdditionalFields->find($field_id);
60
            $af->set({
68
            $af->set($set_fields);
61
                name => $name,
62
                authorised_value_category => $authorised_value_category,
63
                marcfield => $marcfield,
64
                marcfield_mode => $marcfield_mode,
65
                searchable => $searchable,
66
            });
67
            $updated = $af->store ? 1 : 0;
69
            $updated = $af->store ? 1 : 0;
68
        };
70
        };
69
        push @messages, {
71
        push @messages, {
Lines 71-86 if ( $op eq 'add' ) { Link Here
71
            number => $updated,
73
            number => $updated,
72
        };
74
        };
73
    } elsif ( $name ) {
75
    } elsif ( $name ) {
74
        my $inserted = 0;
76
        my $inserted   = 0;
77
        my $set_fields = {
78
            tablename      => $tablename,
79
            name           => $name,
80
            marcfield      => $marcfield,
81
            marcfield_mode => $marcfield_mode,
82
            searchable     => $searchable,
83
        };
84
        $set_fields->{authorised_value_category} = $authorised_value_category if $authorised_value_category;
85
75
        eval {
86
        eval {
76
            my $af = Koha::AdditionalField->new({
87
            my $af = Koha::AdditionalField->new($set_fields);
77
                tablename => $tablename,
78
                name => $name,
79
                authorised_value_category => $authorised_value_category,
80
                marcfield => $marcfield,
81
                marcfield_mode => $marcfield_mode,
82
                searchable => $searchable,
83
            });
84
            $inserted = $af->store ? 1 : 0;
88
            $inserted = $af->store ? 1 : 0;
85
        };
89
        };
86
        push @messages, {
90
        push @messages, {
87
- 

Return to bug 35190