Bugzilla – Attachment 110620 Details for
Bug 26507
New items not indexed
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 26507: Elasticsearch - Index records after storing new item
Bug-26507-Elasticsearch---Index-records-after-stor.patch (text/plain), 2.25 KB, created by
Katrin Fischer
on 2020-09-23 19:44:51 UTC
(
hide
)
Description:
Bug 26507: Elasticsearch - Index records after storing new item
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2020-09-23 19:44:51 UTC
Size:
2.25 KB
patch
obsolete
>From 91caf72013641bc446fd5cf45f6faf94c66a054c Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Tue, 22 Sep 2020 14:14:55 +0000 >Subject: [PATCH] Bug 26507: Elasticsearch - Index records after storing new > item > >Currently Item->store indexes the record before the DB update - that is wrong > >To test: > 1 - Find/create a bib with no items > 2 - add an item with barcode "abc123" > 3 - do a general keyword search for "abc123," see your bib is not in the results > 4 - perform a search that includes your bib in the results, confirm it shows as having no items > 5 - click through to bib details, confirm it shows your item here > 6 - edit and save your item > 7 - confirm barcode is now searchable > 8 - apply patches > 9 - Add a new item "cde456" >10 - Confirm it returns in searches >11 - Edit 'cde456' and change barcode to 'fgh789' >12 - Confirm the new abrcode is searchable > >Signed-off-by: Lisette Scheer <lisettes@latahlibrary.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > Koha/Item.pm | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > >diff --git a/Koha/Item.pm b/Koha/Item.pm >index 8b7a12653f..add258491e 100644 >--- a/Koha/Item.pm >+++ b/Koha/Item.pm >@@ -113,9 +113,6 @@ sub store { > $self->cn_sort($cn_sort); > } > >- C4::Biblio::ModZebra( $self->biblionumber, "specialUpdate", "biblioserver" ) >- unless $params->{skip_modzebra_update}; >- > logaction( "CATALOGUING", "ADD", $self->itemnumber, "item" ) > if $log_action && C4::Context->preference("CataloguingLog"); > >@@ -189,9 +186,6 @@ sub store { > $self->paidfor(''); > } > >- C4::Biblio::ModZebra( $self->biblionumber, "specialUpdate", "biblioserver" ) >- unless $params->{skip_modzebra_update}; >- > logaction( "CATALOGUING", "MODIFY", $self->itemnumber, "item " . Dumper($self->unblessed) ) > if $log_action && C4::Context->preference("CataloguingLog"); > } >@@ -201,6 +195,8 @@ sub store { > } > > my $result = $self->SUPER::store; >+ C4::Biblio::ModZebra( $self->biblionumber, "specialUpdate", "biblioserver" ) >+ unless $params->{skip_modzebra_update}; > $self->get_from_storage->_after_item_action_hooks({ action => $plugin_action }); > > return $result; >-- >2.11.0
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 26507
:
110558
|
110567
| 110620 |
110879