https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28885

Joonas Kylmälä <joonas.kylm...@iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #124009|0                           |1
        is obsolete|                            |

--- Comment #3 from Joonas Kylmälä <joonas.kylm...@iki.fi> ---
Created attachment 124252
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124252&action=edit
Bug 28885: Skip invalid biblios for OpacBrowseResults

If a record is deleted from Koha, but is for some reason not deleted from the
search indexes, OpacBrowseResults can cause an ISE if the deleted record is in
the search results for any given item. OpacBrowseResults loops through the
search results, and checks if there is a biblionumber, but does *not* check to
see if a result was pulled from the database for that biblionumber. It simply
assumes the result must exist.

We should be checking to ensure the biblionumber was valid before operating on
the biblio object.

Test Plan:
1) Use zebra for searching
2) Disable koha-indexer
3) Enable OpacBrowseResults
4) Perform a search
5) Delete an item in the search results
6) View on of the remaining items in the search results
7) Note the error
8) Apply this patch
9) Restart plack
10) Reload the page
11) The error should be gone!

Signed-off-by: David Nind <da...@davidnind.com>

Signed-off-by: Joonas Kylmälä <joonas.kylm...@iki.fi>

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to