Attachment 103210 Details for Bug 20501 – Bug 20501: Don't cut strings when unhighlighting in intranet search results

[patch] Bug 20501: Don't cut strings when unhighlighting in intranet search results

Bug-20501-Dont-cut-strings-when-unhighlighting-in-.patch (text/plain), 1.75 KB, created by Phil Ringnalda on 2020-04-19 02:53:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Phil Ringnalda

Created: 2020-04-19 02:53:44 UTC

Size: 1.75 KB

patch

obsolete

>From 2c4e59beeef2145aa6d04c4dfb0f3cb7c1cca7df Mon Sep 17 00:00:00 2001
>From: Phil Ringnalda <phil@chetcolibrary.org>
>Date: Sun, 19 Apr 2020 02:37:49 +0000
>Subject: [PATCH] Bug 20501: Don't cut strings when unhighlighting in intranet
> search results
>
>When you click Unhighlight on the search results of a search in the staff client
>when the search terms contain the same word twice, like "ma ma", the highlighted
>words disappear, and don't reappear if you click Highlight.
>
>This patch makes the same change to the intranet copy of jquery.highlight-3.js that
>bug 5844 made to the OPAC copy.
>
>Test plan:
>1) Without this patch, search in the intranet for ma ma and verify you have at least
>   two matching records.
>2) Notice that the Ma in Material type: Book is highlighted.
>3) Click Unhighlight, and notice that it has become terial type: Book
>4) Apply this patch, repeat the search with a shift+reload to reload the .js
>5) Click Unhighlight, and verify that Material type: Book remains Material
>---
> koha-tmpl/intranet-tmpl/lib/jquery/plugins/jquery.highlight-3.js | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/jquery.highlight-3.js b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/jquery.highlight-3.js
>index 69bb52d3b1..8eabca4bfd 100644
>--- a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/jquery.highlight-3.js
>+++ b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/jquery.highlight-3.js
>@@ -44,10 +44,6 @@ jQuery.fn.highlight = function(pat) {
> 
> jQuery.fn.removeHighlight = function() {
>  return this.find("span.term").each(function() {
>-  this.parentNode.firstChild.nodeName;
>-  with (this.parentNode) {
>-   replaceChild(this.firstChild, this);
>-   normalize();
>-  }
>- }).end();
>+  $(this).contents().unwrap();
>+ });
> };
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 20501: 103210 | 103319 | 103387