Koha-clms/Koha/REST/Plugin
Tomas Cohen Arazi b3ed9a4042
Bug 25493: Make ->unhandled_exception use Koha::Logger
To test:
1. Edit the Koha/REST/V1/Cities.pm 'list' method adding
   die("Nada"); before the render step.
2. Restart plack and try the endpoint
=> SUCCESS: The message is generic, but you see something is
logged in /var/log/koha/kohadev/api-error.log
3. Change die("Nada"); for a real exception like:

    use Koha::Exceptions;
    Koha::Exceptions::DuplicateObject->throw("Nada");
4. Repeat 2.
=> SUCCESS: The message is generic, but a meaningful text is added to
the logs.
5. Point your browser to the /api/v1/hola route from your dev
   environment
=> FAIL: Wow, such a weird error
6. Apply this patch
7. Restart plack and repeat 2, 3 and 4
=> SUCCESS: No behaviour change
8. Repeat 5
=> SUCCESS: The regular Mojolicious 404 weird page
9. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
2020-05-19 15:20:43 +01:00
..
Exceptions.pm Bug 25493: Make ->unhandled_exception use Koha::Logger 2020-05-19 15:20:43 +01:00
Objects.pm Bug 24502: object.search also filter by prefetched columns 2020-03-03 09:16:55 +00:00
Pagination.pm Bug 24545: Fix license statements 2020-02-24 13:31:26 +00:00
PluginRoutes.pm Bug 25131: (QA follow-up) Move test for needs_install outside the plugin 2020-04-16 19:21:19 +01:00
Query.pm Bug 24830: Fix parse_prefetch recursion and +count cases 2020-03-20 15:18:40 +00:00