[Debian-l10n-commits] r1122 - in /dl10n/trunk: Changelog lib/Debian/L10n/Spider.pm

[nekral-guest at users.alioth.debian.org]

Author: nekral-guest
Date: Mon Aug  4 18:10:12 2008
New Revision: 1122

URL: http://svn.debian.org/wsvn/?sc=1&rev=1122
Log:
	* lib/Debian/L10n/Spider.pm: After looking at the BTS, only update
	the database if there were some changes. Differentiate changes
	which require adding a status line and the one which only produce
	an update (date & name fixes).

Modified:
    dl10n/trunk/Changelog
    dl10n/trunk/lib/Debian/L10n/Spider.pm

Modified: dl10n/trunk/Changelog
URL: http://svn.debian.org/wsvn/dl10n/trunk/Changelog?rev=1122&op=diff
==============================================================================
--- dl10n/trunk/Changelog (original)
+++ dl10n/trunk/Changelog Mon Aug  4 18:10:12 2008
@@ -1,3 +1,10 @@
+2008-08-04  Nicolas François  <nicolas.francois at centraliens.net>
+
+	* lib/Debian/L10n/Spider.pm: After looking at the BTS, only update
+	the database if there were some changes. Differentiate changes
+	which require adding a status line and the one which only produce
+	an update (date & name fixes).
+
 2008-08-04  Nicolas François  <nicolas.francois at centraliens.net>
 
 	* lib/Debian/L10n/Spider.pm: There has beenno issues so far with

Modified: dl10n/trunk/lib/Debian/L10n/Spider.pm
URL: http://svn.debian.org/wsvn/dl10n/trunk/lib/Debian/L10n/Spider.pm?rev=1122&op=diff
==============================================================================
--- dl10n/trunk/lib/Debian/L10n/Spider.pm (original)
+++ dl10n/trunk/lib/Debian/L10n/Spider.pm Mon Aug  4 18:10:12 2008
@@ -448,35 +448,41 @@
 			}
 
 			if ($closedate{$bug_nb}) {
-				$date = $closedate{$bug_nb};
+				if ($closedate{$bug_nb} ne $date) {
+					$date = $closedate{$bug_nb};
+					$changed = 1 unless $changed == 2;
+				}
 				if ($status_from_db ne 'done') {
 					print "close #$bug_nb of $pkg (at $closedate{$bug_nb})\n";
 					$status_from_db = 'done';
-					$changed = 1;
+					$changed = 2;
 				}
 			} else {
 				if ($opendate{$bug_nb} ne $date) {
 					print "fix date of #$bug_nb of $pkg from $date to $opendate{$bug_nb}.\n";
 					$date = $opendate{$bug_nb};
+					$changed = 1 unless $changed == 2;
 				}
 				if ($bugwontfix and $status_from_db ne 'wontfix') {
 					print "wontfix #$bug_nb of $pkg\n";
 					$status_from_db = 'wontfix';
-					$changed = 1;
+					$changed = 2;
 				}
 			}
 			if ($bugsubmitter{$bug_nb} ne $translator) {
 				print "fix submitter of #$bug_nb of $pkg from $translator to $bugsubmitter{$bug_nb}.\n";
 				$translator = $bugsubmitter{$bug_nb};
+				$changed = 1 unless $changed == 2;
 			}
 			if ($status_from_db eq 'wontfix' and not $bugwontfix) {
 				print "removing wontfix tag for #$bug_nb of $pkg\n";
 				$status_from_db = 'bts';
-				$changed = 1;
+				$changed = 2;
 			}
-			if ($changed) {
+
+			if ($changed == 2) {
 				$db->add_status($pkg, $type, $file, $date, $status_from_db, $translator, $list, $url, $bug_nb);
-			} else {
+			} elsif ($changed == 1) {
 				$db->set_status($pkg, $type, $file, $date, $status_from_db, $translator, $list, $url, $bug_nb, $statusline);
 			}
 		}