From 0b60dcbeacad972431298868fa8c4bdf2db40d77 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Fri, 30 Oct 2015 10:27:10 -0400 Subject: [PATCH] Bug 14310 [QA Followup] - Deal with error conditions Signed-off-by: Jonathan Druart Signed-off-by: Brendan A Gallagher --- .../prog/en/includes/strings.inc | 3 +++ koha-tmpl/intranet-tmpl/prog/en/js/holds.js | 21 +++++++++++++++-- svc/hold/resume | 8 ++++++- svc/hold/suspend | 23 ++++++++++++++----- 4 files changed, 46 insertions(+), 9 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/strings.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/strings.inc index 3995b6d113..62af2a3275 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/strings.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/strings.inc @@ -35,5 +35,8 @@ var SUSPEND = _("Suspend"); var SUSPEND_HOLD_ON = _("Suspend hold on"); var CLEAR_DATE_TO_SUSPEND_INDEFINITELY = _("Clear date to suspend indefinitely"); + var SUSPEND_HOLD_ERROR_DATE = _("Unable to suspend hold, invalid date"); + var SUSPEND_HOLD_ERROR_NOT_FOUND = _("Unable to suspend hold, hold not found"); + var RESUME_HOLD_ERROR_NOT_FOUND = _("Unable to resume, hold not found"); //]]> diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/holds.js b/koha-tmpl/intranet-tmpl/prog/en/js/holds.js index a6c8ca6ea6..c009ba7651 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/js/holds.js +++ b/koha-tmpl/intranet-tmpl/prog/en/js/holds.js @@ -162,7 +162,14 @@ $(document).ready(function() { var id = $(this).attr("id").replace("resume", ""); var hold = holds[id]; $.post('/cgi-bin/koha/svc/hold/resume', { "reserve_id": hold.reserve_id }, function( data ){ - holdsTable.api().ajax.reload(); + if ( data.success ) { + holdsTable.api().ajax.reload(); + } else { + if ( data.error == "HOLD_NOT_FOUND" ) { + alert ( RESUME_HOLD_ERROR_NOT_FOUND ); + holdsTable.api().ajax.reload(); + } + } }); }); }); @@ -209,7 +216,17 @@ $(document).ready(function() { e.preventDefault(); $.post('/cgi-bin/koha/svc/hold/suspend', $('#suspend-modal-form').serialize(), function( data ){ $('#suspend-modal').modal('hide'); - holdsTable.api().ajax.reload(); + if ( data.success ) { + holdsTable.api().ajax.reload(); + } else { + if ( data.error == "INVALID_DATE" ) { + alert( SUSPEND_HOLD_ERROR_DATE ); + } + else if ( data.error == "HOLD_NOT_FOUND" ) { + alert ( SUSPEND_HOLD_ERROR_NOT_FOUND ); + holdsTable.api().ajax.reload(); + } + } }); }); diff --git a/svc/hold/resume b/svc/hold/resume index 03cd017d84..52f35084d0 100755 --- a/svc/hold/resume +++ b/svc/hold/resume @@ -38,11 +38,17 @@ if ( $auth_status ne "ok" ) { } binmode STDOUT, ":encoding(UTF-8)"; -print $input->header( -type => 'text/plain', -charset => 'UTF-8' ); +print $input->header( -type => 'text/json', -charset => 'UTF-8' ); my $reserve_id = $input->param('reserve_id'); my $hold = Koha::Holds->find( $reserve_id ); + +unless ( $hold ) { + print to_json( { success => 0, error => "HOLD_NOT_FOUND" } ); + exit; +} + $hold->resume(); print to_json( { success => !$hold->suspend() } ); diff --git a/svc/hold/suspend b/svc/hold/suspend index 880e41aee5..bd5382b1cb 100755 --- a/svc/hold/suspend +++ b/svc/hold/suspend @@ -30,22 +30,33 @@ use Koha::Holds; my $input = new CGI; my ( $auth_status, $sessionID ) = - check_cookie_auth( $input->cookie('CGISESSID'), - { circulate => 'circulate_remaining_permissions' } ); + check_cookie_auth( $input->cookie('CGISESSID'), { circulate => 'circulate_remaining_permissions' } ); if ( $auth_status ne "ok" ) { exit 0; } binmode STDOUT, ":encoding(UTF-8)"; -print $input->header( -type => 'text/plain', -charset => 'UTF-8' ); +print $input->header( -type => 'text/json', -charset => 'UTF-8' ); my $reserve_id = $input->param('reserve_id'); my $suspend_until = $input->param('suspend_until') || undef; -$suspend_until = dt_from_string( $suspend_until ) if $suspend_until; +if ($suspend_until) { + eval { $suspend_until = dt_from_string($suspend_until) }; -my $hold = Koha::Holds->find( $reserve_id ); -$hold->suspend_hold( $suspend_until ); + if ($@) { + print to_json( { success => 0, error => 'INVALID_DATE' } ); + exit; + } +} + +my $hold = Koha::Holds->find($reserve_id); +unless ($hold) { + print to_json( { success => 0, error => 'HOLD_NOT_FOUND' } ); + exit; +} + +$hold->suspend_hold($suspend_until); print to_json( { success => $hold->suspend() } ); -- 2.39.5