From: moodler Date: Wed, 9 Oct 2002 14:20:24 +0000 (+0000) Subject: Arg .. so much for quick hacks ... this code is so messy it's X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=bc0750aaaec4a91245e161295471a74d44622038;p=moodle.git Arg .. so much for quick hacks ... this code is so messy it's a hassle to hack ... going back to the way it was originally, with just a more informative error message instead. --- diff --git a/lib/ipatlas/plot.inc b/lib/ipatlas/plot.inc index 0baf8ae3e3..5f5aabfbad 100644 --- a/lib/ipatlas/plot.inc +++ b/lib/ipatlas/plot.inc @@ -48,13 +48,11 @@ function s10_rfc_1918_ip($in) { function stuffanalyze($stuff) { -$badstuff = array("bad", "bad", "bad", "bad", "bad"); - -$stuff2[0] = array_values(preg_grep ("/CITY:/", $stuff)) or return $badstuff; -$stuff2[1] = array_values(preg_grep ("/STATE:/", $stuff)) or return $badstuff; -$stuff2[2] = array_values(preg_grep ("/COUNTRY:/", $stuff)) or return $badstuff; -$stuff2[3] = array_values(preg_grep ("/LAT:/", $stuff)) or return $badstuff; -$stuff2[4] = array_values(preg_grep ("/LONG:/", $stuff)) or return $badstuff; +$stuff2[0] = array_values(preg_grep ("/CITY:/", $stuff)) or die("Sorry, but the lookup for this IP address failed! (CITY)"); +$stuff2[1] = array_values(preg_grep ("/STATE:/", $stuff)) or die("Sorry, but the lookup for this IP address failed! (STATE)"); +$stuff2[2] = array_values(preg_grep ("/COUNTRY:/", $stuff)) or die("Sorry, but the lookup for this IP address failed! (COUNTRY)"); +$stuff2[3] = array_values(preg_grep ("/LAT:/", $stuff)) or die("Sorry, but the lookup for this IP address failed! (LAT)"); +$stuff2[4] = array_values(preg_grep ("/LONG:/", $stuff)) or die("Sorry, but the lookup for this IP address failed! (LONG)"); // all the stuff2 values are actually arrays, making the following code look like crap @@ -70,10 +68,14 @@ for ($count = 0; $count < count($stuff2); $count++) { } else { // if no data was found - return $badstuff; + $stuff2 = array("bad", "bad", "bad", "bad", "bad"); } +$count = 0; + + return $stuff2; + } function getlatdata($ip) {