Bug 15378

Summary: Remove 'lost' items from course reserves
Product: Koha Reporter: Nick Clemens <nick>
Component: Course reservesAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Josef Moravec <josef.moravec>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, josef.moravec, kyle, nicolas.legrand, sandboxes, severine.queune, tmisilo
Version: master   
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 15378: Allow removal of lost items from course reserves
Bug 15378: Allow removal of lost items from course reserves
Bug 15378: Allow removal of lost items from course reserves

Description Nick Clemens 2015-12-15 14:16:59 UTC
Currently lost items cannot be removed from course reserves.  It should be possible to remove these from a course without deleting the item.
Comment 1 Owen Leonard 2016-07-12 15:58:23 UTC
I'm working on a cosmetic change to this template, and it looks like the prevention of lost item removal is controlled by template logic. It doesn't appear that there is any real problem removing lost items from course reserves.

Kyle, if you could confirm that there isn't an unseen side effect of removing lost items I will submit a patch enabling it.
Comment 2 Séverine Queune 2017-09-11 15:09:29 UTC
The BULAC (http://www.bulac.fr/) would like to sponsore this development.

Nick, Kyle, did you work further on the patch ?

As long as “Change sponsored?” is set on “Seeking developer”, anyone wanting to quote us for the development of such a feature or just ask for more information is welcome to write to us: <sigb@bulac.fr>.
Comment 3 Jonathan Druart 2017-10-12 16:40:09 UTC
Created attachment 68033 [details] [review]
Bug 15378: Allow removal of lost items from course reserves

It seems that this restriction is not necessary, it should be possible
to remove lost items from course reserves.

Test plan:
Create a course and add reserve with a lost item

=> Without this patch the "Remove" button is disabled and the item
cannot be removed from the course
=> With this patch applied the button is enable and the item can be
removed
Comment 4 Biblibre Sandboxes 2017-10-13 12:52:11 UTC
Patch tested with a sandbox, by Séverine QUEUNE <severine.queune@bulac.fr>
Comment 5 Biblibre Sandboxes 2017-10-13 12:52:32 UTC
Created attachment 68071 [details] [review]
Bug 15378: Allow removal of lost items from course reserves

It seems that this restriction is not necessary, it should be possible
to remove lost items from course reserves.

Test plan:
Create a course and add reserve with a lost item

=> Without this patch the "Remove" button is disabled and the item
cannot be removed from the course
=> With this patch applied the button is enable and the item can be
removed

Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Comment 6 Séverine Queune 2017-10-13 12:52:54 UTC
Items can now be removed of a course reserve applying this patch.
The value of lost status is untouched.
Thanks Jonathan !
Comment 7 Josef Moravec 2018-01-02 11:20:36 UTC
Created attachment 70225 [details] [review]
Bug 15378: Allow removal of lost items from course reserves

It seems that this restriction is not necessary, it should be possible
to remove lost items from course reserves.

Test plan:
Create a course and add reserve with a lost item

=> Without this patch the "Remove" button is disabled and the item
cannot be removed from the course
=> With this patch applied the button is enable and the item can be
removed

Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 8 Jonathan Druart 2018-01-02 15:12:40 UTC
Pushed to master for 18.05, thanks to everybody involved!