[Bug 17074] New: Fix links in result list of 'scan indexes' search and keep search term
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Bug ID: 17074 Summary: Fix links in result list of 'scan indexes' search and keep search term Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Searching Assignee: katrin.fischer@bsz-bw.de Reporter: katrin.fischer@bsz-bw.de QA Contact: testopia@bugs.koha-community.org The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with: - The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |normal -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 --- Comment #1 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 54123 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54123&action=edit Bug 17074: Fix links in result list of 'scan indexes' and keep search term visible The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with: - The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results. To test: - Go to the advanced search, select an index to search in - Enter a search term and check 'scan indexes' - Submit search - Check if the search term is visible in the input box - Check if the result links contain your selected index and give you correct results (count and the number of results should match) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Christopher Brannon <cbrannon@cdalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |cbrannon@cdalibrary.org Status|Needs Signoff |Failed QA --- Comment #2 from Christopher Brannon <cbrannon@cdalibrary.org> --- (In reply to Katrin Fischer from comment #1)
Created attachment 54123 [details] [review] Bug 17074: Fix links in result list of 'scan indexes' and keep search term visible
The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with:
- The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results.
To test: - Go to the advanced search, select an index to search in - Enter a search term and check 'scan indexes' - Submit search - Check if the search term is visible in the input box - Check if the result links contain your selected index and give you correct results (count and the number of results should match)
Katrin, I would like to sign off on this, but as far as I can tell, the patch only addresses the links. I can't confirm an issue with the counts, most likely because of the limited number of records I have at the moment, so for that, someone else should test at this time. Also, I see no change with the search term in the input box, unless I am looking at the wrong thing. To me, on this page, the input box should be the textbox labeled "Scan Index for:". If this is the case, the box is still blank, and therefore does not pass your test plan. Please clarify. Thank you. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am not sure what it could be, it works for me. Can please someone else take another look to get a better picture? The counts are correct, the problem is when you click on the link and it's missing the index to search in - so it does a keyword search. So say you did a search for Author, then the counts will show the counts in the author index, but the link would start a keyword search which is likely to give you a different number of results. Best to test this not using the 'keyword' option -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Christopher, I tried various searches - the "Scan Index for:" is always set for me with the patch applied. I tried different indexes. I would be interested to see your link to the page when it doesn't work. Also, could you check the patch applied nicely? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #5 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hm, just found a little problem - if you enter 2 search terms, you will see something like: all%20this - url encoded... I have to check on how to fix this. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54123|0 |1 is obsolete| | --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 54133 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54133&action=edit Bug 17028: Follow-up: fixing encoding issues with multiple search terms If there was more than one search term you could see that that it was url encoded. Also problems with search terms with umlauts and other diacritics. Patch should fix that. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54133|0 |1 is obsolete| | --- Comment #7 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 54134 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54134&action=edit Bug 17074: Fix links in result list of 'scan indexes' and keep search term visible The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with: - The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results. To test: - Go to the advanced search, select an index to search in - Enter a search term and check 'scan indexes' - Submit search - Check if the search term is visible in the input box - Check if the result links contain your selected index and give you correct results (count and the number of results should match) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 54135 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54135&action=edit Bug 17074: Follow-up: fixing encoding issues with multiple search terms If there was more than one search term you could see that that it was url encoded. Also problems with search terms with umlauts and other diacritics. Patch should fix that. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54134|0 |1 is obsolete| | --- Comment #9 from Marc Véron <veron@veron.ch> --- Created attachment 54138 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54138&action=edit Bug 17074: Fix links in result list of 'scan indexes' and keep search term visible The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with: - The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results. To test: - Go to the advanced search, select an index to search in - Enter a search term and check 'scan indexes' - Submit search - Check if the search term is visible in the input box - Check if the result links contain your selected index and give you correct results (count and the number of results should match) Tested both patches together, works as expected. Signed-off-by: Marc <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54135|0 |1 is obsolete| | --- Comment #10 from Marc Véron <veron@veron.ch> --- Created attachment 54139 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54139&action=edit Bug 17074: Follow-up: fixing encoding issues with multiple search terms If there was more than one search term you could see that that it was url encoded. Also problems with search terms with umlauts and other diacritics. Patch should fix that. https://bugs.koha-community.org/show_bug.cgi?id=17074 Signed-off-by: Marc <veron@veron.ch> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Marc Véron <veron@veron.ch> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |veron@veron.ch Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54138|0 |1 is obsolete| | --- Comment #11 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 54190 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54190&action=edit Bug 17074: Fix links in result list of 'scan indexes' and keep search term visible The 'scan indexes' search that can be reached from the advanced search has 2 problems to begin with: - The search term you searched for is not displayed in the input field. - The links in the result list are missing the index and because of that, are not giving the correct results. To test: - Go to the advanced search, select an index to search in - Enter a search term and check 'scan indexes' - Submit search - Check if the search term is visible in the input box - Check if the result links contain your selected index and give you correct results (count and the number of results should match) Tested both patches together, works as expected. Signed-off-by: Marc <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #54139|0 |1 is obsolete| | --- Comment #12 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 54191 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=54191&action=edit Bug 17074: Follow-up: fixing encoding issues with multiple search terms If there was more than one search term you could see that that it was url encoded. Also problems with search terms with umlauts and other diacritics. Patch should fix that. https://bugs.koha-community.org/show_bug.cgi?id=17074 Signed-off-by: Marc <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #13 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 16.11, thanks Katrin! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |frederic@tamil.fr --- Comment #14 from Frédéric Demians <frederic@tamil.fr> --- Pushed in 16.05. Will be in 16.05.03. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Christopher Brannon <cbrannon@cdalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|cbrannon@cdalibrary.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17074 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com --- Comment #15 from Julian Maurice <julian.maurice@biblibre.com> --- Pushed to 3.22.x, will be in 3.22.10 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org