Fix some warnings.
authorSteinar H. Gunderson <sgunderson@bigfoot.com>
Mon, 24 Nov 2014 19:41:10 +0000 (20:41 +0100)
committerSteinar H. Gunderson <sgunderson@bigfoot.com>
Mon, 24 Nov 2014 19:41:10 +0000 (20:41 +0100)
remoteglot.pl

index 917a6a0..61e7237 100755 (executable)
@@ -760,7 +760,7 @@ sub output_json {
        $json->{'refutation_lines'} = \%refutation_lines;
 
        my $encoded = JSON::XS::encode_json($json);
-       unless ($historic_json_only) {
+       unless ($historic_json_only || !defined($remoteglotconf::json_output)) {
                atomic_set_contents($remoteglotconf::json_output, $encoded);
        }
 
@@ -1014,9 +1014,10 @@ sub find_clock_start {
        my $pos = shift;
 
        # If the game is over, the clock is stopped.
-       if ($pos->{'result'} eq '1-0' ||
-           $pos->{'result'} eq '1/2-1/2' ||
-           $pos->{'result'} eq '0-1') {
+       if (exists($pos->{'result'}) &&
+           ($pos->{'result'} eq '1-0' ||
+            $pos->{'result'} eq '1/2-1/2' ||
+            $pos->{'result'} eq '0-1')) {
                return;
        }
 
@@ -1025,6 +1026,11 @@ sub find_clock_start {
                return;
        }
 
+       # TODO(sesse): Maybe we can get the number of moves somehow else for FICS games.
+       if (!exists($pos->{'pretty_history'})) {
+               return;
+       }
+
        my $id = id_for_pos($pos);
        if (exists($clock_target_for_pos{$id})) {
                if ($pos->{'toplay'} eq 'W') {