From c2cb04fdbf23eedbf2806c054d43bc3eb867bbee Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc=20V=C3=A9ron?= Date: Tue, 5 Jul 2016 19:23:59 +0200 Subject: [PATCH] Bug 14138: Patroncard: Warn user if PDF creation fails First step: Change patroncards/create-pdf.pl to redirect with an an error code instead of writing an ivalid pdf. Updated patch to display error and test plan follows. --- .../prog/en/includes/patroncards-errors.inc | 12 +++++- .../prog/en/modules/patroncards/manage.tt | 1 + patroncards/create-pdf.pl | 44 +++++++++++++++----- 3 files changed, 45 insertions(+), 12 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patroncards-errors.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patroncards-errors.inc index 136cfc4..1907eb0 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/patroncards-errors.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patroncards-errors.inc @@ -1,7 +1,17 @@ -[% IF ( error ) %] +[% IF ( error || CGI.param('pdferr') ) %]

WARNING: + [% IF CGI.param('pdferr') %] + Error while creating the PDF file. + [% IF CGI.param('errnocards') %]No cards created (empty batch or list?)[% END %] + [% IF CGI.param('errba') %]Batch: [% CGI.param('errba') %][% END %] + [% IF CGI.param('errpl') %]Patron list: [% CGI.param('errpl') %][% END %] + [% IF CGI.param('errlo') %]Layout: [% CGI.param('errlo') %][% END %] + [% IF CGI.param('errtpl') %]Template: [% CGI.param('errtpl') %][% END %] + + Please have your system administrator check the error log for details. + [% END %] [% IF ( error == 101 ) %] The database returned an error while [% IF ( card_element ) %]saving [% card_element %] [% element_id %][% ELSE %]attempting a save operation[% END %]. Please have your system administrator check the error log for details. [% ELSIF ( error == 102 ) %] diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt index a75a924..efdf9fb 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/patroncards/manage.tt @@ -1,3 +1,4 @@ +[% USE CGI %] [% BLOCK translate_card_element %] [%- SWITCH element -%] [%- CASE 'layout' -%]layout diff --git a/patroncards/create-pdf.pl b/patroncards/create-pdf.pl index 1811f0f..7b12e2e 100755 --- a/patroncards/create-pdf.pl +++ b/patroncards/create-pdf.pl @@ -45,8 +45,6 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user({ flagsrequired => { tools => 'label_creator' }, debug => 1, }); - - my $batch_id = $cgi->param('batch_id') if $cgi->param('batch_id'); my $template_id = $cgi->param('template_id') || undef; my $layout_id = $cgi->param('layout_id') || undef; @@ -58,13 +56,17 @@ my $patronlist_id = $cgi->param('patronlist_id'); my $items = undef; # items = cards my $new_page = 0; -my $pdf_file = (@label_ids || @borrower_numbers ? "card_single_" . scalar(@label_ids || @borrower_numbers) : "card_batch_$batch_id"); -print $cgi->header( -type => 'application/pdf', - -encoding => 'utf-8', - -attachment => "$pdf_file.pdf", - ); +# Wrap pdf creation part into an eval, some vars need scope outside eval +my $pdf_ok; +my $pdf; +my $pdf_file; +my $cardscount = 0; + +#Note fo bug 14138: Indenting follows in separate patch to ease review +eval { +$pdf_file = (@label_ids || @borrower_numbers ? "card_single_" . scalar(@label_ids || @borrower_numbers) : "card_batch_$batch_id"); -my $pdf = C4::Creators::PDF->new(InitVars => 0); +$pdf = C4::Creators::PDF->new(InitVars => 0); my $batch = C4::Patroncards::Batch->retrieve(batch_id => $batch_id); my $pc_template = C4::Patroncards::Template->retrieve(template_id => $template_id, profile_id => 1); my $layout = C4::Patroncards::Layout->retrieve(layout_id => $layout_id); @@ -125,6 +127,7 @@ if ($layout_xml->{'page_side'} eq 'B') { # rearrange items on backside of page t CARD_ITEMS: foreach my $item (@{$items}) { if ($item) { + $cardscount ++; my $borrower_number = $item->{'borrower_number'}; my $card_number = GetMember(borrowernumber => $borrower_number)->{'cardnumber'}; @@ -231,9 +234,28 @@ foreach my $item (@{$items}) { ($llx, $lly, $new_page) = $pc_template->get_next_label_pos(); $pdf->Page() if $new_page; } +# No errors occured within eval, we can issue the pdf +$pdf_ok = 1 if ($cardscount > 0); +}; # end of eval block -$pdf->End(); - -# FIXME: Possibly do a redirect here if there were error encountered during PDF creation. +if ($pdf_ok) { + #issue the pdf + print $cgi->header( -type => 'application/pdf', + -encoding => 'utf-8', + -attachment => "$pdf_file.pdf", + ); + $pdf->End(); +} +else { + # warn user that pdf is not created + my $errparams = '&pdferr=1'; + $errparams .= "&errba=$batch_id" if $batch_id; + $errparams .= "&errpl=$patronlist_id" if $patronlist_id; + $errparams .= "&errlo=$layout_id" if $layout_id; + $errparams .= "&errtpl=$template_id" if $template_id; + $errparams .= "&errnocards=1" if !$cardscount; + + print $cgi->redirect("/cgi-bin/koha/patroncards/manage.pl?card_element=batch$errparams"); +} 1; -- 1.7.10.4