# HG changeset patch
# User Dominic Cleal <dominic@computerkb.co.uk>
# Date 1228584437 0
# Node ID 10587dbd634b76259f9869e2da8379104dce0f82
# Parent  bef72aa85f2b8440856eb27d6ada8ef2ad91234f
Ignoring bad data which was actually negative

diff -r bef72aa85f2b -r 10587dbd634b skillbot.pl
--- a/skillbot.pl	Sat Dec 06 16:49:36 2008 +0000
+++ b/skillbot.pl	Sat Dec 06 17:27:17 2008 +0000
@@ -367,7 +367,7 @@
 					$loaded++;
 				}
 				
-				if (defined $c && $loaded == 0)
+				if (defined $c && $loaded eq 0)
 				{
 					irc_debug("EVE: Unable to find character %s for ID %lu",
 							  $c, $uid);
@@ -415,19 +415,19 @@
 
 	# Check for changes in the skill, skip or cancel announcement
 	if (defined $f->{skill}) {
-		next if $skill->id == $f->{skill}->id;
+		return if $skill->id eq $f->{skill}->id;
 		del_one_shot_timer($f->{timer});
 	}
 
 	my $finish = $skill->finish_time - time();
 	my $prefinish = $finish - 60;
 
-	irc_debug("Character %s is training %s %lu (%s == %lu sec)",
+	irc_debug("Character %s is training %s %d (%s == %d sec)",
 			  $f->{char}->name, $skill->name, $skill->level,
 			  $skill->time_remaining, $finish);
 
-	# Cap time at 3 months to ignore bad data
-	if ($finish > 60 * 60 * 24 * 90)
+	# Ignore bad data
+	if ($finish < 0)
 	{
 		irc_debug("Unreasonable finish time given of %s seconds, ignoring",
 				  $finish);