Bug 8479 - Opac Holds being created with priority set to NULL
Summary: Opac Holds being created with priority set to NULL
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Hold requests (show other bugs)
Version: 3.10
Hardware: All All
: P5 - low normal (vote)
Assignee: Colin Campbell
QA Contact: Mason James
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-20 09:31 UTC by Colin Campbell
Modified: 2013-12-05 20:05 UTC (History)
5 users (show)

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


Attachments
Proposed Patch (1.80 KB, patch)
2012-07-20 10:22 UTC, Colin Campbell
Details | Diff | Splinter Review
Bug 8479 Calculate rank irrespective of display option (1.84 KB, patch)
2012-08-03 16:23 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Colin Campbell 2012-07-20 09:31:18 UTC
As part of the patch introduced by bug 7398 the calculation of priority is governed by the display option. However the if the rank is not calculated because you have not opted to display it undef is passed to AddReserve and holds are not being picked up by subsequent pull lists
Comment 1 Colin Campbell 2012-07-20 10:22:39 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2012-08-03 16:23:51 UTC
Created attachment 11344 [details] [review]
Bug 8479 Calculate rank irrespective of display option

Calculation of rank was being skipped based on display
option but it is used in calculating the rank passed
to AddReserve resulting in reserves being created with
null priorities causing them to be skipped in subsequent
processing

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 3 Mason James 2012-08-10 07:27:30 UTC
(In reply to comment #2)
> Created attachment 11344 [details] [review]
> Bug 8479 Calculate rank irrespective of display option
> 
> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

patch looks good, passing QA


mason@xen1:~/git/head$ koha-qa.pl
        * 90dd666 Bug 8479 Calculate rank irrespective of display option
                opac/opac-reserve.pl

- perlcritic-progressive tests... OK
- perl -c syntax tests... OK
- xt/tt_valid.t tests... OK
- xt/author/valid-template.t tests... OK
Comment 4 Chris Cormack 2012-08-12 06:12:18 UTC
Patch doesn't apply cleanly on 3.8.x, if it is needed please submit a patch based on 3.8.x
Comment 5 Colin Campbell 2012-08-13 15:12:33 UTC
3.8 does not have the syspref introduced with bug 7398 so should not require this.
But will need applying id bug 7398 applied
Comment 6 Chris Cormack 2012-08-14 01:10:03 UTC
(In reply to comment #5)
> 3.8 does not have the syspref introduced with bug 7398 so should not require
> this.
> But will need applying id bug 7398 applied

Thanks for this Colin.
Comment 7 Chris Cormack 2013-04-20 22:10:28 UTC
Released in 3.10.0