Bug 7254

Summary: Show pending orders from baskets with closedate > 180 days
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: AcquisitionsAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch

Description Marcel de Rooy 2011-11-23 12:44:14 UTC
Current code for selecting pending orders does not select orders from baskets with a closedate > 180 days.
The code in Acquisition.pm already includes a FIXME for that.
IMHO this part of the where clause should just be removed; an order is pending until it is filled. If you do not want to fill it, delete it.
Comment 1 Marcel de Rooy 2011-11-23 12:55:55 UTC
Created attachment 6376 [details] [review]
Patch
Comment 2 Ian Walls 2011-11-23 13:57:16 UTC
SQL-only change that removes an unjustified restriction.  Signing off and passing QA in one.
Comment 3 Marcel de Rooy 2011-11-23 13:58:33 UTC
(In reply to comment #2)
> SQL-only change that removes an unjustified restriction.  Signing off and
> passing QA in one.

Within an hour! Great..
Comment 4 Paul Poulain 2011-11-24 14:46:29 UTC
Comment: this patch also removes the OR clodedate IS NULL, that is not related to the 180 days limit. Baskets were not removed if they are not closed. It seems logic to consider that a basket is pending if there are still things to recieve, the SQL query do that, so I accept the patch

Patch pushed, please test