Bugzilla – Attachment 67564 Details for
Bug 19122
IncludeSeeFromInSearches is broken
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 19122: (bug 18098 follow-up) Fix IncludeSeeFromInSearches behaviour
Bug-19122-bug-18098-follow-up-Fix-IncludeSeeFromIn.patch (text/plain), 2.31 KB, created by
Tomás Cohen Arazi (tcohen)
on 2017-10-03 17:22:23 UTC
(
hide
)
Description:
Bug 19122: (bug 18098 follow-up) Fix IncludeSeeFromInSearches behaviour
Filename:
MIME Type:
Creator:
Tomás Cohen Arazi (tcohen)
Created:
2017-10-03 17:22:23 UTC
Size:
2.31 KB
patch
obsolete
>From 48d7c9276b30085312f27977ded3780bb4d34f9f Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 2 Oct 2017 14:11:52 -0300 >Subject: [PATCH] Bug 19122: (bug 18098 follow-up) Fix IncludeSeeFromInSearches > behaviour > >The IncludeSeeFromInSearches system preference is designed so that 'See from' headings from the authorities are included when you search in the catalog. >That means that you could find an author not only by the name printed on the book, but for example also by their pseudonym or a different spelling of their name. > >It was added by bug 7417. > >This regression has been introduced by > commit 5ef1b6710e7520b844e145e248da0deeee707fde > Bug 18098: Add an index with the count of not onloan items > >- } elsif ($record_type eq 'biblio' && C4::Context->preference('IncludeSeeFromInSearches')) { >- my $normalizer = Koha::RecordProcessor->new( { filters => 'EmbedSeeFromHeadings' } ); >[...] >+ push @filters, 'IncludeSeeFromInSearches' >+ if C4::Context->preference('IncludeSeeFromInSearches'); > >Test plan: >- Activate IncludeSeeFromInSearches >- Catalog an authority for a person > - main heading in 100 > - see from headings in 400 >- Catalog a bibliographic record and link it to the authority >- Make sure the record is indexed >- Verify that the record can be found searching for the main heading >- Verify that the record can be found searching for the see from headings > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> >Yet another reason to get rid of all this functions from this script. >--- > misc/migration_tools/rebuild_zebra.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/misc/migration_tools/rebuild_zebra.pl b/misc/migration_tools/rebuild_zebra.pl >index c3f41df..c39ce63 100755 >--- a/misc/migration_tools/rebuild_zebra.pl >+++ b/misc/migration_tools/rebuild_zebra.pl >@@ -656,7 +656,7 @@ sub get_corrected_marc_record { > > my @filters; > push @filters, 'EmbedItemsAvailability'; >- push @filters, 'IncludeSeeFromInSearches' >+ push @filters, 'EmbedSeeFromHeadings' > if C4::Context->preference('IncludeSeeFromInSearches'); > > my $normalizer = Koha::RecordProcessor->new( { filters => \@filters } ); >-- >2.7.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 19122
:
67526
|
67535
|
67536
| 67564