change it_url::is_reachable to static api consistent with it_url::get

1 files changed, 4 insertions, 8 deletions

diff --git a/it_url.class b/it_url.class
index 1b90344..ad28f2c 100644
--- a/it_url.class
+++ b/it_url.class
@@ -68,15 +68,13 @@ function __construct($url = null)
 /**
  * Check if a given url (currently http:port80-only) can be fetched
  * Note: Redirects are treated as succesful
- * $timeout Timeout for connection in seconds
+ * @param $p parameter array passed on to get
  * @return true if url could be fetched
  */
-function is_reachable($timeout = 5)
+static function is_reachable($p = [])
 {
-	$url = new it_url($this->url);
-	@$url->_get(['method' => 'HEAD', 'totaltimeout' => $timeout]);
-
-	return $url->result >= 200 && $url->result < 400;
+	$result = static::get((is_array($p) ? $p : ['url' => $p]) + ['method' => 'HEAD', 'totaltimeout' => 5, 'assoc' => true, 'it_error' => false]);
+	return $result['status'] >= 200 && $result['status'] < 400;
 }
 
 # internal
@@ -91,8 +89,6 @@ static function _postprocess($data, $p)
 /**
  * Get simple URL with timeout and one retry. Can be called statically. Times out, calls it::error for all errs
  *
- * If the protocol is not http, only features of get_multi are supported.
- *
  * @param $p parameter array with the following keys
  * @param $p['url']           url to get, defaults to constructor URL
  * @param $p['assoc']         Return [ 'data' => string, 'status' => int, 'cookies' => array, 'headers' => array, 'errstr' => string ] instead of just data