From d537b9311721fded29028ac60740368ec5afe07c Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Thu, 2 Aug 2012 16:55:24 +0200 Subject: [PATCH] [PASSED QA] 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 --- C4/Serials.pm | 2 +- .../prog/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 3cbd042..5676883 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 30e8627..79fd2af 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() { -- 1.7.9.5