Merge branch 'master' of git@lx-office.linet-services.de:lx-office-erp
authorSven Schöling <s.schoeling@linet-services.de>
Fri, 15 Apr 2011 09:51:32 +0000 (11:51 +0200)
committerSven Schöling <s.schoeling@linet-services.de>
Fri, 15 Apr 2011 09:51:32 +0000 (11:51 +0200)
SL/Auth.pm
SL/DBUpgrade2.pm
SL/Form.pm
SL/Iconv.pm
SL/Template/OpenDocument.pm
SL/Template/Plugin/T8.pm
bin/mozilla/ustva.pl
modules/fallback/List/MoreUtils.pm
t/structure/globals.t [new file with mode: 0644]

index 0951883..978fe49 100644 (file)
@@ -971,7 +971,7 @@ sub delete_group {
   my $self = shift;
   my $id   = shift;
 
-  my $form = $main::from;
+  my $form = $main::form;
 
   my $dbh  = $self->dbconnect();
   $dbh->begin_work;
index af5037a..a583af8 100644 (file)
@@ -245,7 +245,7 @@ sub process_perl_script {
 
   $db_charset ||= Common::DEFAULT_CHARSET;
 
-  my $iconv = SL::Iconv::get_converter($file_charset, $db_charset);
+  my $iconv = SL::Iconv->new($file_charset, $db_charset);
 
   $dbh->begin_work();
 
index bcb7cd5..31020ae 100644 (file)
@@ -746,7 +746,7 @@ sub redirect_header {
   my $base_uri = $self->_get_request_uri;
   my $new_uri  = URI->new_abs($new_url, $base_uri);
 
-  die "Headers already sent" if $::self->{header};
+  die "Headers already sent" if $self->{header};
   $self->{header} = 1;
 
   my $cgi = $main::cgi || CGI->new('');
index c2f2247..d6ea0cb 100644 (file)
@@ -19,7 +19,7 @@ sub new {
   return $self;
 }
 
-sub get_converter {
+sub _get_converter {
   my ($from_charset, $to_charset) = @_;
 
   my $index             = join $SUBSCRIPT_SEPARATOR, $from_charset, $to_charset;
@@ -36,7 +36,7 @@ sub convert {
   $from_charset ||= Common::DEFAULT_CHARSET;
   $to_charset   ||= Common::DEFAULT_CHARSET;
 
-  my $converter = get_converter($from_charset, $to_charset);
+  my $converter = _get_converter($from_charset, $to_charset);
   return $converter->convert($text);
 }
 
index 552bc2f..88b4cff 100644 (file)
@@ -37,7 +37,7 @@ sub parse_foreach {
 
   my $ary = $self->_get_loop_variable($var, 1, @indices);
 
-  for (my $i = 0; $i < scalar(@{$ary}); $i++) {
+  for (my $i = 0; $i < scalar(@{$ary || []}); $i++) {
     $form->{"__first__"} = $i == 0;
     $form->{"__last__"} = ($i + 1) == scalar(@{$ary});
     $form->{"__odd__"} = (($i + 1) % 2) == 1;
index 6c584d7..ca62e86 100644 (file)
@@ -1,14 +1,19 @@
 package SL::Template::Plugin::T8;
 
 use strict;
+use parent qw( Template::Plugin::Filter );
 
-use Template::Plugin::Filter;
-use base qw( Template::Plugin::Filter );
+my $cached_instance;
+
+sub new {
+  my ($class) = shift;
+
+  return $cached_instance ||= $class->SUPER::new(@_);
+}
 
 sub init {
   my $self = shift;
 
-  # first arg can specify filter name
   $self->install_filter($self->{ _ARGS }->[0] || 'T8');
 
   return $self;
index 6466565..9953806 100644 (file)
@@ -74,11 +74,13 @@ use SL::User;
 #############################
 
 sub report {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  my $form     = $::form;
+  my $locale   = $::locale;
+  my %myconfig = %::myconfig;
 
-  my $myconfig = \%myconfig;
+  $::auth->assert('advance_turnover_tax_return');
 
   $form->{title} = $locale->text('UStVA');
   $form->{kz10}  = '';                       #Berichtigte Anmeldung? Ja =1 Nein=0
@@ -87,12 +89,12 @@ sub report {
              0, 4);
 
   my $department = '';
-  local $hide = '';
+  my $hide = '';
   $form->header;
 
   # Einlesen der Finanzamtdaten
   my $ustva = USTVA->new();
-  $ustva->get_config($userspath, 'finanzamt.ini');
+  $ustva->get_config($::userspath, 'finanzamt.ini');
 
   # Hier Einlesen der user-config
   # steuernummer entfernt für prerelease
@@ -109,7 +111,7 @@ sub report {
     co_accountnr3
   );
 
-  map { $form->{$_} = $myconfig->{$_} } @a;
+  map { $form->{$_} = $myconfig{$_} } @a;
 
   my $openings = $form->{FA_Oeffnungszeiten};
   $openings =~ s/\\\\n/<br>/g;
@@ -189,14 +191,14 @@ sub report {
 
   # Which COA is in use?
 
-  $ustva->get_coa($form, $myconfig);
+  $ustva->get_coa($form, \%myconfig);
 
   my $template_ref = {
     openings         => $openings,
     company_given    => $company_given,
     address_given    => $address_given,
     taxnumber_given  => $taxnumber_given,
-    taxnumber        => $myconfig->{taxnumber},
+    taxnumber        => $myconfig{taxnumber},
     select_year      => $select_year,
     period_local     => $period_local,
     method_local     => $method_local,
@@ -211,44 +213,46 @@ sub report {
 
 
 
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 
 
 sub help {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
   # parse help documents under doc
-  my $tmp = $form->{templates};
-  $form->{templates} = 'doc';
-  $form->{help}      = 'ustva';
-  $form->{type}      = 'help';
-  $form->{format}    = 'html';
-  &generate_ustva();
+  my $tmp = $::form->{templates};
+  $::form->{templates} = 'doc';
+  $::form->{help}      = 'ustva';
+  $::form->{type}      = 'help';
+  $::form->{format}    = 'html';
+  generate_ustva();
 
   #$form->{templates} = $tmp;
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 sub show {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
   #&generate_ustva();
-  no strict 'refs';
-  $lxdebug->leave_sub();
-  call_sub($form->{"nextsub"});
-  use strict 'refs';
+  $::lxdebug->leave_sub();
+  call_sub($::form->{"nextsub"});
 }
 
 sub ustva_vorauswahl {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  my $form     = $::form;
+  my $locale   = $::locale;
+  my %myconfig = %::myconfig;
+
+  $::auth->assert('advance_turnover_tax_return');
 
   my $select_vorauswahl;
 
@@ -259,7 +263,7 @@ sub ustva_vorauswahl {
   $form->{day}   = substr($date, 6, 2);
   $form->{month} = substr($date, 4, 2);
   $form->{year}  = substr($date, 0, 4);
-  $lxdebug->message(LXDebug::DEBUG1, qq|
+  $::lxdebug->message(LXDebug->DEBUG1, qq|
     Actual date from Database: $date\n
     Actual year from Database: $form->{year}\n
     Actual day from Database: $form->{day}\n
@@ -466,27 +470,27 @@ sub ustva_vorauswahl {
     }
     $select_vorauswahl .= qq|</select>|;
   }
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 
   return $select_vorauswahl;
 }
 
 #sub config {
-#  $lxdebug->enter_sub();
+#  $::lxdebug->enter_sub();
 #  config_step1();
-#  $lxdebug->leave_sub();
+#  $::lxdebug->leave_sub();
 #}
 
 sub debug {
-  $lxdebug->enter_sub();
-  $form->debug();
-  $lxdebug->leave_sub();
+  $::lxdebug->enter_sub();
+  $::form->debug();
+  $::lxdebug->leave_sub();
 }
 
 sub show_options {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
   #  $form->{PD}{$form->{type}} = "selected";
   #  $form->{DF}{$form->{format}} = "selected";
@@ -496,24 +500,24 @@ sub show_options {
   my $media  = qq|      <input type=hidden name="media" value="screen">|;
   my $format =
       qq|       <option value=html selected>|
-    . $locale->text('Preview')
+    . $::locale->text('Preview')
     . qq|</option>|;
-  if ($latex_templates) {
+  if ($::latex_templates) {
     $format .=
         qq|    <option value=pdf>|
-      . $locale->text('UStVA (PDF-Dokument)')
+      . $::locale->text('UStVA (PDF-Dokument)')
       . qq|</option>|;
   }
 
   #my $disabled= qq|disabled="disabled"|;
   #$disabled='' if ($form->{elster} eq '1' );
-  if ($form->{elster} eq '1') {
+  if ($::form->{elster} eq '1') {
     $format .=
         qq|<option value=elsterwinston>|
-      . $locale->text('ELSTER Export (Winston)')
+      . $::locale->text('ELSTER Export (Winston)')
       . qq|</option>|
       . qq|<option value=elstertaxbird>|
-      . $locale->text('ELSTER Export (Taxbird)')
+      . $::locale->text('ELSTER Export (Taxbird)')
       . qq|</option>|;
   }
 
@@ -522,22 +526,26 @@ sub show_options {
     $type
     $media
     <select name=format title = "|
-    . $locale->text('Choose Outputformat') . qq|">$format</select>
+    . $::locale->text('Choose Outputformat') . qq|">$format</select>
   |;
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 
   return $show_options;
 }
 
 sub generate_ustva {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
+
+  my $form     = $::form;
+  my $locale   = $::locale;
+  my %myconfig = %::myconfig;
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
   # Aufruf von get_config zum Einlesen der Finanzamtdaten aus finanzamt.ini
 
   my $ustva = USTVA->new();
-  $ustva->get_config($userspath, 'finanzamt.ini');
+  $ustva->get_config($::userspath, 'finanzamt.ini');
 
   # init some form vars
   my @anmeldungszeitraum =
@@ -558,7 +566,7 @@ sub generate_ustva {
                                     $form->current_date(\%myconfig), \%myconfig
                              ),
                              0, 4);
-      $lxdebug->message(LXDebug::DEBUG1,
+      $::lxdebug->message(LXDebug->DEBUG1,
                         qq|Actual year from Database: $form->{year}\n|);
     }
 
@@ -691,13 +699,13 @@ sub generate_ustva {
   # if there are any dates construct a where
   if ($form->{fromdate} || $form->{todate}) {
 
-    $form->{todate} = $form->current_date($myconfig)  unless ($form->{todate});
+    $form->{todate} = $form->current_date(\%myconfig)  unless ($form->{todate});
 
-    my $longtodate  = $locale->date($myconfig, $form->{todate}, 1, 0, 0);
-    my $shorttodate = $locale->date($myconfig, $form->{todate}, 0, 0, 0);
+    my $longtodate  = $locale->date(\%myconfig, $form->{todate}, 1, 0, 0);
+    my $shorttodate = $locale->date(\%myconfig, $form->{todate}, 0, 0, 0);
 
-    my $longfromdate  = $locale->date($myconfig, $form->{fromdate}, 1, 0, 0);
-    my $shortfromdate = $locale->date($myconfig, $form->{fromdate}, 0, 0, 0);
+    my $longfromdate  = $locale->date(\%myconfig, $form->{fromdate}, 1, 0, 0);
+    my $shortfromdate = $locale->date(\%myconfig, $form->{fromdate}, 0, 0, 0);
 
     $form->{this_period} = "$shortfromdate<br>\n$shorttodate";
     $form->{longperiod}      =
@@ -811,11 +819,11 @@ sub generate_ustva {
       $form->{br}      = "<br>";
       $form->{address} =~ s/\\n/\n/g;
 
-      foreach $number (@category_cent) {
+      foreach my $number (@category_cent) {
         $form->{$number} = $form->format_amount(\%myconfig, $form->{$number}, '2', '0');
       }
 
-      foreach $number (@category_euro) {
+      foreach my $number (@category_euro) {
         $form->{$number} = $form->format_amount(\%myconfig, $form->{$number}, '0', '0');
       }
 
@@ -839,7 +847,7 @@ sub generate_ustva {
       #file suffix
       $file .= '.xml';
       $file =~ s|.*/||;
-      $form->{tmpfile} = "$userspath/$file";
+      $form->{tmpfile} = "$::userspath/$file";
 
       $form->{attachment_filename} = $file;
 
@@ -889,7 +897,7 @@ sub generate_ustva {
       . sprintf("%02d", $form->{year} % 100) . ".txb";
 
       $form->{attachment_filename} =~ s|.*/||;
-      $form->{tmpfile} = "$userspath/" . $form->{attachment_filename};
+      $form->{tmpfile} = "$::userspath/" . $form->{attachment_filename};
 
       # TODO: set Output to UTF-8 or system Preference
       #$form->{"iconv"} = Text::Iconv->new($myconfig{dbcharset}, "UTF-8");
@@ -909,7 +917,7 @@ sub generate_ustva {
       } elsif ($form->{period} =~ /^\d+$/ ) {
         $form->{period} =~ s/^0//g;
         my $period = $form->{period};
-        $period * 1;
+        $period *= 1;
         $period--;
         $form->{taxbird_period} = $period;
       } else {
@@ -1037,8 +1045,8 @@ sub generate_ustva {
         $rec_ref->{id} = $kennziffer;
         $rec_ref->{amount} = $form->format_amount(\%myconfig, $form->{$kennziffer}, 2, '0');
 
-        $lxdebug->message($LXDebug::DEBUG, "Kennziffer $kennziffer: '$form->{$kennziffer}'" );
-        $lxdebug->dump($LXDebug::DEBUG, $rec_ref );
+        $::lxdebug->message($LXDebug::DEBUG, "Kennziffer $kennziffer: '$form->{$kennziffer}'" );
+        $::lxdebug->dump($LXDebug::DEBUG, $rec_ref );
         push @ { $form->{USTVA} }, $rec_ref;
       }
 
@@ -1061,7 +1069,7 @@ sub generate_ustva {
 
     $form->header();
 
-    $template_ref = {
+    my $template_ref = {
         taxnumber => $myconfig{taxnumber},
     };
 
@@ -1070,46 +1078,46 @@ sub generate_ustva {
   } else
   {
 
-    $form->parse_template(\%myconfig, $userspath);
+    $form->parse_template(\%myconfig, $::userspath);
 
   }
 
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 sub config_step1 {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
-$form->{title} = $locale->text('Tax Office Preferences');
+$::form->{title} = $::locale->text('Tax Office Preferences');
 
   # edit all taxauthority prefs
 
-  $form->header;
+  $::form->header;
 
   my $ustva = USTVA->new();
-  $ustva->get_config($userspath, 'finanzamt.ini');
+  $ustva->get_config($::userspath, 'finanzamt.ini');
 
-  my $land = $form->{elsterland};
-  my $amt  = $form->{elsterFFFF};
+  my $land = $::form->{elsterland};
+  my $amt  = $::form->{elsterFFFF};
 
 
-  $form->{title} = $locale->text('Tax Office Preferences');
+  $::form->{title} = $::locale->text('Tax Office Preferences');
 
 
-  my $select_tax_office = $ustva->fa_auswahl($land, $amt, $ustva->query_finanzamt(\%myconfig, $form));
-  my $checked_accrual = q|checked="checked"| if ($form->{method} eq 'accrual');
-  my $checked_cash = q|checked="checked"| if ($form->{method} eq 'cash');
-  my $checked_monthly = "checked" if ($form->{FA_voranmeld} eq 'month');
-  my $checked_quarterly = "checked" if ($form->{FA_voranmeld} eq 'quarter');
-  my $checked_dauerfristverlaengerung = "checked" if ($form->{FA_dauerfrist} eq '1');
-  my $checked_kz_71 = "checked" if ($form->{FA_71} eq 'X');
+  my $select_tax_office = $ustva->fa_auswahl($land, $amt, $ustva->query_finanzamt(\%::myconfig, $::form));
+  my $checked_accrual = q|checked="checked"| if ($::form->{method} eq 'accrual');
+  my $checked_cash = q|checked="checked"| if ($::form->{method} eq 'cash');
+  my $checked_monthly = "checked" if ($::form->{FA_voranmeld} eq 'month');
+  my $checked_quarterly = "checked" if ($::form->{FA_voranmeld} eq 'quarter');
+  my $checked_dauerfristverlaengerung = "checked" if ($::form->{FA_dauerfrist} eq '1');
+  my $checked_kz_71 = "checked" if ($::form->{FA_71} eq 'X');
 
   my $_hidden_variables_ref;
 
   my %_hidden_local_variables = (
-    'saved'       => $locale->text('Check Details'),
+    'saved'       => $::locale->text('Check Details'),
     'nextsub'     => 'config_step2',
     'warnung'     => '0',
   );
@@ -1132,12 +1140,12 @@ $form->{title} = $locale->text('Tax Office Preferences');
 
   foreach my $variable (@_hidden_form_variables) {
     push @{ $_hidden_variables_ref},
-        { 'variable' => $variable, 'value' => $form->{$variable} };
+        { 'variable' => $variable, 'value' => $::form->{$variable} };
   }
 
 # Which COA is in use?
 
-  $ustva->get_coa($form, \%myconfig);
+  $ustva->get_coa($::form, \%::myconfig);
 
   # hä? kann die weg?
   my $steuernummer_new = '';
@@ -1155,15 +1163,19 @@ $form->{title} = $locale->text('Tax Office Preferences');
   };
 
   # Ausgabe des Templates
-  print($form->parse_html_template('ustva/config_step1', $template_ref));
+  print($::form->parse_html_template('ustva/config_step1', $template_ref));
 
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 sub config_step2 {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
+
+  my $form     = $::form;
+  my $locale   = $::locale;
+  my %myconfig = %::myconfig;
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
   $form->header();
 
@@ -1177,7 +1189,7 @@ sub config_step2 {
   my $elstersteuernummer = '';
 
   my $ustva = USTVA->new();
-  $ustva->get_config($userspath, 'finanzamt.ini')
+  $ustva->get_config($::userspath, 'finanzamt.ini')
     if ($form->{saved} eq $locale->text('saved'));
 
   # Auf Übergabefehler checken
@@ -1242,7 +1254,7 @@ sub config_step2 {
   my $patterncount   = $form->{patterncount};
   my $elster_pattern = $form->{elster_pattern};
   my $delimiter      = $form->{delimiter};
-  my $steuernummer = $form->{steuernummer} if ($steuernummer eq '');
+  my $steuernummer = $form->{steuernummer} if ($stnr eq '');
 
   $form->{FA_Oeffnungszeiten} =~ s/\\\\n/\n/g;
 
@@ -1255,7 +1267,7 @@ sub config_step2 {
                              $form->{steuernummer}
   );
 
-  $lxdebug->message(LXDebug::DEBUG1, qq|$input_steuernummer|);
+  $::lxdebug->message(LXDebug->DEBUG1, qq|$input_steuernummer|);
 
 
   my $_hidden_variables_ref;
@@ -1263,7 +1275,7 @@ sub config_step2 {
   my %_hidden_local_variables = (
       'elsterland'          => $elsterland,
       'elsterFFFF'          => $elsterFFFF,
-      'warnung'             => $warnung,
+      'warnung'             => 0,
       'elstersteuernummer'  => $elstersteuernummer,
       'steuernummer'        => $stnr,
       'lastsub'             => 'config_step1',
@@ -1292,10 +1304,9 @@ sub config_step2 {
   }
 
   my $template_ref = {
-     tax_office_data                 => $tax_office_data,
      input_steuernummer              => $input_steuernummer,
      readonly                        => '', #q|disabled="disabled"|,
-     callback                        => $callback,
+     callback                        => $form->{callback},
      hidden_variables                => $_hidden_variables_ref,
   };
 
@@ -1303,18 +1314,18 @@ sub config_step2 {
   print($form->parse_html_template('ustva/config_step2', $template_ref));
 
 
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 sub create_steuernummer {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
-  my $part           = $form->{part};
-  my $patterncount   = $form->{patterncount};
-  my $delimiter      = $form->{delimiter};
-  my $elster_pattern = $form->{elster_pattern};
+  my $part           = $::form->{part};
+  my $patterncount   = $::form->{patterncount};
+  my $delimiter      = $::form->{delimiter};
+  my $elster_pattern = $::form->{elster_pattern};
 
   # rebuild steuernummer and elstersteuernummer
   # es gibt eine gespeicherte steuernummer $form->{steuernummer}
@@ -1324,33 +1335,33 @@ sub create_steuernummer {
   my $i = 0;
 
   my $steuernummer_new       = $part;
-  my $elstersteuernummer_new = $form->{elster_FFFF};
+  my $elstersteuernummer_new = $::form->{elster_FFFF};
   $elstersteuernummer_new .= '0';
 
   for ($h = 1; $h < $patterncount; $h++) {
     $steuernummer_new .= qq|$delimiter|;
     for (my $i = 1; $i <= length($elster_pattern); $i++) {
-      $steuernummer_new       .= $form->{"part_$h\_$i"};
-      $elstersteuernummer_new .= $form->{"part_$h\_$i"};
+      $steuernummer_new       .= $::form->{"part_$h\_$i"};
+      $elstersteuernummer_new .= $::form->{"part_$h\_$i"};
     }
   }
-  if ($form->{steuernummer} ne $steuernummer_new) {
-    $form->{steuernummer}       = $steuernummer_new;
-    $form->{elstersteuernummer} = $elstersteuernummer_new;
-    $form->{steuernummer_new}   = $steuernummer_new;
+  if ($::form->{steuernummer} ne $steuernummer_new) {
+    $::form->{steuernummer}       = $steuernummer_new;
+    $::form->{elstersteuernummer} = $elstersteuernummer_new;
+    $::form->{steuernummer_new}   = $steuernummer_new;
   } else {
-    $form->{steuernummer_new}       = '';
-    $form->{elstersteuernummer_new} = '';
+    $::form->{steuernummer_new}       = '';
+    $::form->{elstersteuernummer_new} = '';
   }
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 sub save {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
-  $auth->assert('advance_turnover_tax_return');
+  $::auth->assert('advance_turnover_tax_return');
 
-  my $filename = "$form->{login}_$form->{filename}";
+  my $filename = "$::form->{login}_$::form->{filename}";
   $filename =~ s|.*/||;
 
   #zuerst die steuernummer aus den part, parts_X_Y und delimiter herstellen
@@ -1358,11 +1369,11 @@ sub save {
 
   # Textboxen formatieren: Linebreaks entfernen
   #
-  $form->{FA_Oeffnungszeiten} =~ s/\r\n/\\n/g;
+  $::form->{FA_Oeffnungszeiten} =~ s/\r\n/\\n/g;
 
   #URL mit http:// davor?
-  $form->{FA_Internet} =~ s/^http:\/\///;
-  $form->{FA_Internet} = 'http://' . $form->{FA_Internet};
+  $::form->{FA_Internet} =~ s/^http:\/\///;
+  $::form->{FA_Internet} = 'http://' . $::form->{FA_Internet};
 
   my @config = qw(
     elster              elsterland            elstersteuernummer  steuernummer
@@ -1376,55 +1387,53 @@ sub save {
     FA_71 FA_dauerfrist);
 
   # Hier kommt dann die Plausibilitätsprüfung der ELSTERSteuernummer
-  if ($form->{elstersteuernummer} ne '000000000') {
+  if ($::form->{elstersteuernummer} ne '000000000') {
 
-    $form->{elster} = '1';
+    $::form->{elster} = '1';
 
-    open my $ustvaconfig, ">", "$userspath/$filename" or $form->error("$filename : $!");
+    open my $ustvaconfig, ">", "$::userspath/$filename" or $::form->error("$filename : $!");
 
     # create the config file
     print {$ustvaconfig} qq|# Configuration file for USTVA\n\n|;
     my $key = '';
     foreach $key (sort @config) {
-      $form->{$key} =~ s/\\/\\\\/g;
+      $::form->{$key} =~ s/\\/\\\\/g;
       # strip M
-      $form->{$key} =~ s/\r\n/\n/g;
+      $::form->{$key} =~ s/\r\n/\n/g;
 
       print {$ustvaconfig} qq|$key=|;
-      if ($form->{$key} ne 'Y') {
-        print {$ustvaconfig} qq|$form->{$key}\n|;
+      if ($::form->{$key} ne 'Y') {
+        print {$ustvaconfig} qq|$::form->{$key}\n|;
       }
-      if ($form->{$key} eq 'Y') {
+      if ($::form->{$key} eq 'Y') {
         print {$ustvaconfig} qq|checked \n|;
       }
     }
     print {$ustvaconfig} qq|\n\n|;
     close $ustvaconfig;
-    $form->{saved} = $locale->text('saved');
+    $::form->{saved} = $::locale->text('saved');
 
   } else {
 
-    $form->{saved} = $locale->text('Choose a Tax Number');
+    $::form->{saved} = $::locale->text('Choose a Tax Number');
   }
 
   config_step2();
-  $lxdebug->leave_sub();
+  $::lxdebug->leave_sub();
 }
 
 
 sub continue {
-  $lxdebug->enter_sub();
+  $::lxdebug->enter_sub();
 
   # allow Symbolic references just here:
-  no strict 'refs';
-  call_sub($form->{"nextsub"});
-  use strict 'refs';
-  $lxdebug->leave_sub();
+  call_sub($::form->{"nextsub"});
+  $::lxdebug->leave_sub();
 }
 
 sub back {
-  $lxdebug->enter_sub();
-  call_sub($form->{"lastsub"});
-  $lxdebug->leave_sub();
+  $::lxdebug->enter_sub();
+  call_sub($::form->{"lastsub"});
+  $::lxdebug->leave_sub();
 }
 
index a380138..1251b53 100644 (file)
@@ -2,386 +2,390 @@ package List::MoreUtils;
 
 use 5.00503;
 use strict;
+use Exporter   ();
+use DynaLoader ();
+
+use vars qw{ $VERSION @ISA @EXPORT_OK %EXPORT_TAGS };
+BEGIN {
+    $VERSION   = '0.30';
+    @ISA       = qw{ Exporter DynaLoader };
+    @EXPORT_OK = qw{
+        any all none notall true false
+        firstidx first_index lastidx last_index
+        insert_after insert_after_string
+        apply indexes
+        after after_incl before before_incl
+        firstval first_value lastval last_value
+        each_array each_arrayref
+        pairwise natatime
+        mesh zip uniq distinct
+        minmax part
+    };
+    %EXPORT_TAGS = (
+        all => \@EXPORT_OK,
+    );
 
-require Exporter;
-require DynaLoader;
-
+    # Load the XS at compile-time so that redefinition warnings will be
+    # thrown correctly if the XS versions of part or indexes loaded
+    eval {
+        # PERL_DL_NONLAZY must be false, or any errors in loading will just
+        # cause the perl code to be tested
+        local $ENV{PERL_DL_NONLAZY} = 0 if $ENV{PERL_DL_NONLAZY};
 
-use vars qw($VERSION @ISA @EXPORT_OK %EXPORT_TAGS);
-@ISA = qw(Exporter DynaLoader);
+        bootstrap List::MoreUtils $VERSION;
+        1;
 
-%EXPORT_TAGS = ( 
-    all => [ qw(any all none notall true false firstidx first_index lastidx
-               last_index insert_after insert_after_string apply after after_incl before
-               before_incl indexes firstval first_value lastval last_value each_array
-               each_arrayref pairwise natatime mesh zip uniq minmax part bsearch) ],
-);
+    } unless $ENV{LIST_MOREUTILS_PP};
+}
 
-@EXPORT_OK = ( @{ $EXPORT_TAGS{'all'} } );
+# Always use Perl apply() until memory leaks are resolved.
+sub apply (&@) {
+    my $action = shift;
+    &$action foreach my @values = @_;
+    wantarray ? @values : $values[-1];
+}
 
-$VERSION = '0.25_02';
+# Always use Perl part() until memory leaks are resolved.
+sub part (&@) {
+    my ($code, @list) = @_;
+    my @parts;
+    push @{ $parts[ $code->($_) ] }, $_  foreach @list;
+    return @parts;
+}
 
-eval {
-    local $ENV{PERL_DL_NONLAZY} = 0 if $ENV{PERL_DL_NONLAZY};
-    bootstrap List::MoreUtils $VERSION;
-    1;
-} if not $ENV{LIST_MOREUTILS_PP};
+# Always use Perl indexes() until memory leaks are resolved.
+sub indexes (&@) {
+    my $test = shift;
+    grep {
+        local *_ = \$_[$_];
+        $test->()
+    } 0 .. $#_;
+}
 
-eval <<'EOP' if not defined &any;
+# Load the pure-Perl versions of the other functions if needed
+eval <<'END_PERL' unless defined &any;
 
-require POSIX;
+# Use pure scalar boolean return values for compatibility with XS
+use constant YES => ! 0;
+use constant NO  => ! 1;
 
 sub any (&@) {
     my $f = shift;
-    return if ! @_;
-    for (@_) {
-       return 1 if $f->();
+    foreach ( @_ ) {
+        return YES if $f->();
     }
-    return 0;
+    return NO;
 }
-    
+
 sub all (&@) {
     my $f = shift;
-    return if ! @_;
-    for (@_) {
-       return 0 if ! $f->();
+    foreach ( @_ ) {
+        return NO unless $f->();
     }
-    return 1;
+    return YES;
 }
 
 sub none (&@) {
     my $f = shift;
-    return 1 if ! @_;
-    for (@_) {
-       return 0 if $f->();
+    foreach ( @_ ) {
+        return NO if $f->();
     }
-    return 1;
+    return YES;
 }
 
 sub notall (&@) {
     my $f = shift;
-    return if ! @_;
-    for (@_) {
-       return 1 if ! $f->();
+    foreach ( @_ ) {
+        return YES unless $f->();
     }
-    return 0;
+    return NO;
 }
 
 sub true (&@) {
-    my $f = shift;
+    my $f     = shift;
     my $count = 0;
-    for (@_) {
-       $count++ if $f->();
+    foreach ( @_ ) {
+        $count++ if $f->();
     }
     return $count;
 }
 
 sub false (&@) {
-    my $f = shift;
+    my $f     = shift;
     my $count = 0;
-    for (@_) {
-       $count++ if ! $f->();
+    foreach ( @_ ) {
+        $count++ unless $f->();
     }
     return $count;
 }
 
 sub firstidx (&@) {
     my $f = shift;
-    for my $i (0 .. $#_) {
-       local *_ = \$_[$i];     
-       return $i if $f->();
+    foreach my $i ( 0 .. $#_ ) {
+        local *_ = \$_[$i];
+        return $i if $f->();
     }
     return -1;
 }
 
 sub lastidx (&@) {
     my $f = shift;
-    for my $i (reverse 0 .. $#_) {
-       local *_ = \$_[$i];
-       return $i if $f->();
+    foreach my $i ( reverse 0 .. $#_ ) {
+        local *_ = \$_[$i];
+        return $i if $f->();
     }
     return -1;
 }
 
 sub insert_after (&$\@) {
-    my ($code, $val, $list) = @_;
+    my ($f, $val, $list) = @_;
     my $c = -1;
     local *_;
-    for my $i (0 .. $#$list) {
-       $_ = $list->[$i];
-       $c = $i, last if $code->();
+    foreach my $i ( 0 .. $#$list ) {
+        $_ = $list->[$i];
+        $c = $i, last if $f->();
     }
-    @$list = (@{$list}[0..$c], $val, @{$list}[$c+1..$#$list]) and return 1 if $c != -1;
+    @$list = (
+        @{$list}[ 0 .. $c ],
+        $val,
+        @{$list}[ $c + 1 .. $#$list ],
+    ) and return 1 if $c != -1;
     return 0;
 }
 
 sub insert_after_string ($$\@) {
     my ($string, $val, $list) = @_;
     my $c = -1;
-    for my $i (0 .. $#$list) {
-       local $^W = 0;
-       $c = $i, last if $string eq $list->[$i];
+    foreach my $i ( 0 .. $#$list ) {
+        local $^W = 0;
+        $c = $i, last if $string eq $list->[$i];
     }
-    @$list = (@{$list}[0..$c], $val, @{$list}[$c+1..$#$list]) and return 1 if $c != -1;
+    @$list = (
+        @{$list}[ 0 .. $c ],
+        $val,
+        @{$list}[ $c + 1 .. $#$list ],
+    ) and return 1 if $c != -1;
     return 0;
 }
 
-sub apply (&@) {
-    my $action = shift;
-    &$action for my @values = @_;
-    wantarray ? @values : $values[-1];
-}
-
-sub after (&@)
-{
+sub after (&@) {
     my $test = shift;
     my $started;
     my $lag;
-    grep $started ||= do { my $x=$lag; $lag=$test->(); $x},  @_;
+    grep $started ||= do {
+        my $x = $lag;
+        $lag = $test->();
+        $x
+    }, @_;
 }
 
-sub after_incl (&@)
-{
+sub after_incl (&@) {
     my $test = shift;
     my $started;
     grep $started ||= $test->(), @_;
 }
 
-sub before (&@)
-{
+sub before (&@) {
     my $test = shift;
-    my $keepgoing=1;
-    grep $keepgoing &&= !$test->(),  @_;
+    my $more = 1;
+    grep $more &&= ! $test->(), @_;
 }
 
-sub before_incl (&@)
-{
+sub before_incl (&@) {
     my $test = shift;
-    my $keepgoing=1;
-    my $lag=1;
-    grep $keepgoing &&= do { my $x=$lag; $lag=!$test->(); $x},  @_;
+    my $more = 1;
+    my $lag  = 1;
+    grep $more &&= do {
+        my $x = $lag;
+        $lag = ! $test->();
+        $x
+    }, @_;
 }
 
-sub indexes (&@)
-{
-    my $test = shift;
-    grep {local *_=\$_[$_]; $test->()} 0..$#_;
-}
-
-sub lastval (&@)
-{
+sub lastval (&@) {
     my $test = shift;
     my $ix;
-    for ($ix=$#_; $ix>=0; $ix--)
-    {
+    for ( $ix = $#_; $ix >= 0; $ix-- ) {
         local *_ = \$_[$ix];
         my $testval = $test->();
-        $_[$ix] = $_;    # simulate $_ as alias
+
+        # Simulate $_ as alias
+        $_[$ix] = $_;
         return $_ if $testval;
     }
     return undef;
 }
 
-sub firstval (&@)
-{
+sub firstval (&@) {
     my $test = shift;
-    foreach (@_)
-    {
+    foreach ( @_ ) {
         return $_ if $test->();
     }
     return undef;
 }
 
-sub pairwise(&\@\@)
-{
+sub pairwise (&\@\@) {
     my $op = shift;
-    use vars qw/@A @B/;
-    local (*A, *B) = @_;    # syms for caller's input arrays
+
+    # Symbols for caller's input arrays
+    use vars qw{ @A @B };
+    local ( *A, *B ) = @_;
 
     # Localise $a, $b
-    my ($caller_a, $caller_b) = do
-    {
+    my ( $caller_a, $caller_b ) = do {
         my $pkg = caller();
         no strict 'refs';
         \*{$pkg.'::a'}, \*{$pkg.'::b'};
     };
 
-    my $limit = $#A > $#B? $#A : $#B;    # loop iteration limit
+    # Loop iteration limit
+    my $limit = $#A > $#B? $#A : $#B;
 
-    local(*$caller_a, *$caller_b);
-    map    # This map expression is also the return value.
-    {
-        # assign to $a, $b as refs to caller's array elements
-        (*$caller_a, *$caller_b) = \($A[$_], $B[$_]);
-        $op->();    # perform the transformation
+    # This map expression is also the return value
+    local( *$caller_a, *$caller_b );
+    map {
+        # Assign to $a, $b as refs to caller's array elements
+        ( *$caller_a, *$caller_b ) = \( $A[$_], $B[$_] );
+
+        # Perform the transformation
+        $op->();
     }  0 .. $limit;
 }
 
-sub each_array (\@;\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@)
-{
+sub each_array (\@;\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@) {
     return each_arrayref(@_);
 }
 
-sub each_arrayref
-{
-    my @arr_list  = @_;     # The list of references to the arrays
-    my $index     = 0;      # Which one the caller will get next
-    my $max_num   = 0;      # Number of elements in longest array
+sub each_arrayref {
+    my @list  = @_; # The list of references to the arrays
+    my $index = 0;  # Which one the caller will get next
+    my $max   = 0;  # Number of elements in longest array
 
     # Get the length of the longest input array
-    foreach (@arr_list)
-    {
-        unless (ref($_) eq 'ARRAY')
-        {
+    foreach ( @list ) {
+        unless ( ref $_ eq 'ARRAY' ) {
             require Carp;
-            Carp::croak "each_arrayref: argument is not an array reference\n";
+            Carp::croak("each_arrayref: argument is not an array reference\n");
         }
-        $max_num = @$_  if @$_ > $max_num;
+        $max = @$_ if @$_ > $max;
     }
 
     # Return the iterator as a closure wrt the above variables.
-    return sub
-    {
-        if (@_)
-        {
+    return sub {
+        if ( @_ ) {
             my $method = shift;
-            if ($method eq 'index')
-            {
-                # Return current (last fetched) index
-                return undef if $index == 0  ||  $index > $max_num;
-                return $index-1;
-            }
-            else
-            {
+            unless ( $method eq 'index' ) {
                 require Carp;
-                Carp::croak "each_array: unknown argument '$method' passed to iterator.";
+                Carp::croak("each_array: unknown argument '$method' passed to iterator.");
             }
+
+            # Return current (last fetched) index
+            return undef if $index == 0  ||  $index > $max;
+            return $index - 1;
         }
 
-        return if $index >= $max_num;     # No more elements to return
+        # No more elements to return
+        return if $index >= $max;
         my $i = $index++;
-        return map $_->[$i], @arr_list;   # Return ith elements
+
+        # Return ith elements
+        return map $_->[$i], @list; 
     }
 }
 
-sub natatime ($@)
-{
-    my $n = shift;
+sub natatime ($@) {
+    my $n    = shift;
     my @list = @_;
-
-    return sub
-    {
+    return sub {
         return splice @list, 0, $n;
     }
 }
 
 sub mesh (\@\@;\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@\@) {
     my $max = -1;
-    $max < $#$_  &&  ($max = $#$_)  for @_;
-
-    map { my $ix = $_; map $_->[$ix], @_; } 0..$max; 
+    $max < $#$_ && ( $max = $#$_ ) foreach @_;
+    map {
+        my $ix = $_;
+        map $_->[$ix], @_;
+    } 0 .. $max; 
 }
 
 sub uniq (@) {
-    my %h;
-    my $ref = \1;
-    map { $h{defined $_ ? $_ : $ref}++ == 0 ? $_ : () } @_;
+    my %seen = ();
+    grep { not $seen{$_}++ } @_;
 }
 
 sub minmax (@) {
-    return if ! @_;
+    return unless @_;
     my $min = my $max = $_[0];
 
-    for (my $i = 1; $i < @_; $i += 2) {
-       if ($_[$i-1] <= $_[$i]) {
-           $min = $_[$i-1] if $min > $_[$i-1];
-           $max = $_[$i]   if $max < $_[$i];
-       } else {
-           $min = $_[$i]   if $min > $_[$i];
-           $max = $_[$i-1] if $max < $_[$i-1];
-       }
-    }
-
-    if (@_ & 1) {
-       my $i = $#_;
-       if ($_[$i-1] <= $_[$i]) {
-           $min = $_[$i-1] if $min > $_[$i-1];
-           $max = $_[$i]   if $max < $_[$i];
-       } else {
-           $min = $_[$i]   if $min > $_[$i];
-           $max = $_[$i-1] if $max < $_[$i-1];
-       }
+    for ( my $i = 1; $i < @_; $i += 2 ) {
+        if ( $_[$i-1] <= $_[$i] ) {
+            $min = $_[$i-1] if $min > $_[$i-1];
+            $max = $_[$i]   if $max < $_[$i];
+        } else {
+            $min = $_[$i]   if $min > $_[$i];
+            $max = $_[$i-1] if $max < $_[$i-1];
+        }
     }
 
-    return ($min, $max);
-}
-
-sub part(&@) {
-    my ($code, @list) = @_;
-    my @parts;
-    push @{ $parts[$code->($_)] }, $_  for @list;
-    return @parts;
-}
-
-sub bsearch(&@) {
-    my $code = shift;
-
-    my $rc;
-    my $i = 0;
-    my $j = @_;
-    do {
-        my $k = int(($i + $j) / 2);
-
-        return if $k >= @_;
-
-        local *_ = \$_[$k];
-        $rc = $code->();
-
-        $rc == 0 and
-            return wantarray ? $_ : 1;
-
-        if ($rc < 0) {
-            $i = $k + 1;
+    if ( @_ & 1 ) {
+        my $i = $#_;
+        if ($_[$i-1] <= $_[$i]) {
+            $min = $_[$i-1] if $min > $_[$i-1];
+            $max = $_[$i]   if $max < $_[$i];
         } else {
-            $j = $k - 1;
+            $min = $_[$i]   if $min > $_[$i];
+            $max = $_[$i-1] if $max < $_[$i-1];
         }
-    } until $i > $j;
+    }
 
-    return;
+    return ($min, $max);
 }
 
 sub _XScompiled {
     return 0;
 }
 
-EOP
+END_PERL
 die $@ if $@;
 
+# Function aliases
 *first_index = \&firstidx;
-*last_index = \&lastidx;
+*last_index  = \&lastidx;
 *first_value = \&firstval;
-*last_value = \&lastval;
-*zip = \&mesh;
+*last_value  = \&lastval;
+*zip         = \&mesh;
+*distinct    = \&uniq;
 
 1;
+
 __END__
 
+=pod
+
 =head1 NAME
 
 List::MoreUtils - Provide the stuff missing in List::Util
 
 =head1 SYNOPSIS
 
-    use List::MoreUtils qw(any all none notall true false firstidx first_index 
-                           lastidx last_index insert_after insert_after_string 
-                           apply after after_incl before before_incl indexes 
-                           firstval first_value lastval last_value each_array
-                           each_arrayref pairwise natatime mesh zip uniq minmax);
+    use List::MoreUtils qw{
+        any all none notall true false
+        firstidx first_index lastidx last_index
+        insert_after insert_after_string
+        apply indexes
+        after after_incl before before_incl
+        firstval first_value lastval last_value
+        each_array each_arrayref
+        pairwise natatime
+        mesh zip uniq distinct minmax part
+    };
 
 =head1 DESCRIPTION
 
-C<List::MoreUtils> provides some trivial but commonly needed functionality on lists
-which is not going to go into C<List::Util>.
+B<List::MoreUtils> provides some trivial but commonly needed functionality on
+lists which is not going to go into L<List::Util>.
 
 All of the below functions are implementable in only a couple of lines of Perl
 code. Using the functions from this module however should give slightly better
@@ -397,9 +401,9 @@ Returns a true value if any item in LIST meets the criterion given through
 BLOCK. Sets C<$_> for each item in LIST in turn:
 
     print "At least one value undefined"
-        if any { !defined($_) } @list;
+        if any { ! defined($_) } @list;
 
-Returns false otherwise, or C<undef> if LIST is empty.
+Returns false otherwise, or if LIST is empty.
 
 =item all BLOCK LIST
 
@@ -409,39 +413,40 @@ BLOCK. Sets C<$_> for each item in LIST in turn:
     print "All items defined"
         if all { defined($_) } @list;
 
-Returns false otherwise, or C<undef> if LIST is empty.
+Returns false otherwise, or if LIST is empty.
 
 =item none BLOCK LIST
 
-Logically the negation of C<any>. Returns a true value if no item in LIST meets the
-criterion given through BLOCK. Sets C<$_> for each item in LIST in turn:
+Logically the negation of C<any>. Returns a true value if no item in LIST meets
+the criterion given through BLOCK. Sets C<$_> for each item in LIST in turn:
 
     print "No value defined"
         if none { defined($_) } @list;
 
-Returns false otherwise, or C<undef> if LIST is empty.
+Returns false otherwise, or if LIST is empty.
 
 =item notall BLOCK LIST
 
-Logically the negation of C<all>. Returns a true value if not all items in LIST meet
-the criterion given through BLOCK. Sets C<$_> for each item in LIST in turn:
+Logically the negation of C<all>. Returns a true value if not all items in LIST
+meet the criterion given through BLOCK. Sets C<$_> for each item in LIST in
+turn:
 
     print "Not all values defined"
         if notall { defined($_) } @list;
 
-Returns false otherwise, or C<undef> if LIST is empty.
+Returns false otherwise, or if LIST is empty.
 
 =item true BLOCK LIST
 
-Counts the number of elements in LIST for which the criterion in BLOCK is true. Sets C<$_> for 
-each item in LIST in turn:
+Counts the number of elements in LIST for which the criterion in BLOCK is true.
+Sets C<$_> for  each item in LIST in turn:
 
     printf "%i item(s) are defined", true { defined($_) } @list;
 
 =item false BLOCK LIST
 
-Counts the number of elements in LIST for which the criterion in BLOCK is false. Sets C<$_> for
-each item in LIST in turn:
+Counts the number of elements in LIST for which the criterion in BLOCK is false.
+Sets C<$_> for each item in LIST in turn:
 
     printf "%i item(s) are not defined", false { defined($_) } @list;
 
@@ -449,8 +454,8 @@ each item in LIST in turn:
 
 =item first_index BLOCK LIST
 
-Returns the index of the first element in LIST for which the criterion in BLOCK is true. Sets C<$_>
-for each item in LIST in turn:
+Returns the index of the first element in LIST for which the criterion in BLOCK
+is true. Sets C<$_> for each item in LIST in turn:
 
     my @list = (1, 4, 3, 2, 4, 6);
     printf "item with index %i in list is 4", firstidx { $_ == 4 } @list;
@@ -465,8 +470,8 @@ C<first_index> is an alias for C<firstidx>.
 
 =item last_index BLOCK LIST
 
-Returns the index of the last element in LIST for which the criterion in BLOCK is true. Sets C<$_>
-for each item in LIST in turn:
+Returns the index of the last element in LIST for which the criterion in BLOCK
+is true. Sets C<$_> for each item in LIST in turn:
 
     my @list = (1, 4, 3, 2, 4, 6);
     printf "item with index %i in list is 4", lastidx { $_ == 4 } @list;
@@ -479,8 +484,8 @@ C<last_index> is an alias for C<lastidx>.
 
 =item insert_after BLOCK VALUE LIST
 
-Inserts VALUE after the first item in LIST for which the criterion in BLOCK is true. Sets C<$_> for
-each item in LIST in turn.
+Inserts VALUE after the first item in LIST for which the criterion in BLOCK is
+true. Sets C<$_> for each item in LIST in turn.
 
     my @list = qw/This is a list/;
     insert_after { $_ eq "a" } "longer" => @list;
@@ -517,6 +522,15 @@ Think of it as syntactic sugar for
 
     for (my @mult = @list) { $_ *= 2 }
 
+=item before BLOCK LIST
+
+Returns a list of values of LIST upto (and not including) the point where BLOCK
+returns a true value. Sets C<$_> for each element in LIST in turn.
+
+=item before_incl BLOCK LIST
+
+Same as C<before> but also includes the element for which BLOCK is true.
+
 =item after BLOCK LIST
 
 Returns a list of the values of LIST after (and not including) the point
@@ -528,15 +542,6 @@ where BLOCK returns a true value. Sets C<$_> for each element in LIST in turn.
 
 Same as C<after> but also inclues the element for which BLOCK is true.
 
-=item before BLOCK LIST
-
-Returns a list of values of LIST upto (and not including) the point where BLOCK
-returns a true value. Sets C<$_> for each element in LIST in turn.
-
-=item before_incl BLOCK LIST
-
-Same as C<before> but also includes the element for which BLOCK is true.
-
 =item indexes BLOCK LIST
 
 Evaluates BLOCK for each element in LIST (assigned to C<$_>) and returns a list
@@ -646,6 +651,8 @@ C<zip> is an alias for C<mesh>.
 
 =item uniq LIST
 
+=item distinct LIST
+
 Returns a new list by stripping duplicate values in LIST. The order of
 elements in the returned list is the same as in LIST. In scalar context,
 returns the number of unique elements in LIST.
@@ -656,22 +663,23 @@ returns the number of unique elements in LIST.
 =item minmax LIST
 
 Calculates the minimum and maximum of LIST and returns a two element list with
-the first element being the minimum and the second the maximum. Returns the empty
-list if LIST was empty.
+the first element being the minimum and the second the maximum. Returns the
+empty list if LIST was empty.
 
-The minmax algorithm differs from a naive iteration over the list where each element
-is compared to two values being the so far calculated min and max value in that it
-only requires 3n/2 - 2 comparisons. Thus it is the most efficient possible algorithm.
+The C<minmax> algorithm differs from a naive iteration over the list where each
+element is compared to two values being the so far calculated min and max value
+in that it only requires 3n/2 - 2 comparisons. Thus it is the most efficient
+possible algorithm.
 
 However, the Perl implementation of it has some overhead simply due to the fact
 that there are more lines of Perl code involved. Therefore, LIST needs to be
-fairly big in order for minmax to win over a naive implementation. This
+fairly big in order for C<minmax> to win over a naive implementation. This
 limitation does not apply to the XS version.
 
 =item part BLOCK LIST
 
-Partitions LIST based on the return value of BLOCK which denotes into which partition
-the current value is put.
+Partitions LIST based on the return value of BLOCK which denotes into which
+partition the current value is put.
 
 Returns a list of the partitions thusly created. Each partition created is a
 reference to an array.
@@ -692,18 +700,9 @@ Be careful with negative values, though:
 
 Negative values are only ok when they refer to a partition previously created:
 
-    my @idx = (0, 1, -1);
-    my $i = 0;
-    my @part = part { $idx[$++ % 3] } 1 .. 8;  # [1, 4, 7], [2, 3, 5, 6, 8]
-
-=item bsearch BLOCK LIST
-
-Performs a binary search on LIST which must be a sorted list of values. BLOCK
-must return a negative value if the current element (stored in C<$_>) is smaller,
-a positive value if it is bigger and zero if it matches.
-
-Returns a boolean value in scalar context. In list context, it returns the element
-if it was found, otherwise the empty list.
+    my @idx  = ( 0, 1, -1 );
+    my $i    = 0;
+    my @part = part { $idx[$++ % 3] } 1 .. 8; # [1, 4, 7], [2, 3, 5, 6, 8]
 
 =back
 
@@ -711,11 +710,12 @@ if it was found, otherwise the empty list.
 
 Nothing by default. To import all of this module's symbols, do the conventional
 
-    use List::MoreUtils qw/:all/;
+    use List::MoreUtils ':all';
 
-It may make more sense though to only import the stuff your program actually needs:
+It may make more sense though to only import the stuff your program actually
+needs:
 
-    use List::MoreUtils qw/any firstidx/;
+    use List::MoreUtils qw{ any firstidx };
 
 =head1 ENVIRONMENT
 
@@ -725,16 +725,12 @@ there for the test-suite to force testing the Perl implementation, and possibly
 for reporting of bugs. I don't see any reason to use it in a production
 environment.
 
-=head1 VERSION
-
-This is version 0.25_01.
-
 =head1 BUGS
 
 There is a problem with a bug in 5.6.x perls. It is a syntax error to write
 things like:
 
-    my @x = apply { s/foo/bar/ } qw/foo bar baz/;
+    my @x = apply { s/foo/bar/ } qw{ foo bar baz };
 
 It has to be written as either
 
@@ -744,22 +740,28 @@ or
 
     my @x = apply { s/foo/bar/ } my @dummy = qw/foo bar baz/;
 
-Perl5.5.x and perl5.8.x don't suffer from this limitation.
+Perl 5.5.x and Perl 5.8.x don't suffer from this limitation.
 
 If you have a functionality that you could imagine being in this module, please
-drop me a line. This module's policy will be less strict than C<List::Util>'s when
-it comes to additions as it isn't a core module.
+drop me a line. This module's policy will be less strict than L<List::Util>'s
+when it comes to additions as it isn't a core module.
 
 When you report bugs, it would be nice if you could additionally give me the
 output of your program with the environment variable C<LIST_MOREUTILS_PP> set
 to a true value. That way I know where to look for the problem (in XS,
 pure-Perl or possibly both).
 
+=head1 SUPPORT
+
+Bugs should always be submitted via the CPAN bug tracker.
+
+L<http://rt.cpan.org/NoAuth/ReportBug.html?Queue=List-MoreUtils>
+
 =head1 THANKS
 
 Credits go to a number of people: Steve Purkis for giving me namespace advice
 and James Keenan and Terrence Branno for their effort of keeping the CPAN
-tidier by making List::Utils obsolete. 
+tidier by making L<List::Utils> obsolete. 
 
 Brian McCauley suggested the inclusion of apply() and provided the pure-Perl
 implementation for it.
@@ -788,11 +790,16 @@ XS-implementation of part() work.
 
 =head1 TODO
 
-A pile of requests from other people is still pending further processing in my
-mailbox. This includes:
+A pile of requests from other people is still pending further processing in
+my mailbox. This includes:
 
 =over 4
 
+=item * List::Util export pass-through
+
+Allow B<List::MoreUtils> to pass-through the regular L<List::Util>
+functions to end users only need to C<use> the one module.
+
 =item * uniq_by(&@)
 
 Use code-reference to extract a key based on which the uniqueness is
@@ -816,8 +823,8 @@ These were all suggested by Dan Muey.
 
 =item * listify
 
-Always return a flat list when either a simple scalar value was passed or an array-reference.
-Suggested by Mark Summersault.
+Always return a flat list when either a simple scalar value was passed or an
+array-reference. Suggested by Mark Summersault.
 
 =back
 
@@ -827,11 +834,11 @@ L<List::Util>
 
 =head1 AUTHOR
 
-Tassilo von Parseval, E<lt>vparseval@gmail.comE<gt>
+Tassilo von Parseval E<lt>tassilo.von.parseval@rwth-aachen.deE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright (C) 2004-2009 by Tassilo von Parseval
+Copyright 2004 - 2010 by Tassilo von Parseval
 
 This library is free software; you can redistribute it and/or modify
 it under the same terms as Perl itself, either Perl version 5.8.4 or,
diff --git a/t/structure/globals.t b/t/structure/globals.t
new file mode 100644 (file)
index 0000000..b1b351f
--- /dev/null
@@ -0,0 +1,63 @@
+#!/usr/bin/perl
+
+use strict;
+use lib 't';
+use Support::Files;
+
+my (@globals, $testcount);
+
+BEGIN {
+  @globals = qw(lxdebug auth myconfig form cgi lx_office_conf locale);
+  $testcount = scalar(@Support::Files::testitems);
+}
+
+use Test::More tests => $testcount;
+
+# Capture the TESTOUT from Test::More or Test::Builder for printing errors.
+# This will handle verbosity for us automatically.
+my $fh;
+{
+    local $^W = 0;  # Don't complain about non-existent filehandles
+    if (-e \*Test::More::TESTOUT) {
+        $fh = \*Test::More::TESTOUT;
+    } elsif (-e \*Test::Builder::TESTOUT) {
+        $fh = \*Test::Builder::TESTOUT;
+    } else {
+        $fh = \*STDOUT;
+    }
+}
+
+my @testitems = @Support::Files::testitems;
+
+# at last, here we actually run the test...
+my $evilwordsregexp = join('|', @globals);
+
+foreach my $file (@testitems) {
+    $file =~ s/\s.*$//; # nuke everything after the first space (#comment)
+    next if (!$file); # skip null entries
+
+    if (open (FILE, $file)) { # open the file for reading
+
+        my $found_word = '';
+
+        while (my $file_line = <FILE>) { # and go through the file line by line
+            if ($file_line =~ /([\$%@](?:main)?::(?!$evilwordsregexp)\w+\b)/i) { # found an evil word
+                $found_word = $1;
+                last;
+            }
+        }
+
+        close (FILE);
+
+        if ($found_word) {
+            ok(0,"$file: found UNREGISTERED GLOBAL $found_word --WARNING");
+        } else {
+            ok(1,"$file does only contain registered globals");
+        }
+    } else {
+        ok(0,"could not open $file for globals check --WARNING");
+    }
+}
+
+exit 0;
+