[Koha-bugs] [Bug 14364] Allow automatically canceled expired waiting holds to fill the next hold

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14364

Héctor Eduardo Castro Avalos <hector.hecaxmmx at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA
                 CC|                            |hector.hecaxmmx at gmail.com,
                   |                            |veron at veron.ch

--- Comment #17 from Héctor Eduardo Castro Avalos <hector.hecaxmmx at gmail.com> ---
(In reply to Nick Clemens from comment #16)
> The holds transfer as expected, and the patron gets the email but I am not
> getting the admin email.
> 
> I also got the error below when running
> misc/cronjobs/holds/cancel_expired_holds.pl:
> 
> DBD::mysql::st execute failed: Duplicate entry '6' for key 'PRIMARY' [for
> Statement "
>             INSERT INTO old_reserves
>             SELECT * FROM reserves
>             WHERE  reserve_id = ?
>         " with ParamValues: 0='6'] at C4/Reserves.pm line 1080.
> Can't locate object method "new" via package "C4::Dates" (perhaps you forgot
> to load "C4::Dates"?) at C4/Reserves.pm line 2073.

You're right Nick, patch Fail QA since Marc has been removed C4::Dates from
system.

Kyle, please remove C4::Dates and use Koha::DateUtils instead.

substitute  => { today => C4::Dates->new()->output() }, in line 2077

I add here Marc since he removed the C4::Dates from Koha.

-- 
You are receiving this mail because:
You are watching all bug changes.