[Koha-devel] SIG DIE in returns.pl

[Jesse]

Just to confirm, that was added as a debugging aid and I forgot to take it
out when I was done. Thanks for catching it!

2017-12-11 1:31 GMT-07:00 Fridolin SOMERS <fridolin.somers at biblibre.com>:

> Cool, thanks for creating the post on bugzilla.
> I see it is Passed QA
>
>
> Le 04/12/2017 à 20:09, Jonathan Druart a écrit :
>
>> See bug 19746, please SO+QA
>>
>> On Thu, 30 Nov 2017 at 05:57 Fridolin SOMERS <
>> fridolin.somers at biblibre.com>
>> wrote:
>>
>> Hie,
>>>
>>> With plack when some code fails, there is always in log a reference to
>>> circ/returns.pl o_O at start of stack trace.
>>>
>>> I found that there is in this page :
>>>
>>> use Carp 'verbose';
>>> $SIG{ __DIE__ } = sub { Carp::confess( @_ ) };
>>>
>>> Added by commit bb6277ffcc593685554112d770ac273c9efeda33
>>> Bug 14464: Add ability to cancel waiting holds from checkin screen
>>>
>>> Is this for debug purpopse only ?
>>> Why only this page ?
>>> Should it be moved to plack.psgi ?
>>>
>>> Regards,
>>>
>>> --
>>> Fridolin SOMERS <fridolin.somers at biblibre.com>
>>> BibLibre - software and system maintainer
>>> _______________________________________________
>>> Koha-devel mailing list
>>> Koha-devel at lists.koha-community.org
>>> http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
>>> website : http://www.koha-community.org/
>>> git : http://git.koha-community.org/
>>> bugs : http://bugs.koha-community.org/
>>>
>>>
>>
> --
> Fridolin SOMERS <fridolin.somers at biblibre.com>
> BibLibre - software and system maintainer
> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha-community.org
> http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
> website : http://www.koha-community.org/
> git : http://git.koha-community.org/
> bugs : http://bugs.koha-community.org/
>



-- 
Jesse Weaver
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20171211/abc2ba98/attachment.html>