[Debian-l10n-commits] [SCM] Debian i18n - l10n development - dl10n robots branch, master, updated. b56e82943f9a78d15598e630d7881e7a19ae95fd

[Nicolas François]

The following commit has been merged in the master branch:
commit b56e82943f9a78d15598e630d7881e7a19ae95fd
Author: Nicolas François <nicolas.francois at centraliens.net>
Date:   Sun Jun 17 00:35:31 2012 +0200

    Fix warnings triggered by spam removed from web archive.
    
    * lib/Debian/L10n/Spider.pm: Do not parse subject when it could
    not be retrieved (e.g. spam removed from web archive).

diff --git a/Changelog b/Changelog
index 416c4d9..2088aee 100644
--- a/Changelog
+++ b/Changelog
@@ -1,5 +1,10 @@
 2012-06-17  Nicolas François  <nicolas.francois at centraliens.net>
 
+	* lib/Debian/L10n/Spider.pm: Do not parse subject when it could
+	not be retrieved (e.g. spam removed from web archive).
+
+2012-06-17  Nicolas François  <nicolas.francois at centraliens.net>
+
 	* cron/spiderinit.2006: Removed: use cron/spiderinit with FAST set
 	to 1 (default).
 
diff --git a/lib/Debian/L10n/Spider.pm b/lib/Debian/L10n/Spider.pm
index 3fc935f..5ac2645 100644
--- a/lib/Debian/L10n/Spider.pm
+++ b/lib/Debian/L10n/Spider.pm
@@ -251,8 +251,12 @@ sub spider($$$$$@) {
 
 				if (not defined ${$messages}{$key}) {
 				  my ($s) = grep(/Subject: / , @$header);
-				  ($status, $type, $bug_nb, @names) = Debian::L10n::Utils::parse_subject($s);
-				  print "Spider.pm: [$lang:$year/$month/$message] $s\n";
+				  if (defined $s) {
+				    ($status, $type, $bug_nb, @names) = Debian::L10n::Utils::parse_subject($s);
+				    print "Spider.pm: [$lang:$year/$month/$message] $s\n";
+				  } else {
+				    warn "Spider.pm: [$lang:$year/$month/$message] no subject: @$header\n";
+				  }
 				  next unless $status; 	# unparsable mail
 				}
 

-- 
Debian i18n - l10n development - dl10n robots