Bug 13343

Summary: Items are not embedded when sending a basket/shelf
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: chris, katrin.fischer, mtj, tomascohen
Version: Main   
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:
Bug Depends on:    
Bug Blocks: 13345    
Attachments: B
Bug 13343: Embed items when send a basket/shelf
[PASSED QA] Bug 13343: Embed items when send a basket/shelf

Description Jonathan Druart 2014-11-26 14:13:38 UTC
When a basket/shelf is downloaded, items are embedded, but not when sending it by email.

This seems to be a bug, we would expect to get the same file.
Comment 1 Jonathan Druart 2014-11-26 14:14:38 UTC
Set component to "architecture", the bug is on both interfaces (opac and intranet) and for baskets and shelves.
Comment 2 Jonathan Druart 2014-11-26 14:17:27 UTC Comment hidden (obsolete)
Comment 3 Chris Cormack 2014-12-22 07:21:24 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2014-12-27 12:30:35 UTC
Created attachment 34708 [details] [review]
[PASSED QA] Bug 13343: Embed items when send a basket/shelf

When a basket/shelf is downloaded, items are embedded, but not when
sending it by email.

We would expect to get the same file.

Test plan:
On Opac and intranet:
1/ Add records with items to your basket and send it by email.
Verify the file contains items.
2/ Add records with items to a list and send it by email.
Verify the file contains items.

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2015-02-11 07:15:05 UTC
A note about this: as things currently are, we will send out information that has been marked as not visible in the OPAC or labelled as internal note. I wonder if we should make including items in the staff/cart optional for the time being as fixing the problem with visibility is a much bigger issue.
There has been a discussion about this for OAI as well, but things OPAC side might need a different treatment (bug 12252).

I know this patch just makes things more consistent as one download includes items and the other does not - if people are ok with the idea of optional, I will try to work up a patch on a separate bug - not blocking this.
Comment 6 Jonathan Druart 2015-02-11 08:53:03 UTC
yes, we should at least call GetMarcBiblio with the 3rd param set to 1 (opac=1).
It is introduced by bug 12252.
Comment 7 Katrin Fischer 2015-02-11 09:12:33 UTC
The opac=1 flag will remove items hidden with opachiddenitems - so it would be really good to add that here.
Comment 8 Tomás Cohen Arazi 2015-02-12 18:38:31 UTC
Patch pushed to master.

Thanks Jonathan!
Comment 9 Chris Cormack 2015-02-14 08:15:59 UTC
Pushed to 3.18.x will be in 3.18.4
Comment 10 Mason James 2015-02-26 05:04:48 UTC
Pushed to 3.16.x, will be in 3.16.8r
Comment 11 Chris Cormack 2015-08-09 22:55:43 UTC
(In reply to Katrin Fischer from comment #5)
> A note about this: as things currently are, we will send out information
> that has been marked as not visible in the OPAC or labelled as internal
> note. I wonder if we should make including items in the staff/cart optional
> for the time being as fixing the problem with visibility is a much bigger
> issue.
> There has been a discussion about this for OAI as well, but things OPAC side
> might need a different treatment (bug 12252).
> 
> I know this patch just makes things more consistent as one download includes
> items and the other does not - if people are ok with the idea of optional, I
> will try to work up a patch on a separate bug - not blocking this.

I'd still like to see this, Koha should be the most privacy conscious library system out there. Let's try to make this our goal.

I'd be happy to test and sign off a patch that respects the hidden settings