Summary: | Staff client shows "Your lists:" and "public lists:" when there are none | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Lists | Assignee: | Colin Campbell <colin.campbell> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | normal | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, colin.campbell, frederic.durand |
Version: | 3.4 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Proposed Patch
Revised patch |
Description
Owen Leonard
2010-12-20 20:56:06 UTC
Testing a fix. Problem is some poor code in C4::VirtualShelves, it assigns a array_ref to an array in array context so that if you look at pubshelves in search.pl it is actually a 1 element array that element being a reference to an empty array i.e. [ [], ] so it is true consisting of 1 element. Need to ensure nowhere relies on incorrect behaviour Created attachment 3751 [details] [review] Proposed Patch With no lists already created (private or public ones) the "Add to" just contains "Cart" option, then all is working good for this point. As well the option to add biblios to 'New list' disappeared too, meaning it s quite more complicated and longer to create a new list than before. Created attachment 3781 [details] [review] Revised patch Add to patch: change to template to display new list option irrespective of existence of existing lists All is working perfectly. Pushed please test |