From ed49537002c3e0b8d9022ed878b63516a77e91f0 Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Thu, 2 Aug 2012 16:55:24 +0200 Subject: [PATCH] Bug 7688 follow-up: Small fixes for QA - # Subroutines::ProhibitExplicitReturnUndef: Got 1 violation(s) in C4::Serials::GetSubscriptionIrregularities - Bad template constructions fixed in serials/subscription-add.tt Signed-off-by: Kyle M Hall Signed-off-by: Bernardo Gonzalez Kriegel Signed-off-by: Paul Poulain Signed-off-by: Jared Camins-Esakov --- C4/Serials.pm | 2 +- .../en/modules/serials/subscription-add.tt | 20 ++++++++++++++++--- 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/C4/Serials.pm b/C4/Serials.pm index 3cbd0424ad..5676883c40 100644 --- a/C4/Serials.pm +++ b/C4/Serials.pm @@ -1335,7 +1335,7 @@ get the list of irregularities for a subscription sub GetSubscriptionIrregularities { my $subscriptionid = shift; - return undef unless $subscriptionid; + return unless $subscriptionid; my $dbh = C4::Context->dbh; my $query = qq{ diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt index 30e8627949..79fd2afdc1 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/subscription-add.tt @@ -653,7 +653,11 @@ $(document).ready(function() { [% FOREACH subt IN subtype %] - [% END %] @@ -684,7 +692,13 @@ $(document).ready(function() { -- 2.39.5