Bug 30081

Summary: Display item type in patron's holds table
Product: Koha Reporter: Lari Taskula <lari.taskula>
Component: Staff interfaceAssignee: Lari Taskula <lari.taskula>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, gmcharlt, kyle, magnus
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.05.00
Attachments: Bug 30081: Add item type column to holds table
Bug 30081: Use itemtype description instead of itemtype code
Bug 30081: Add item type column to holds table
Bug 30081: Use itemtype description instead of itemtype code
Bug 30081: Add item type column to holds table
Bug 30081: Use itemtype description instead of itemtype code

Description Lari Taskula 2022-02-11 12:53:35 UTC
Add new column item type to holds table in patron's details and check out views. Some libraries will find this information useful for distinguishing inter library holds from normal holds.
Comment 1 Lari Taskula 2022-02-11 13:01:03 UTC
Created attachment 130490 [details] [review]
Bug 30081: Add item type column to holds table

Add new column item type to holds table in patron's details and check
out views. Some libraries will find this information useful for
distinguishing inter library holds from normal holds.

To test:
1. Apply patch
2. Place a hold and confirm it
3. Go to cgi-bin/koha/circ/circulation.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
4. Click "1 Hold(s)" tab
5. Observe new column "Item type"
6. Confirm the item type is correct
7. Go to moremember.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
8. Repeat steps 4-6
Comment 2 Lari Taskula 2022-02-11 13:08:57 UTC
Created attachment 130492 [details] [review]
Bug 30081: Use itemtype description instead of itemtype code

To test:
1. Use same plan as previous patch, but observe full item type description
   instead of item type code
Comment 3 Owen Leonard 2022-02-11 13:15:03 UTC
Created attachment 130493 [details] [review]
Bug 30081: Add item type column to holds table

Add new column item type to holds table in patron's details and check
out views. Some libraries will find this information useful for
distinguishing inter library holds from normal holds.

To test:
1. Apply patch
2. Place a hold and confirm it
3. Go to cgi-bin/koha/circ/circulation.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
4. Click "1 Hold(s)" tab
5. Observe new column "Item type"
6. Confirm the item type is correct
7. Go to moremember.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
8. Repeat steps 4-6

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 4 Owen Leonard 2022-02-11 13:15:07 UTC
Created attachment 130494 [details] [review]
Bug 30081: Use itemtype description instead of itemtype code

To test:
1. Use same plan as previous patch, but observe full item type description
   instead of item type code

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Lari Taskula 2022-02-11 13:20:08 UTC
That has to be the quickest sign-off I've seen in all these years :D
Comment 6 Katrin Fischer 2022-02-13 14:20:25 UTC
Created attachment 130539 [details] [review]
Bug 30081: Add item type column to holds table

Add new column item type to holds table in patron's details and check
out views. Some libraries will find this information useful for
distinguishing inter library holds from normal holds.

To test:
1. Apply patch
2. Place a hold and confirm it
3. Go to cgi-bin/koha/circ/circulation.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
4. Click "1 Hold(s)" tab
5. Observe new column "Item type"
6. Confirm the item type is correct
7. Go to moremember.pl?borrowernumber=1 with
   borrowernumber being the id of your patron
8. Repeat steps 4-6

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Katrin Fischer 2022-02-13 14:20:28 UTC
Created attachment 130540 [details] [review]
Bug 30081: Use itemtype description instead of itemtype code

To test:
1. Use same plan as previous patch, but observe full item type description
   instead of item type code

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 8 Katrin Fischer 2022-02-13 14:21:03 UTC
(In reply to Lari Taskula from comment #5)
> That has to be the quickest sign-off I've seen in all these years :D

Maybe fastest QA too? The wonder of small changes :)
Comment 9 Katrin Fischer 2022-02-13 14:36:38 UTC
*** Bug 29982 has been marked as a duplicate of this bug. ***
Comment 10 Fridolin Somers 2022-02-16 21:30:24 UTC
(In reply to Katrin Fischer from comment #8)
> (In reply to Lari Taskula from comment #5)
> > That has to be the quickest sign-off I've seen in all these years :D
> 
> Maybe fastest QA too? The wonder of small changes :)

Go for fastest master push ;)
Comment 11 Fridolin Somers 2022-02-16 21:52:39 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄
Comment 12 Fridolin Somers 2022-02-17 07:24:24 UTC
We really should add columns settings in this table.
Comment 13 Katrin Fischer 2022-02-19 13:25:39 UTC
(In reply to Fridolin Somers from comment #12)
> We really should add columns settings in this table.

I filed it: Bug 30138 - Add table configuration to holds table in patron account in staff