Bug 29859: Use iterator instead of as_list

[srvgit] / serials / subscription-numberpatterns.pl

diff --git a/serials/subscription-numberpatterns.pl b/serials/subscription-numberpatterns.pl
index 99969b1..19b917b 100755 (executable)
--- a/serials/subscription-numberpatterns.pl
+++ b/serials/subscription-numberpatterns.pl
@@ -30,10 +30,18 @@ Manage numbering patterns
 use Modern::Perl;
 use CGI qw ( -utf8 );
 
-use C4::Auth;
-use C4::Output;
-use C4::Serials::Numberpattern;
-use C4::Serials::Frequency;
+use C4::Auth qw( get_template_and_user );
+use C4::Output qw( output_html_with_http_headers );
+use C4::Serials::Numberpattern qw(
+    AddSubscriptionNumberpattern
+    DelSubscriptionNumberpattern
+    GetSubscriptionNumberpattern
+    GetSubscriptionNumberpatternByName
+    GetSubscriptionNumberpatterns
+    GetSubscriptionsWithNumberpattern
+    ModSubscriptionNumberpattern
+);
+use C4::Serials::Frequency qw( GetSubscriptionFrequencies );
 
 my $input = CGI->new;
 my ($template, $loggedinuser, $cookie, $flags) = get_template_and_user( {
@@ -102,9 +110,6 @@ if($op && ($op eq 'new' || $op eq 'modify')) {
         }
     }
     my @frequencies = GetSubscriptionFrequencies();
-    my @subtypes;
-    push @subtypes, { value => $_ } for (qw/ issues weeks months /);
-
     my $languages = [ map {
         {
             language => $_->{iso639_2_code},
@@ -115,7 +120,6 @@ if($op && ($op eq 'new' || $op eq 'modify')) {
     $template->param(
         $op => 1,
         frequencies_loop => \@frequencies,
-        subtypes_loop => \@subtypes,
         locales => $languages,
     );
     output_html_with_http_headers $input, $cookie, $template->output;