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

(-)a/Koha/Schema/Result/SearchField.pm (+6 lines)
Lines 68-73 __PACKAGE__->add_columns( Link Here
68
    extra => { list => ["", "string", "date", "number", "boolean", "sum"] },
68
    extra => { list => ["", "string", "date", "number", "boolean", "sum"] },
69
    is_nullable => 0,
69
    is_nullable => 0,
70
  },
70
  },
71
  "boost",
72
  { data_type => "decimal", size => [ 5, 2 ], is_nullable => 1 },
73
  "staff_client",
74
  { data_type => "tinyint", size => 1, is_nullable => 0, default => 1 },
75
  "opac",
76
  { data_type => "tinyint", size => 1, is_nullable => 0, default => 1 },
71
);
77
);
72
78
73
=head1 PRIMARY KEY
79
=head1 PRIMARY KEY
(-)a/Koha/Schema/Result/SearchMarcToField.pm (+2 lines)
Lines 71-76 __PACKAGE__->add_columns( Link Here
71
  { data_type => "tinyint", default_value => 0, is_nullable => 1 },
71
  { data_type => "tinyint", default_value => 0, is_nullable => 1 },
72
  "sort",
72
  "sort",
73
  { data_type => "tinyint", is_nullable => 1 },
73
  { data_type => "tinyint", is_nullable => 1 },
74
  "search",
75
  { data_type => "tinyint", size => 1, is_nullable => 0, default => 1 },
74
);
76
);
75
77
76
=head1 PRIMARY KEY
78
=head1 PRIMARY KEY
(-)a/Koha/SearchEngine/Elasticsearch.pm (-5 / +4 lines)
Lines 173-183 sub get_elasticsearch_mappings { Link Here
173
    # TODO cache in the object?
173
    # TODO cache in the object?
174
    my $mappings = {
174
    my $mappings = {
175
        data => {
175
        data => {
176
            _all => {type => "string", analyzer => "analyser_standard"},
177
            properties => {
176
            properties => {
178
                record => {
177
                record => {
179
                    store          => "true",
178
                    store          => "true",
180
                    include_in_all => JSON::false,
181
                    type           => "text",
179
                    type           => "text",
182
                },
180
                },
183
            }
181
            }
Lines 226-232 sub get_elasticsearch_mappings { Link Here
226
                    search_analyzer => "analyser_phrase",
224
                    search_analyzer => "analyser_phrase",
227
                    analyzer  => "analyser_phrase",
225
                    analyzer  => "analyser_phrase",
228
                    type            => "text",
226
                    type            => "text",
229
                    include_in_all  => JSON::false,
230
                    fields          => {
227
                    fields          => {
231
                        phrase => {
228
                        phrase => {
232
                            type            => "keyword",
229
                            type            => "keyword",
Lines 286-296 sub reset_elasticsearch_mappings { Link Here
286
        while ( my ( $field_name, $data ) = each %$fields ) {
283
        while ( my ( $field_name, $data ) = each %$fields ) {
287
            my $field_type = $data->{type};
284
            my $field_type = $data->{type};
288
            my $field_label = $data->{label};
285
            my $field_label = $data->{label};
286
            my $staff_client = exists $data->{staff_client} ? $data->{staff_client} : 1;
287
            my $opac = exists $data->{opac} ? $data->{opac} : 1;
289
            my $mappings = $data->{mappings};
288
            my $mappings = $data->{mappings};
290
            my $search_field = Koha::SearchFields->find_or_create({ name => $field_name, label => $field_label, type => $field_type }, { key => 'name' });
289
            my $search_field = Koha::SearchFields->find_or_create({ name => $field_name, label => $field_label, type => $field_type, staff_client => $staff_client, opac => $opac }, { key => 'name' });
291
            for my $mapping ( @$mappings ) {
290
            for my $mapping ( @$mappings ) {
292
                my $marc_field = Koha::SearchMarcMaps->find_or_create({ index_name => $index_name, marc_type => $mapping->{marc_type}, marc_field => $mapping->{marc_field} });
291
                my $marc_field = Koha::SearchMarcMaps->find_or_create({ index_name => $index_name, marc_type => $mapping->{marc_type}, marc_field => $mapping->{marc_field} });
293
                $search_field->add_to_search_marc_maps($marc_field, { facet => $mapping->{facet} || 0, suggestible => $mapping->{suggestible} || 0, sort => $mapping->{sort} } );
292
                $search_field->add_to_search_marc_maps($marc_field, { facet => $mapping->{facet} || 0, suggestible => $mapping->{suggestible} || 0, sort => $mapping->{sort}, search => $mapping->{search} || 1 } );
294
            }
293
            }
295
        }
294
        }
296
    }
295
    }
(-)a/Koha/SearchEngine/Elasticsearch/QueryBuilder.pm (-27 / +119 lines)
Lines 48-53 use URI::Escape; Link Here
48
48
49
use C4::Context;
49
use C4::Context;
50
use Koha::Exceptions;
50
use Koha::Exceptions;
51
use Koha::Caches;
51
52
52
=head2 build_query
53
=head2 build_query
53
54
Lines 90-96 sub build_query { Link Here
90
            query            => $query,
91
            query            => $query,
91
            fuzziness        => $fuzzy_enabled ? 'auto' : '0',
92
            fuzziness        => $fuzzy_enabled ? 'auto' : '0',
92
            default_operator => 'AND',
93
            default_operator => 'AND',
93
            default_field    => '_all',
94
            fields          => $self->_query_string_fields({ is_opac => $options{is_opac} }),
94
            lenient          => JSON::true,
95
            lenient          => JSON::true,
95
        }
96
        }
96
    };
97
    };
Lines 233-238 sub build_query_compat { Link Here
233
    my %options;
234
    my %options;
234
    $options{sort} = \@sort_params;
235
    $options{sort} = \@sort_params;
235
    $options{expanded_facet} = $params->{expanded_facet};
236
    $options{expanded_facet} = $params->{expanded_facet};
237
    $options{is_opac} = exists $params->{is_opac} ? $params->{is_opac} : 0;
236
    my $query = $self->build_query( $query_str, %options );
238
    my $query = $self->build_query( $query_str, %options );
237
239
238
    #die Dumper($query);
240
    #die Dumper($query);
Lines 293-308 sub build_authorities_query { Link Here
293
    my @filter_parts;
295
    my @filter_parts;
294
    foreach my $s ( @{ $search->{searches} } ) {
296
    foreach my $s ( @{ $search->{searches} } ) {
295
        my ( $wh, $op, $val ) = @{$s}{qw(where operator value)};
297
        my ( $wh, $op, $val ) = @{$s}{qw(where operator value)};
296
        $wh = '_all' if $wh eq '';
298
        if (!$wh) {
297
        if ( $op eq 'is' || $op eq '=' ) {
299
            my %query_string_params = (
298
300
                default_operator => 'AND',
301
            );
302
            if ( $op eq 'is' || $op eq '=' ) {
303
                push @query_parts, {
304
                    multi_match => {
305
                        query => $val,
306
                        fields => $self->_query_string_fields({ subfield => 'raw' }),
307
                    }
308
                };
309
            }
310
            elsif ( $op eq 'exact' ) {
311
                # left and right truncation, otherwise an exact phrase
312
                push @query_parts, {
313
                    query_string => {
314
                        query => qq("$val"),
315
                        fields => $self->_query_string_fields({ subfield => 'phrase' }),
316
                        %query_string_params,
317
                    }
318
                };
319
            }
320
            elsif ( $op eq 'start') {
321
                push @query_parts, {
322
                    query_string => {
323
                        query => "$val*",
324
                        $self->_query_string_fields({ subfield => 'phrase' }),
325
                        %query_string_params,
326
                    }
327
                };
328
            }
329
            else {
330
                push @query_parts, {
331
                    query_string => {
332
                        query => $val,
333
                        fields => $self->_query_string_fields(),
334
                        %query_string_params,
335
                    }
336
                };
337
            }
338
        }
339
        elsif ( $op eq 'is' || $op eq '=' ) {
299
            # look for something that matches completely
340
            # look for something that matches completely
300
            # note, '=' is about numerical vals. May need special handling.
341
            # note, '=' is about numerical vals. May need special handling.
301
            # _allphrase is a special field that only groups the exact
342
            # raw is a special subfield that only groups the exact
302
            # matches. Also, we lowercase our search because the ES
343
            # matches. Also, we lowercase our search because the ES
303
            # index lowercases its values, and term searches don't get the
344
            # index lowercases its values, and term searches don't get the
304
            # search analyzer applied to them.
345
            # search analyzer applied to them.
305
            push @filter_parts, { term => { "$wh.phrase" => lc $val } };
346
            push @query_parts, { term => { "$wh.raw" => lc $val } };
306
        }
347
        }
307
        elsif ( $op eq 'exact' ) {
348
        elsif ( $op eq 'exact' ) {
308
349
Lines 320-332 sub build_authorities_query { Link Here
320
        }
361
        }
321
    }
362
    }
322
363
323
    # Merge the query and filter parts appropriately
324
    # 'should' behaves like 'or', if we want 'and', use 'must'
325
    my $query_part  = { bool => { should => \@query_parts } };
326
    my $filter_part = { bool => { should => \@filter_parts } };
327
328
    # We need to add '.phrase' to all the sort headings otherwise it'll sort
364
    # We need to add '.phrase' to all the sort headings otherwise it'll sort
329
    # based on the tokenised form.
365
    # based on the tokenised form.
366
    # TODO: .phrase currently contains the tokenized form, so this will not work
367
    # instead .raw should be used?
330
    my %s;
368
    my %s;
331
    if ( exists $search->{sort} ) {
369
    if ( exists $search->{sort} ) {
332
        foreach my $k ( keys %{ $search->{sort} } ) {
370
        foreach my $k ( keys %{ $search->{sort} } ) {
Lines 339-359 sub build_authorities_query { Link Here
339
    # extract the sort stuff
377
    # extract the sort stuff
340
    my %sort;
378
    my %sort;
341
    %sort = ( sort => [ $search->{sort} ] ) if exists $search->{sort};
379
    %sort = ( sort => [ $search->{sort} ] ) if exists $search->{sort};
342
    my $query;
380
381
    # Merge the query and filter parts appropriately
382
    # 'should' behaves like 'or', if we want 'and', use 'must'
383
    my $elastic_query = {};
343
    if (@filter_parts) {
384
    if (@filter_parts) {
344
        $query =
385
        $elastic_query->{bool}->{filter} = {
345
          { query =>
386
            bool => {
346
              { filtered => { filter => $filter_part, query => $query_part } }
387
                must => \@filter_parts,
347
          };
388
            }
348
    }
389
        };
349
    else {
350
        $query = { query => $query_part };
351
    }
390
    }
352
    $query = { %$query, %sort };
391
    if (@query_parts) {
353
    return $query;
392
        $elastic_query->{bool}->{should} = \@query_parts;
393
    };
394
    my $params = { query => $elastic_query };
395
    return { %$params, %sort };
354
}
396
}
355
397
356
357
=head2 build_authorities_query_compat
398
=head2 build_authorities_query_compat
358
399
359
    my ($query) =
400
    my ($query) =
Lines 513-519 types. Link Here
513
=cut
554
=cut
514
555
515
our %index_field_convert = (
556
our %index_field_convert = (
516
    'kw'      => '_all',
557
    'kw'      => '',
517
    'ti'      => 'title',
558
    'ti'      => 'title',
518
    'au'      => 'author',
559
    'au'      => 'author',
519
    'su'      => 'subject',
560
    'su'      => 'subject',
Lines 539-545 sub _convert_index_fields { Link Here
539
    # If a field starts with mc- we save it as it's used (and removed) later
580
    # If a field starts with mc- we save it as it's used (and removed) later
540
    # when joining things, to indicate we make it an 'OR' join.
581
    # when joining things, to indicate we make it an 'OR' join.
541
    # (Sorry, this got a bit ugly after special cases were found.)
582
    # (Sorry, this got a bit ugly after special cases were found.)
542
    grep { $_->{field} } map {
583
    map {
543
        my ( $f, $t ) = split /,/;
584
        my ( $f, $t ) = split /,/;
544
        my $mc = '';
585
        my $mc = '';
545
        if ($f =~ /^mc-/) {
586
        if ($f =~ /^mc-/) {
Lines 551-557 sub _convert_index_fields { Link Here
551
            type  => $index_type_convert{ $t // '__default' }
592
            type  => $index_type_convert{ $t // '__default' }
552
        };
593
        };
553
        $r->{field} = ($mc . $r->{field}) if $mc && $r->{field};
594
        $r->{field} = ($mc . $r->{field}) if $mc && $r->{field};
554
        $r;
595
        $r->{field} ? $r : undef;
555
    } @indexes;
596
    } @indexes;
556
}
597
}
557
598
Lines 580-587 sub _convert_index_strings { Link Here
580
            push @res, $s;
621
            push @res, $s;
581
            next;
622
            next;
582
        }
623
        }
583
        push @res, $conv->{field} . ":"
624
        push @res, ($conv->{field} ? $conv->{field} . ':' : '')
584
          . $self->_modify_string_by_type( %$conv, operand => $term );
625
            . $self->_modify_string_by_type( %$conv, operand => $term );
585
    }
626
    }
586
    return @res;
627
    return @res;
587
}
628
}
Lines 813-816 sub _truncate_terms { Link Here
813
    return join ' ', @terms;
854
    return join ' ', @terms;
814
}
855
}
815
856
857
sub _query_string_fields {
858
    my ($self, $params) = @_;
859
    $params //= {
860
        is_opac => 0,
861
        # This is a terrible hack for authorities build_authorities_query
862
        # can hopefully be removed in the future
863
        subfield => undef,
864
    };
865
866
    my $cache = Koha::Caches->get_instance();
867
    my $cache_key = 'elasticsearch_search_fields' . ($params->{is_opac} ? '_opac' : '_staff_client');
868
    my $search_fields = $cache->get_from_cache($cache_key, { unsafe => 1 });
869
870
    if (!$search_fields) {
871
        my $schema = Koha::Database->schema;
872
        my $result = $schema->resultset('SearchField')->search(
873
            {
874
                $params->{is_opac} ? (
875
                    'opac' => 1,
876
                ) : (
877
                    'staff_client' => 1
878
                ),
879
                'search_marc_map.index_name' => $self->index,
880
                'search_marc_map.marc_type' => C4::Context->preference('marcflavour'),
881
                'search_marc_to_fields.search' => 1,
882
            },
883
            {
884
                collapse => 1,
885
                join => {search_marc_to_fields => 'search_marc_map'},
886
            }
887
        );
888
        my @search_fields;
889
        while (my $search_field = $result->next) {
890
            push @search_fields, $search_field->name .
891
                ($search_field->boost ? '^' . $search_field->boost : '');
892
        }
893
        $search_fields = \@search_fields;
894
        $cache->set_in_cache($cache_key, $search_fields);
895
    }
896
    if ($params->{subfield}) {
897
        my $subfield = $params->{subfield};
898
        $search_fields = [
899
            map {
900
                my ($field, $boost) = $_ =~ /^([^\^]+)(?:\^(.+))?$/;
901
                "${field}.${subfield}" . ($boost ? "^$boost" : '');
902
            } @{$search_fields}
903
        ];
904
    }
905
    return $search_fields;
906
}
907
816
1;
908
1;
(-)a/admin/searchengine/elasticsearch/mappings.pl (-15 / +59 lines)
Lines 16-21 Link Here
16
# along with Koha; if not, see <http://www.gnu.org/licenses>.
16
# along with Koha; if not, see <http://www.gnu.org/licenses>.
17
17
18
use Modern::Perl;
18
use Modern::Perl;
19
use Scalar::Util qw(looks_like_number);
19
use CGI;
20
use CGI;
20
use C4::Koha;
21
use C4::Koha;
21
use C4::Output;
22
use C4::Output;
Lines 24-29 use C4::Auth; Link Here
24
use Koha::SearchEngine::Elasticsearch;
25
use Koha::SearchEngine::Elasticsearch;
25
use Koha::SearchMarcMaps;
26
use Koha::SearchMarcMaps;
26
use Koha::SearchFields;
27
use Koha::SearchFields;
28
use Koha::Caches;
27
29
28
my $input = new CGI;
30
my $input = new CGI;
29
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
31
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
Lines 44-49 my $schema = $database->schema; Link Here
44
46
45
my $marc_type = lc C4::Context->preference('marcflavour');
47
my $marc_type = lc C4::Context->preference('marcflavour');
46
48
49
my $cache = Koha::Caches->get_instance();
50
my $clear_cache = sub {
51
    $cache->clear_from_cache('elasticsearch_search_fields_staff_client');
52
    $cache->clear_from_cache('elasticsearch_search_fields_opac');
53
};
54
47
if ( $op eq 'edit' ) {
55
if ( $op eq 'edit' ) {
48
56
49
    $schema->storage->txn_begin;
57
    $schema->storage->txn_begin;
Lines 51-62 if ( $op eq 'edit' ) { Link Here
51
    my @field_name = $input->param('search_field_name');
59
    my @field_name = $input->param('search_field_name');
52
    my @field_label = $input->param('search_field_label');
60
    my @field_label = $input->param('search_field_label');
53
    my @field_type = $input->param('search_field_type');
61
    my @field_type = $input->param('search_field_type');
62
    my @field_boost = $input->param('search_field_boost');
63
    my @field_staff_client = $input->param('search_field_staff_client');
64
    my @field_opac = $input->param('search_field_opac');
54
65
55
    my @index_name          = $input->param('mapping_index_name');
66
    my @index_name          = $input->param('mapping_index_name');
56
    my @search_field_name  = $input->param('mapping_search_field_name');
67
    my @search_field_name   = $input->param('mapping_search_field_name');
57
    my @mapping_sort        = $input->param('mapping_sort');
68
    my @mapping_sort        = $input->param('mapping_sort');
58
    my @mapping_facet       = $input->param('mapping_facet');
69
    my @mapping_facet       = $input->param('mapping_facet');
59
    my @mapping_suggestible = $input->param('mapping_suggestible');
70
    my @mapping_suggestible = $input->param('mapping_suggestible');
71
    my @mapping_search      = $input->param('mapping_search');
60
    my @mapping_marc_field  = $input->param('mapping_marc_field');
72
    my @mapping_marc_field  = $input->param('mapping_marc_field');
61
73
62
    eval {
74
    eval {
Lines 65-73 if ( $op eq 'edit' ) { Link Here
65
            my $field_name = $field_name[$i];
77
            my $field_name = $field_name[$i];
66
            my $field_label = $field_label[$i];
78
            my $field_label = $field_label[$i];
67
            my $field_type = $field_type[$i];
79
            my $field_type = $field_type[$i];
80
            my $field_boost = $field_boost[$i];
81
            my $field_staff_client = $field_staff_client[$i];
82
            my $field_opac = $field_opac[$i];
68
            my $search_field = Koha::SearchFields->find( { name => $field_name }, { key => 'name' } );
83
            my $search_field = Koha::SearchFields->find( { name => $field_name }, { key => 'name' } );
69
            $search_field->label($field_label);
84
            $search_field->label($field_label);
70
            $search_field->type($field_type);
85
            $search_field->type($field_type);
86
            $search_field->boost($field_boost) if looks_like_number($field_boost);
87
            $search_field->staff_client($field_staff_client ? 1 : 0);
88
            $search_field->opac($field_opac ? 1 : 0);
71
            $search_field->store;
89
            $search_field->store;
72
        }
90
        }
73
91
Lines 75-92 if ( $op eq 'edit' ) { Link Here
75
93
76
        for my $i ( 0 .. scalar(@index_name) - 1 ) {
94
        for my $i ( 0 .. scalar(@index_name) - 1 ) {
77
            my $index_name          = $index_name[$i];
95
            my $index_name          = $index_name[$i];
78
            my $search_field_name  = $search_field_name[$i];
96
            my $search_field_name   = $search_field_name[$i];
79
            my $mapping_marc_field  = $mapping_marc_field[$i];
97
            my $mapping_marc_field  = $mapping_marc_field[$i];
80
            my $mapping_facet       = $mapping_facet[$i];
98
            my $mapping_facet       = $mapping_facet[$i];
81
            my $mapping_suggestible = $mapping_suggestible[$i];
99
            my $mapping_suggestible = $mapping_suggestible[$i];
82
            my $mapping_sort        = $mapping_sort[$i];
100
            my $mapping_sort        = $mapping_sort[$i] eq 'undef' ? undef : $mapping_sort[$i];
83
            $mapping_sort = undef if $mapping_sort eq 'undef';
101
            my $mapping_search      = $mapping_search[$i];
84
102
85
            my $search_field = Koha::SearchFields->find({ name => $search_field_name }, { key => 'name' });
103
            my $search_field = Koha::SearchFields->find({ name => $search_field_name }, { key => 'name' });
86
            # TODO Check mapping format
104
            # TODO Check mapping format
87
            my $marc_field = Koha::SearchMarcMaps->find_or_create({ index_name => $index_name, marc_type => $marc_type, marc_field => $mapping_marc_field });
105
            my $marc_field = Koha::SearchMarcMaps->find_or_create({
88
            $search_field->add_to_search_marc_maps($marc_field, { facet => $mapping_facet, suggestible => $mapping_suggestible, sort => $mapping_sort } );
106
                index_name => $index_name,
89
107
                marc_type => $marc_type,
108
                marc_field => $mapping_marc_field
109
            });
110
            $search_field->add_to_search_marc_maps($marc_field, {
111
                facet => $mapping_facet,
112
                suggestible => $mapping_suggestible,
113
                sort => $mapping_sort,
114
                search => $mapping_search,
115
            });
90
        }
116
        }
91
    };
117
    };
92
    if ($@) {
118
    if ($@) {
Lines 96-101 if ( $op eq 'edit' ) { Link Here
96
        push @messages, { type => 'message', code => 'success_on_update' };
122
        push @messages, { type => 'message', code => 'success_on_update' };
97
        $schema->storage->txn_commit;
123
        $schema->storage->txn_commit;
98
    }
124
    }
125
    $clear_cache->();
99
}
126
}
100
elsif( $op eq 'reset' ) {
127
elsif( $op eq 'reset' ) {
101
    # TODO Move this feature to the interface
128
    # TODO Move this feature to the interface
Lines 104-135 elsif( $op eq 'reset' ) { Link Here
104
        Koha::SearchMarcMaps->search->delete;
131
        Koha::SearchMarcMaps->search->delete;
105
        Koha::SearchEngine::Elasticsearch->reset_elasticsearch_mappings;
132
        Koha::SearchEngine::Elasticsearch->reset_elasticsearch_mappings;
106
    }
133
    }
134
    $clear_cache->();
107
}
135
}
108
136
109
my @indexes;
137
my @indexes;
110
138
111
for my $index_name (qw| biblios authorities |) {
139
for my $index_name (qw| biblios authorities |) {
112
    my $search_fields = Koha::SearchFields->search(
140
    my $search_fields = Koha::SearchFields->search(
113
        { 'search_marc_map.index_name' => $index_name, 'search_marc_map.marc_type' => $marc_type, },
141
        {
114
        {   join => { search_marc_to_fields => 'search_marc_map' },
142
            'search_marc_map.index_name' => $index_name,
115
            '+select' => [ 'search_marc_to_fields.facet', 'search_marc_to_fields.suggestible', 'search_marc_to_fields.sort', 'search_marc_map.marc_field' ],
143
            'search_marc_map.marc_type' => $marc_type,
116
            '+as'     => [ 'facet',                       'suggestible',                       'sort',                       'marc_field' ],
144
        },
145
        {
146
            join => { search_marc_to_fields => 'search_marc_map' },
147
            '+select' => [
148
                'search_marc_to_fields.facet',
149
                'search_marc_to_fields.suggestible',
150
                'search_marc_to_fields.sort',
151
                'search_marc_to_fields.search',
152
                'search_marc_map.marc_field',
153
            ],
154
            '+as' => [
155
                'facet',
156
                'suggestible',
157
                'sort',
158
                'search',
159
                'marc_field',
160
            ],
117
        }
161
        }
118
    );
162
    );
119
163
120
    my @mappings;
164
    my @mappings;
121
    while ( my $s = $search_fields->next ) {
165
    while ( my $s = $search_fields->next ) {
122
        push @mappings,
166
        push @mappings, {
123
          { search_field_name  => $s->name,
167
            search_field_name  => $s->name,
124
            search_field_label => $s->label,
168
            search_field_label => $s->label,
125
            search_field_type  => $s->type,
169
            search_field_type  => $s->type,
126
            marc_field         => $s->get_column('marc_field'),
170
            marc_field         => $s->get_column('marc_field'),
127
            sort               => $s->get_column('sort') // 'undef', # To avoid warnings "Use of uninitialized value in lc"
171
            sort               => $s->get_column('sort') // 'undef', # To avoid warnings "Use of uninitialized value in lc"
128
            suggestible        => $s->get_column('suggestible'),
172
            suggestible        => $s->get_column('suggestible'),
129
            facet              => $s->get_column('facet'),
173
            facet              => $s->get_column('facet'),
130
          };
174
            search             => $s->get_column('search'),
175
        };
131
    }
176
    }
132
133
    push @indexes, { index_name => $index_name, mappings => \@mappings };
177
    push @indexes, { index_name => $index_name, mappings => \@mappings };
134
}
178
}
135
179
(-)a/installer/data/mysql/atomicupdate/bug_20589_add_columns.perl (+17 lines)
Line 0 Link Here
1
$DBversion = 'XXX';
2
if (CheckVersion($DBversion)) {
3
  if (!column_exists('search_marc_to_field', 'search')) {
4
    $dbh->do("ALTER TABLE `search_marc_to_field` ADD COLUMN `search` tinyint(1) NOT NULL DEFAULT 1");
5
  }
6
  if (!column_exists('search_field', 'boost')) {
7
    $dbh->do("ALTER TABLE `search_field` ADD COLUMN `boost` decimal(5,2) DEFAULT NULL");
8
  }
9
  if (!column_exists('search_field', 'staff_client')) {
10
    $dbh->do("ALTER TABLE `search_field` ADD COLUMN `staff_client` tinyint(1) NOT NULL DEFAULT 1");
11
  }
12
  if (!column_exists('search_field', 'opac')) {
13
    $dbh->do("ALTER TABLE `search_field` ADD COLUMN `opac` tinyint(1) NOT NULL DEFAULT 1");
14
  }
15
  SetVersion($DBversion)
16
}
17
print "Upgrade to $DBversion done (Bug 20589 - Add field boosting and use elastic query fields parameter instead of depricated _all)\n";
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/searchengine/elasticsearch/mappings.tt (-1 / +59 lines)
Lines 126-131 a.add, a.delete { Link Here
126
                    <th>Name</th>
126
                    <th>Name</th>
127
                    <th>Label</th>
127
                    <th>Label</th>
128
                    <th>Type</th>
128
                    <th>Type</th>
129
                    <th colspan="2">Searchable</th>
130
                    <th>Boost</th>
131
                  </tr>
132
                  <tr>
133
                    <th colspan=3>&nbsp;</th>
134
                    <th>Staff client</th>
135
                    <th>OPAC</th>
136
                    <th>&nbsp;</th>
129
                  </tr>
137
                  </tr>
130
                </thead>
138
                </thead>
131
                <tbody>
139
                <tbody>
Lines 134-140 a.add, a.delete { Link Here
134
                      <td>
142
                      <td>
135
                        <input type="text" name="search_field_name" value="[% search_field.name %]" />
143
                        <input type="text" name="search_field_name" value="[% search_field.name %]" />
136
                      </td>
144
                      </td>
137
                      <td><input type="text" name="search_field_label" value="[% search_field.label %]" />
145
                      <td>
146
                        <input type="text" name="search_field_label" value="[% search_field.label %]" />
147
                      </td>
138
                      <td>
148
                      <td>
139
                        <select name="search_field_type">
149
                        <select name="search_field_type">
140
                          <option value=""></option>
150
                          <option value=""></option>
Lines 165-170 a.add, a.delete { Link Here
165
                          [% END %]
175
                          [% END %]
166
                        </select>
176
                        </select>
167
                      </td>
177
                      </td>
178
                      <td>
179
                        <select name="search_field_staff_client">
180
                          [% IF search_field.staff_client %]
181
                            <option value="1" selected="selected">Yes</option>
182
                            <option value="0">No</option>
183
                          [% ELSE %]
184
                            <option value="1">Yes</option>
185
                            <option value="0" selected="selected">No</option>
186
                          [% END %]
187
                        </select>
188
                      </td>
189
                      <td>
190
                        <select name="search_field_opac">
191
                          [% IF search_field.opac %]
192
                            <option value="1" selected="selected">Yes</option>
193
                            <option value="0">No</option>
194
                          [% ELSE %]
195
                            <option value="1">Yes</option>
196
                            <option value="0" selected="selected">No</option>
197
                          [% END %]
198
                        </select>
199
                      </td>
200
                      <td>
201
                        <input type="text" size="3" name="search_field_boost" value="[% search_field.boost %]" />
202
                      </td>
168
                    </tr>
203
                    </tr>
169
                  [% END %]
204
                  [% END %]
170
                </tbody>
205
                </tbody>
Lines 179-184 a.add, a.delete { Link Here
179
                          <th>Sortable</th>
214
                          <th>Sortable</th>
180
                          <th>Facetable</th>
215
                          <th>Facetable</th>
181
                          <th>Suggestible</th>
216
                          <th>Suggestible</th>
217
                          <th>Searchable</th>
182
                          <th>Mapping</th>
218
                          <th>Mapping</th>
183
                          <th></th>
219
                          <th></th>
184
                        </tr>
220
                        </tr>
Lines 233-238 a.add, a.delete { Link Here
233
                              </select>
269
                              </select>
234
                            </td>
270
                            </td>
235
                            <td>
271
                            <td>
272
                              <select name="mapping_search">
273
                                [% IF mapping.search %]
274
                                  <option value="0">No</option>
275
                                  <option value="1" selected="selected">Yes</option>
276
                                [% ELSE %]
277
                                  <option value="0" selected="selected">No</option>
278
                                  <option value="1">Yes</option>
279
                                [% END %]
280
                              </select>
281
                            </td>
282
                            <td>
236
                                <input name="mapping_marc_field" type="text" value="[% mapping.marc_field %]" />
283
                                <input name="mapping_marc_field" type="text" value="[% mapping.marc_field %]" />
237
                            </td>
284
                            </td>
238
                            <td><a class="btn btn-default btn-xs delete" style="cursor: pointer;"><i class="fa fa-trash"></i> Delete</a></td>
285
                            <td><a class="btn btn-default btn-xs delete" style="cursor: pointer;"><i class="fa fa-trash"></i> Delete</a></td>
Lines 278-283 a.add, a.delete { Link Here
278
                              [% END %]
325
                              [% END %]
279
                            </select>
326
                            </select>
280
                          </td>
327
                          </td>
328
                          <td>
329
                            <select data-id="mapping_search">
330
                              [% IF mapping.search %]
331
                                <option value="0">No</option>
332
                                <option value="1" selected="selected">Yes</option>
333
                              [% ELSE %]
334
                                <option value="0" selected="selected">No</option>
335
                                <option value="1">Yes</option>
336
                              [% END %]
337
                            </select>
338
                          </td>
281
                          <td><input data-id="mapping_marc_field" type="text" /></td>
339
                          <td><input data-id="mapping_marc_field" type="text" /></td>
282
                          <td><a class="btn btn-default btn-xs add"><i class="fa fa-plus"></i> Add</a></td>
340
                          <td><a class="btn btn-default btn-xs add"><i class="fa fa-plus"></i> Add</a></td>
283
                        </tr>
341
                        </tr>
(-)a/opac/opac-search.pl (-2 / +1 lines)
Lines 555-561 if (C4::Context->preference('OpacSuppression')) { Link Here
555
    \@sort_by,
555
    \@sort_by,
556
    0,
556
    0,
557
    $lang,
557
    $lang,
558
    { expanded_facet => $expanded_facet, suppress => $suppress }
558
    { expanded_facet => $expanded_facet, suppress => $suppress, is_opac => 1 }
559
    );
559
    );
560
560
561
sub _input_cgi_parse {
561
sub _input_cgi_parse {
562
- 

Return to bug 20589