more separation of complex code into separate functions

include/hubloc.php

@@ -257,6 +257,38 @@ function hubloc_mark_as_down($posturl) {
 }
 
 
+/**
+ * @brief return comma separated string of non-dead clone locations (net addresses) for a given netid
+ *
+ * @param string $netid network identity (typically xchan_hash or hubloc_hash)
+ * @return string
+ */ 
+
+function locations_by_netid($netid) {
+
+	$strloc = '';
+
+	$locs = q("select hubloc_addr as location from hubloc left join site on hubloc_url = site_url where hubloc_hash = '%s' and hubloc_deleted = 0 and site_dead = 0",
+		dbesc($netid)
+	);
+
+	if($locs) {
+		foreach($locs as $l) {
+			if(!($l['location']))
+				continue;
+			if(strpos($strloc,$l['location']) !== false)
+				continue;
+			if(strlen($strloc))
+				$strloc .= ', ';
+			$strloc .= $l['location'];
+		}
+	}
+
+	return $strloc;	
+
+}
+
+
 
 function ping_site($url) {