Bug 10107

Summary: opacbookbag shouldn't prevent from adding items to a list in the OPAC
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: OPACAssignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: chris, katrin.fischer, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10107 - opacbookbag shouldn't prevent from adding items to a list in the OPAC
[SIGNED-OFF] Bug 10107 - opacbookbag shouldn't prevent from adding items to a list in the OPAC
PASSED-QA-Bug-10107-opacbookbag-shouldn-t-prevent...
Bug 10107 - opacbookbag shouldn't prevent from adding items to a list in the OPAC
Bug 10107 - opacbookbag shouldn't prevent from adding items to a list in the OPAC

Description Tomás Cohen Arazi 2013-04-24 14:16:42 UTC
To reproduce:
- Disable the opacbookbag syspref
- Login in the OPAC
- Create a list (I tested a private one)
- Search in your db for some biblios.
- Choose some and add them to the created list
- Go to your users's page, to the my lists tab
- Your list is empty.
Comment 1 Tomás Cohen Arazi 2013-04-24 14:28:33 UTC
I confirm this in both 'prog' and 'ccsr'
Comment 2 Tomás Cohen Arazi 2013-04-24 14:32:33 UTC
(In reply to comment #1)
> I confirm this in both 'prog' and 'ccsr'

Also only a bug on the search results page, in the detail page it works as expected.
Comment 3 Tomás Cohen Arazi 2013-04-24 14:44:08 UTC Comment hidden (obsolete)
Comment 4 Marcel de Rooy 2013-04-24 14:44:32 UTC
Tomas:
Via this report I connect to the encoding stuff: Search for something with diacritics. Try to add it to a list with bookbag disabled. You will see a mangled search expression without results and no item added.
Comment 5 Owen Leonard 2013-04-24 17:38:52 UTC Comment hidden (obsolete)
Comment 6 Katrin Fischer 2013-04-25 06:21:09 UTC
Created attachment 17665 [details] [review]
PASSED-QA-Bug-10107-opacbookbag-shouldn-t-prevent...
Comment 7 Marcel de Rooy 2013-04-25 09:46:30 UTC
(In reply to comment #4)
> Tomas:
> Via this report I connect to the encoding stuff: Search for something with
> diacritics. Try to add it to a list with bookbag disabled. You will see a
> mangled search expression without results and no item added.

Bug 10114 is created for this new bug.
Comment 8 MJ Ray (software.coop) 2013-04-25 11:09:51 UTC Comment hidden (obsolete)
Comment 9 Jared Camins-Esakov 2013-04-26 01:14:21 UTC
I cannot reproduce this problem. Is there anything else I might need to know? However, when I disable the opacbookbag syspref all the other javascript on the page breaks.
Comment 10 Katrin Fischer 2013-04-26 04:47:36 UTC
I did my test in CCSR and it worked like described. There was still a [News list] link in the dropdown on top of the result list, but if you clicked on it, nothing happened. After applying the patch and reloading the patch the normal popup came up and it worked nicely.
Comment 11 Jared Camins-Esakov 2013-04-26 11:56:30 UTC
This patch has been pushed to master and 3.12.x.

I was unable to replicate the problem, but this patch did not make things worse for me and fixed things for others so I have pushed anyway.
Comment 12 Katrin Fischer 2013-04-26 18:25:07 UTC Comment hidden (obsolete)
Comment 13 Chris Cormack 2013-04-27 21:35:14 UTC
Pushed to 3.10.x and 3.8.x will be in 3.10.6 and 3.8.13