From bugzilla-daemon@bugs.koha-community.org Mon Feb 22 15:31:23 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] New: Encoding problem in link to OverDrive results Date: Mon, 22 Feb 2021 14:31:22 +0000 Message-ID: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1371580860390886733==" --===============1371580860390886733== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 Bug ID: 27748 Summary: Encoding problem in link to OverDrive results Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: OPAC Assignee: oleonard@myacpl.org Reporter: magnus@libriotech.no QA Contact: testopia@bugs.koha-community.org If you have OverDrive enabled and do a search in the OPAC, you get a link that says e.g. "Found 1 results in the library's OverDrive collection". But if you search contains non-ASCII characters, the search goes wrong.=20 Example:=20 Do a search for "g=C3=B6teborg": https://kohaopac.alingsas.se/cgi-bin/koha/opac-search.pl?q=3Dg%C3%B6teborg (The URL shows the query as q=3Dg=C3=B6teborg, but when I paste it here it is= turned into q=3Dg%C3%B6teborg.) The link says: Found 4 results in the library's OverDrive collection The actual link is: https://kohaopac.alingsas.se/cgi-bin/koha/opac-overdrive-search.pl?q=3Dg%F6te= borg If you foloow that link you get to this page: https://kohaopac.alingsas.se/cgi-bin/koha/opac-overdrive-search.pl?q=3Dg%F6te= borg which says "OverDrive search for 'g=EF=BF=BDteborg'" and shows 53 results, be= cause the query was cut off at the "=C3=B6", and a search was actually made for just the letter "g" (whatever comes before the first non-ascii-char). If I change the encoding in the link from: https://kohaopac.alingsas.se/cgi-bin/koha/opac-overdrive-search.pl?q=3Dg%F6te= borg to: https://kohaopac.alingsas.se/cgi-bin/koha/opac-overdrive-search.pl?q=3Dg%C3%B= 6teborg or=20 https://kohaopac.alingsas.se/cgi-bin/koha/opac-overdrive-search.pl?q=3Dg=C3= =B6teborg the page shows 4 results, as promised. --=20 You are receiving this mail because: You are watching all bug changes. --===============1371580860390886733==-- From bugzilla-daemon@bugs.koha-community.org Mon Feb 22 15:56:54 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Mon, 22 Feb 2021 14:56:53 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============8929620055242111644==" --===============8929620055242111644== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 --- Comment #1 from Magnus Enger --- Tested on 20.05.05 and 20.05.08.=20 I *think* the problem is the "escape" on line 934 of koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-results.tt: $overdrive_results.html( '' + MSG_RESULTS_FOUND_IN_COLLECTION.format(data.totalItems, 'OverDrive') + '' ); --=20 You are receiving this mail because: You are watching all bug changes. --===============8929620055242111644==-- From bugzilla-daemon@bugs.koha-community.org Mon Feb 22 15:59:54 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Mon, 22 Feb 2021 14:59:54 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2273194504308187111==" --===============2273194504308187111== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 --- Comment #2 from Magnus Enger --- "Bug 26578: Clean overdrive searches before passing to OD" touches the releva= nt parts of the code, but as far as I can see, it does not fix the encoding problem. --=20 You are receiving this mail because: You are watching all bug changes. --===============2273194504308187111==-- From bugzilla-daemon@bugs.koha-community.org Tue Feb 23 12:42:55 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 23 Feb 2021 11:42:51 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1607658942326629966==" --===============1607658942326629966== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Magnus Enger changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. --===============1607658942326629966==-- From bugzilla-daemon@bugs.koha-community.org Tue Feb 23 12:43:05 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 23 Feb 2021 11:42:54 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7734252198075316120==" --===============7734252198075316120== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 --- Comment #3 from Magnus Enger --- Created attachment 117194 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117194&action=edit Bug 27748 - Encoding problem in link to OverDrive results If you have OverDrive enabled and do a search for e.g. "Göteborg", you will get a link above the regular Koha search results that says e.g. "Found 1 results in the library's OverDrive collection". If you follow this link, a search will be made in OverDrive, but only for "G", because the search term is cut off at the first non-ASCII char. To test: - Make sure OverDrive is enabled - Do a search that contains a non-ASCII char, like "Göteborg" - Click on the link "Found x results in the library's OverDrive collection" - Verify the query string is cut off at the first non-ASCII char, and that the number of hits is different from the number given in the link on the previous page - Apply the patch, restart all the things - Repeat the search, and verify that the numbers match and that the query string is intact after clicking on the OverDrive link Lookin at the code, the same problem seems to be shared between OverDrive and RecordedBooks. I have applied the same fix to both, but I do not have access to RecordedBooks, so I have not been able to verify that the problem exists there and is fixed by my patch. -- You are receiving this mail because: You are watching all bug changes. --===============7734252198075316120==-- From bugzilla-daemon@bugs.koha-community.org Tue Feb 23 12:53:48 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 23 Feb 2021 11:53:48 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3244946969933278109==" --===============3244946969933278109== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 --- Comment #4 from Magnus Enger --- Technical note: It looks like the use of escape() has been deprecated in favour of encodeURI and encodeURIComponent: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Obje= cts/escape I have replaced two occurrences of escape with encodeURIComponent. --=20 You are receiving this mail because: You are watching all bug changes. --===============3244946969933278109==-- From bugzilla-daemon@bugs.koha-community.org Fri Mar 19 05:06:30 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Fri, 19 Mar 2021 04:06:29 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5372876075801337107==" --===============5372876075801337107== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Phil Ringnalda changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. --===============5372876075801337107==-- From bugzilla-daemon@bugs.koha-community.org Fri Mar 19 05:06:40 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Fri, 19 Mar 2021 04:06:32 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7765006337041577807==" --===============7765006337041577807== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Phil Ringnalda changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #117194|0 |1 is obsolete| | --- Comment #5 from Phil Ringnalda --- Created attachment 118481 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118481&action=edit Bug 27748 - Encoding problem in link to OverDrive results If you have OverDrive enabled and do a search for e.g. "Göteborg", you will get a link above the regular Koha search results that says e.g. "Found 1 results in the library's OverDrive collection". If you follow this link, a search will be made in OverDrive, but only for "G", because the search term is cut off at the first non-ASCII char. To test: - Make sure OverDrive is enabled - Do a search that contains a non-ASCII char, like "Göteborg" - Click on the link "Found x results in the library's OverDrive collection" - Verify the query string is cut off at the first non-ASCII char, and that the number of hits is different from the number given in the link on the previous page - Apply the patch, restart all the things - Repeat the search, and verify that the numbers match and that the query string is intact after clicking on the OverDrive link Lookin at the code, the same problem seems to be shared between OverDrive and RecordedBooks. I have applied the same fix to both, but I do not have access to RecordedBooks, so I have not been able to verify that the problem exists there and is fixed by my patch. Signed-off-by: Phil Ringnalda -- You are receiving this mail because: You are watching all bug changes. --===============7765006337041577807==-- From bugzilla-daemon@bugs.koha-community.org Fri Mar 19 05:06:56 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Fri, 19 Mar 2021 04:06:56 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5916350467910322628==" --===============5916350467910322628== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Phil Ringnalda changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |phil@chetcolibrary.org -- You are receiving this mail because: You are watching all bug changes. --===============5916350467910322628==-- From bugzilla-daemon@bugs.koha-community.org Sun Apr 4 00:14:17 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Sat, 03 Apr 2021 22:14:16 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============4243790714115473543==" --===============4243790714115473543== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Katrin Fischer changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. --===============4243790714115473543==-- From bugzilla-daemon@bugs.koha-community.org Sun Apr 4 00:14:27 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Sat, 03 Apr 2021 22:14:19 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1585914473891867504==" --===============1585914473891867504== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 --- Comment #6 from Katrin Fischer --- Created attachment 119166 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119166&action=edit Bug 27748: Encoding problem in link to OverDrive results If you have OverDrive enabled and do a search for e.g. "Göteborg", you will get a link above the regular Koha search results that says e.g. "Found 1 results in the library's OverDrive collection". If you follow this link, a search will be made in OverDrive, but only for "G", because the search term is cut off at the first non-ASCII char. To test: - Make sure OverDrive is enabled - Do a search that contains a non-ASCII char, like "Göteborg" - Click on the link "Found x results in the library's OverDrive collection" - Verify the query string is cut off at the first non-ASCII char, and that the number of hits is different from the number given in the link on the previous page - Apply the patch, restart all the things - Repeat the search, and verify that the numbers match and that the query string is intact after clicking on the OverDrive link Lookin at the code, the same problem seems to be shared between OverDrive and RecordedBooks. I have applied the same fix to both, but I do not have access to RecordedBooks, so I have not been able to verify that the problem exists there and is fixed by my patch. Signed-off-by: Phil Ringnalda Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. --===============1585914473891867504==-- From bugzilla-daemon@bugs.koha-community.org Sun Apr 4 00:14:37 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Sat, 03 Apr 2021 22:14:30 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2209785536324579808==" --===============2209785536324579808== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Katrin Fischer changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|oleonard@myacpl.org |magnus@libriotech.no -- You are receiving this mail because: You are watching all bug changes. --===============2209785536324579808==-- From bugzilla-daemon@bugs.koha-community.org Sun Apr 4 00:15:15 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Sat, 03 Apr 2021 22:15:14 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============4172698951387984579==" --===============4172698951387984579== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 --- Comment #7 from Katrin Fischer --- Trusting that Magnus and Phil have tested this with access to Overdrive. Chan= ge makes sense and I will move this to PQA with a code review only. --=20 You are receiving this mail because: You are watching all bug changes. --===============4172698951387984579==-- From bugzilla-daemon@bugs.koha-community.org Tue Apr 6 02:11:03 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 06 Apr 2021 00:11:02 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1031298159175073963==" --===============1031298159175073963== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 --- Comment #8 from Phil Ringnalda --- Correct: I tested with Overdrive, and Magnus' link from comment 0 to show the problem no longer shows the problem, because he is running with his patch :) -- You are receiving this mail because: You are watching all bug changes. --===============1031298159175073963==-- From bugzilla-daemon@bugs.koha-community.org Tue Apr 6 12:18:00 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 06 Apr 2021 10:17:59 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7253957075852565127==" --===============7253957075852565127== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Jonathan Druart changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org Attachment #118481|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes. --===============7253957075852565127==-- From bugzilla-daemon@bugs.koha-community.org Tue Apr 6 15:58:34 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 06 Apr 2021 13:57:47 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5939718649221052803==" --===============5939718649221052803== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Jonathan Druart changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |21.05.00 released in| | -- You are receiving this mail because: You are watching all bug changes. --===============5939718649221052803==-- From bugzilla-daemon@bugs.koha-community.org Tue Apr 6 15:58:44 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Tue, 06 Apr 2021 13:57:49 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0074242258479900092==" --===============0074242258479900092== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3D27748 --- Comment #9 from Jonathan Druart = --- Pushed to master for 21.05, thanks to everybody involved! --=20 You are receiving this mail because: You are watching all bug changes. --===============0074242258479900092==-- From bugzilla-daemon@bugs.koha-community.org Thu Apr 15 13:05:00 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Thu, 15 Apr 2021 11:04:59 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============8572071988996771665==" --===============8572071988996771665== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Fridolin Somers changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|21.05.00 |21.05.00,20.11.05 released in| | CC| |fridolin.somers@biblibre.co | |m --- Comment #10 from Fridolin Somers --- Pushed to 20.11.x for 20.11.05 -- You are receiving this mail because: You are watching all bug changes. --===============8572071988996771665==-- From bugzilla-daemon@bugs.koha-community.org Sun Apr 25 15:48:56 2021 From: bugzilla-daemon@bugs.koha-community.org To: koha-bugs@lists.koha-community.org Subject: [Koha-bugs] [Bug 27748] Encoding problem in link to OverDrive results Date: Sun, 25 Apr 2021 13:48:55 +0000 Message-ID: In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7248224032032958232==" --===============7248224032032958232== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27748 Andrew Fuerste-Henry changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com --- Comment #11 from Andrew Fuerste-Henry --- Doesn't apply cleanly to 20.05, please rebase for backport. I'm not entirely confident in my ability to clear the merge errors correctly and don't have credentials with which to check my work with a real search. Thanks! -- You are receiving this mail because: You are watching all bug changes. --===============7248224032032958232==--