[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Jacek Ablewicz changed: What|Removed |Added CC||a...@biblos.pk.edu.pl See Also||http://bugs.koha-community. ||org/bugzilla3/show_bug.cgi? ||id=12954 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Status|Pushed to Stable|RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Heather Braum changed: What|Removed |Added CC|hbr...@nekls.org| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Heather Braum changed: What|Removed |Added CC||hbr...@nekls.org -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||fridolyn.som...@biblibre.co ||m --- Comment #29 from Fridolin SOMERS --- Patches pushed to 3.14.x, will be in 3.14.4. A nice work on this really ugly bug. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 charlene.cri...@gmail.com changed: What|Removed |Added CC||charlene.cri...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #28 from Galen Charlton --- Pushed to master, along with a follow-up that fixes the doubled (and tripled, etc.) search history issue that Katrin found, and a follow-up that removes that last remaining reference to the KohaOpacRecentSearches cookie. Thanks, Julian! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Kyle M Hall changed: What|Removed |Added Attachment #24138|0 |1 is obsolete|| --- Comment #26 from Kyle M Hall --- Created attachment 24157 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24157&action=edit Bug 10952: comments fixes and unit tests - Remove unit tests for ParseSearchHistoryCookie, which doesn't exist anymore - Add unit tests for ParseSearchHistorySession and SetSearchHistorySession - Remove/Modify comments about search history cookie Signed-off-by: Chris Cormack Tests fixed and moved, and comments tidied up Signed-off-by: Charlene Criton Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Kyle M Hall changed: What|Removed |Added Attachment #24139|0 |1 is obsolete|| --- Comment #27 from Kyle M Hall --- Created attachment 24158 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24158&action=edit Bug 10952: Always flush session after deletion This is recommended in CGI::Session documentation. Signed-off-by: Charlene Criton Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Kyle M Hall changed: What|Removed |Added Attachment #24137|0 |1 is obsolete|| --- Comment #25 from Kyle M Hall --- Created attachment 24156 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24156&action=edit Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. Signed-off-by: Chris Cormack Restoring original sign offs and comments below Signed-off-by: Bernardo Gonzalez Kriegel Work as described. No koha-qa errors Well, search history saving is similar before and after patch. i.e. anonmymous search is saved when user logs in, but cookie KohaOpacRecentSearches is empty. Shows current an previous session searches Signed-off-by: Katrin Fischer All tests and QA script pass, works as described. Signed-off-by: Charlene Criton Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Passed QA CC||k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #23 from sandbo...@biblibre.com --- Created attachment 24138 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24138&action=edit Bug 10952: comments fixes and unit tests - Remove unit tests for ParseSearchHistoryCookie, which doesn't exist anymore - Add unit tests for ParseSearchHistorySession and SetSearchHistorySession - Remove/Modify comments about search history cookie Signed-off-by: Chris Cormack Tests fixed and moved, and comments tidied up Signed-off-by: Charlene Criton -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #22 from sandbo...@biblibre.com --- Created attachment 24137 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24137&action=edit Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. Signed-off-by: Chris Cormack Restoring original sign offs and comments below Signed-off-by: Bernardo Gonzalez Kriegel Work as described. No koha-qa errors Well, search history saving is similar before and after patch. i.e. anonmymous search is saved when user logs in, but cookie KohaOpacRecentSearches is empty. Shows current an previous session searches Signed-off-by: Katrin Fischer All tests and QA script pass, works as described. Signed-off-by: Charlene Criton -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 sandbo...@biblibre.com changed: What|Removed |Added Attachment #23927|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 sandbo...@biblibre.com changed: What|Removed |Added Attachment #23755|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 sandbo...@biblibre.com changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||sandbo...@biblibre.com --- Comment #21 from sandbo...@biblibre.com --- Patch tested with a sandbox, by Charlene Criton -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 sandbo...@biblibre.com changed: What|Removed |Added Attachment #23756|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #24 from sandbo...@biblibre.com --- Created attachment 24139 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=24139&action=edit Bug 10952: Always flush session after deletion This is recommended in CGI::Session documentation. Signed-off-by: Charlene Criton -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 jmbroust changed: What|Removed |Added CC||jean-manuel.broust@univ-lyo ||n2.fr --- Comment #20 from jmbroust --- Tested on sandbox 16 biblibre This patch works well ! Tests with Firefox and Internet Explorer. We wonder if it's not a problem for public computers without identification. For example, in our library, lot of research are performed with computers only catalog access. If a patron log after several anonymous searches, he would have the all precedent anonymous searches in his search history. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #19 from Chris Cormack --- I couldn't reproduce this either, so would like someone who can to sign off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #18 from Julian Maurice --- Created attachment 23927 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23927&action=edit Bug 10952: Always flush session after deletion This is recommended in CGI::Session documentation. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #17 from Julian Maurice --- I was not able to reproduce the problem, but I suspect that the session is not properly deleted when it should. The following patch ensure that by flushing session after each call to $session->delete(). I hope this will fix the problem. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #16 from Katrin Fischer --- Tested again in Chromium and Bootstrap theme: - Logged out. - Closed browser. - Reopened browser. - Did 2 simple searches. - Logged in using the link on top. - Doubled up entries in search history. - Logged out. - Did another serach. - Logged back in. - Doubled up entries in search history. I can't pass this :( -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #15 from Katrin Fischer --- Ok, giving this another go... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #14 from Katrin Fischer --- Hm, and now I can't reproduce it a second time... could someone else give this a try please? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #13 from Katrin Fischer --- Created attachment 23783 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23783&action=edit Screenshot showing doubled searches in search historx I am sorry, but I think I found a problem: - logged out of Koha - did 2 searches (Any: der and Title: der und) - logged into Koha - 4 searches (each doubled up) are added to my user's search history -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Chris Cormack changed: What|Removed |Added Attachment #23619|0 |1 is obsolete|| --- Comment #12 from Chris Cormack --- Created attachment 23756 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23756&action=edit Bug 10952: comments fixes and unit tests - Remove unit tests for ParseSearchHistoryCookie, which doesn't exist anymore - Add unit tests for ParseSearchHistorySession and SetSearchHistorySession - Remove/Modify comments about search history cookie Signed-off-by: Chris Cormack Tests fixed and moved, and comments tidied up -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Chris Cormack changed: What|Removed |Added Attachment #23618|0 |1 is obsolete|| --- Comment #11 from Chris Cormack --- Created attachment 23755 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23755&action=edit Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. Signed-off-by: Chris Cormack Restoring original sign offs and comments below Signed-off-by: Bernardo Gonzalez Kriegel Work as described. No koha-qa errors Well, search history saving is similar before and after patch. i.e. anonmymous search is saved when user logs in, but cookie KohaOpacRecentSearches is empty. Shows current an previous session searches Signed-off-by: Katrin Fischer All tests and QA script pass, works as described. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Chris Cormack changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Chris Cormack changed: What|Removed |Added CC||ch...@bigballofwax.co.nz --- Comment #10 from Chris Cormack --- (In reply to Julian Maurice from comment #9) > 1st patch rebased on master. When you did the rebase, you lost the sign offs on the commit, I have readded them and the comments. Please try to be careful to apply the signed off patch then rebase it. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #9 from Julian Maurice --- 1st patch rebased on master. 2nd patch needs signoff (or maybe only QA since it modifies only comments and unit tests) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Attachment #22133|0 |1 is obsolete|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Attachment #23618|Bug 10952: Store anonymous |[PASSED QA] Bug 10952: description|search history in session |Store anonymous search ||history in session -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #7 from Julian Maurice --- Created attachment 23618 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23618&action=edit Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #8 from Julian Maurice --- Created attachment 23619 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23619&action=edit Bug 10952: comments fixes and unit tests - Remove unit tests for ParseSearchHistoryCookie, which doesn't exist anymore - Add unit tests for ParseSearchHistorySession and SetSearchHistorySession - Remove/Modify comments about search history cookie -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Galen Charlton changed: What|Removed |Added Status|Passed QA |Failed QA --- Comment #6 from Galen Charlton --- This patch needs more work. Most obviously, it breaks the tests in t/db_dependent/Auth_ParseSearchHistoryCookie.t. Of course, ParseSearchHistoryCookie is removed by the patch, but the tests should be rewritten so that they exercise session-based search history. Also, there are a number of comments referring to the search history cookie that should be removed or revised. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Attachment #21550|0 |1 is obsolete|| --- Comment #5 from Katrin Fischer --- Created attachment 22133 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22133&action=edit [PASSED QA] Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. Signed-off-by: Bernardo Gonzalez Kriegel Work as described. No koha-qa errors Well, search history saving is similar before and after patch. i.e. anonmymous search is saved when user logs in, but cookie KohaOpacRecentSearches is empty. Shows current an previous session searches Signed-off-by: Katrin Fischer All tests and QA script pass, works as described. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Katrin Fischer changed: What|Removed |Added Blocks||10857 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Bernardo Gonzalez Kriegel changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||bgkrie...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Bernardo Gonzalez Kriegel changed: What|Removed |Added Attachment #21482|0 |1 is obsolete|| --- Comment #4 from Bernardo Gonzalez Kriegel --- Created attachment 21550 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21550&action=edit [SIGNED-OFF] Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. Signed-off-by: Bernardo Gonzalez Kriegel Work as described. No koha-qa errors Well, search history saving is similar before and after patch. i.e. anonmymous search is saved when user logs in, but cookie KohaOpacRecentSearches is empty. Shows current an previous session searches -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Galen Charlton changed: What|Removed |Added CC||gmcha...@gmail.com --- Comment #3 from Galen Charlton --- Haven't looked at the patch in detail, but +1 to the notion of storing it in the session. I do wonder if there should be a limit on the number of searches kept though -- while there is no specific limit on the amount of data that can be stored in a session, consider the case of a public OPAC station that doesn't get reset often. In principle, it could end up with a session that carries hundreds of searches, which might have a notable impact on performance. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Severity|enhancement |major --- Comment #2 from Julian Maurice --- Increasing severity to major as its more a bugfix than an enhancement (cf bug 10807) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 Julian Maurice changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/
[Koha-bugs] [Bug 10952] Store anonymous search history in session
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10952 --- Comment #1 from Julian Maurice --- Created attachment 21482 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21482&action=edit Bug 10952: Store anonymous search history in session Storing search history into cookie can cause problems, due to the size limitation of 4KB. The solution here is to store search history into the CGI::Session object, so there is no size limitation (but anonymous search history still remember up to 15 requests max.) Test plan: - Go to OPAC in anonymous mode. - Check that the "Search history" link is *not* shown in the top right corner of the page - Make some searches on /cgi-bin/koha/opac-search.pl - The "Search history" link must appear. Click. - Your search history must be displayed. - Try to log in with invalid username/password - Go back to search history, it's still there - Now log in with valid username/password - Your anonymous search history must be saved into your own search history. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/