[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 Marcel de Rooy changed: What|Removed |Added CC||[email protected] QA Contact|[email protected] |[email protected] |y.org | Status|Signed Off |Failed QA --- Comment #4 from Marcel de Rooy --- Hi Julian, Two questions: [1] This is a one-liner and I understand that it may be hard to add a test for this tiny change? Could you confirm? [2] I have the idea that we could test before calling get_session if we had searches before. Otherwise it seems unneeded? Changing status for need of feedback or adjustments. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 David Nind changed: What|Removed |Added Text to go in the||This fixes the search release notes||history recorded in the ||OPAC. The search ||history--before logging in ||to the OPAC--was duplicated ||every time after you log in ||AND reloaded the search ||history page (Your account ||> Search history). CC||[email protected] --- Comment #3 from David Nind --- Testing notes (using KTD): 1. The system preferences are already enabled in KTD. 2. Replicate the problem using the steps in the bug description. 3. Apply the patch, reset_all, and run through the steps in the patch to show that it is fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 David Nind changed: What|Removed |Added Attachment #182724|0 |1 is obsolete|| --- Comment #2 from David Nind --- Created attachment 182738 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182738&action=edit Bug 39967: Reload session after modification Otherwise the existing $session still holds outdated data that is written on the next flush, which overwrites what has been set by C4::Search::History::set_to_session Test plan: 1. Enable LoadSearchHistoryToTheFirstLoggedUser and EnableOpacSearchHistory 2. Go to OPAC. If logged in, log out. 3. Go to search history and remove all searches if any. 4. Perform a search, make sure it appears in your search history 5. Log in and go to your search history. See that your unauthenticated search history has been added to your account's search history 6. Reload the page multiple times. See that the unauthenticated search history is *NOT* re-added after each page reload Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 David Nind 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 [email protected] 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/
[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 Julian Maurice changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
[Koha-bugs] [Bug 39967] When LoadSearchHistoryToTheFirstLoggedUser is enabled, search history is added at every page load
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39967 --- Comment #1 from Julian Maurice --- Created attachment 182724 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=182724&action=edit Bug 39967: Reload session after modification Otherwise the existing $session still holds outdated data that is written on the next flush, which overwrites what has been set by C4::Search::History::set_to_session Test plan: 1. Enable LoadSearchHistoryToTheFirstLoggedUser and EnableOpacSearchHistory 2. Go to OPAC. If logged in, log out. 3. Go to search history and remove all searches if any. 4. Perform a search, make sure it appears in your search history 5. Log in and go to your search history. See that your unauthenticated search history has been added to your account's search history 6. Reload the page multiple times. See that the unauthenticated search history is *NOT* re-added after each page reload -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list [email protected] 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/
