Bug 13430 - Serial enumeration not visible when handling Serial Items. Show enumerations for Serial Items in the circ- and member-modules.
Summary: Serial enumeration not visible when handling Serial Items. Show enumerations ...
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Olli-Antti Kivilahti
QA Contact: Testopia
URL:
Keywords:
Depends on: 12085
Blocks:
  Show dependency treegraph
 
Reported: 2014-12-10 14:47 UTC by Olli-Antti Kivilahti
Modified: 2016-05-03 20:11 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 13430 - Serial enumeration not visible when handling Serial Items. Circ- and member-modules. (11.04 KB, patch)
2014-12-10 15:06 UTC, Olli-Antti Kivilahti
Details | Diff | Splinter Review
Bug 13430 - Simple version - Serial enumeration not visible when handling Serial Items. Circ- and member-modules. (4.50 KB, patch)
2014-12-15 14:45 UTC, Olli-Antti Kivilahti
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Olli-Antti Kivilahti 2014-12-10 14:47:50 UTC
Our librarians need to know which enumerations of an Serial Item are on hold for a borrower and which are checked out. It is not enough to know just the title + author.

This patch simply shows the koha.items.enumchron-field in the AJAX-driven checkout and holds-views and if no Items with enumchron are present, the column is automatically hidden.
Comment 1 Olli-Antti Kivilahti 2014-12-10 15:06:56 UTC
Created attachment 34280 [details] [review]
Bug 13430 - Serial enumeration not visible when handling Serial Items. Circ- and member-modules.

Adding enumchron (Vol info) to circ/circulation.pl and members/moremember.pl.
Making enumchron (Vol info)-column hide when it is not used.

TEST PREPARATIONS:
-1. Find a Borrower/parent with a child/guarantee.
-0. Make a Subscription (creating Items when receiving serial numbers) to a Biblio and receive atleast 4 numbers.

TEST PLAN (HOLDS ENUMERATED):
1. Make a reservation of two of the Serial Items for the Borrower and the Child, totaling 4 reservations.
2. Check-in the reserved books to make sure an Item-level hold is in place and real Items match the reservations.
3. Both the Borrower and Child have a "Vol info"-column with the Item enumchron (enumeration or chronology)
   in their "Holds"-tab in circ/circulation.pl and members/moremember.pl

TEST PLAN (HOLDS UNENUMERATED):
4. Make two arbitrary reserves for both Parent and Child.
5. Observe that in the "Holds"-tab they have now two Items with enumchron and two without.
6. Check-out the reserved Items for the respective Parent/Child.
7. Observe that in the "Holds"-tab there is no "Vol info"-column.

TEST PLAN (CHECKOUTS ENUMERATED):
8.  Make two arbitrary check-outs for the Parent and the Child.
9.  In the "Checkouts"-tab, observe the "Vol info"-column with two Items having an enumchron and two items without.
10. In the "Relatives' checkouts"-tab observe that the Child of the Parent and the Parent of the Child can see
    the "Vol info"-tab with their relatives' loans proper enumerations.

TEST PLAN (CHECKOUTS UNENUMERATED):
11. Check-in the Serial Items for both the Parent and the Child.
12. Observe the missing "Vol info"-tab in the "Checkouts"-tab and in the "Relatives' checkouts"-tab.
Comment 2 Olli-Antti Kivilahti 2014-12-15 14:45:08 UTC
Created attachment 34430 [details] [review]
Bug 13430 - Simple version - Serial enumeration not visible when handling Serial Items. Circ- and member-modules.

Adding enumchron (Vol info) to circ/circulation.pl and members/moremember.pl "Checkouts"-,
"Relatives checkouts"- and "Holds"-views.

TEST PREPARATIONS:
-1. Find a Borrower/parent with a child/guarantee.
-0. Make a Subscription (creating Items when receiving serial numbers) to a Biblio and receive atleast 4 numbers.

TEST PLAN (HOLDS ENUMERATED):
1. Make a reservation of two of the Serial Items for the Borrower and the Child, totaling 4 reservations.
2. Check-in the reserved books to make sure an Item-level hold is in place and real Items match the reservations.
3. Both the Borrower and Child have an enumchron (enumeration or chronology) after the title component of
   the Item's title-column in their "Holds"-tab in circ/circulation.pl and members/moremember.pl

TEST PLAN (CHECKOUTS ENUMERATED):
4.  Make two arbitrary check-outs for the Parent and the Child.
5.  In the "Checkouts"-tab, observe the two Items having an enumchron and two items without.
6.  In the "Relatives' checkouts"-tab observe that the Child of the Parent and the Parent of the Child can see
    the enumchrons on their relatives' loans.
Comment 3 Olli-Antti Kivilahti 2014-12-15 14:46:45 UTC
Added an alternative "Simple version" where no extra column is added like in the opac it is, but simply the enumchron is appended after the title.

Use only the other version. not both!

This is because the initial commit is not compatible with 3.16.4
Comment 4 Paola Rossi 2014-12-19 15:18:00 UTC
I've applied the first patch against master 3.19.00.000

About this step:
>TEST PLAN (HOLDS ENUMERATED):
>1. Make a reservation of two of the Serial Items for the Borrower and the Child, totaling 4 reservations.

I tried to place the second hold/reservation on a serial for the same "Borrower", but koha immediately warned me so:

Cannot place hold
Borrower already has a hold on this item

and, after selecting the "Place hold" button, koha prevented me from placing the hold by the message:

- This patron had already placed a hold on this item
Please cancel the previous hold first

I could not place 2 holds/reservations for a Borrower on the items of 1 serial ERROR, as required.

So I pass the patch to "Failed QA" status.
Comment 5 Olli-Antti Kivilahti 2015-08-10 14:29:57 UTC
It looks like I had missed a dependency there, and the dependency is quite a deep Koha modification allowing people to place multiple holds on one Biblio.

We are rebasing all our production features against master when we upgrade to 3.22.
We will upgrade when 3.22 comes out :)
Comment 6 Blou 2016-05-03 20:11:49 UTC
Olli-Antti, any plan to fix this patch?

Our first attemp at #14668 included changes you're making for this one.  We cut them but are hoping this one goes through.  Please let us know if you do not plan to move ahead.  

Thanks!