[Koha-bugs] [Bug 27145] Patron deletion via intranet doesn't handle exceptions well

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27145

Joonas Kylmälä <joonas.kylmala at iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #127545|0                           |1
        is obsolete|                            |
 Attachment #127546|0                           |1
        is obsolete|                            |

--- Comment #7 from Joonas Kylmälä <joonas.kylmala at iki.fi> ---
Created attachment 127608
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=127608&action=edit
Bug 27145: Rethrow all other exceptions

Bug 14708 introduced a try catch around $patron->delete in commit:
  "Bug 14708: (QA follow-up) Use try/catch blocks when calling"

However, in the catch block it only assumes the exception was from trying to
delete anonymous patron when it can be anything else as well, the code should
be modified so that it will handle the anonymous patron case and if it is
anything else we log the other exception.

Signed-off-by: Martin Renvoize <martin.renvoize at ptfs-europe.com>

Signed-off-by: Joonas Kylmälä <joonas.kylmala at iki.fi>

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