cleanup & comment

siteBuilder.php

@@ -309,15 +309,6 @@ function siteBuilder_SuspendAccount($params) {
 	}
 	// disable all sites but not change status in DB for unsuspend restoring
 	$sites = getSites($params['serviceid']);
-
-	logModuleCall(
-		'siteBuilder',
-		__FUNCTION__,
-		$params,
-		'Getsites',
-		$sites
-		);
-
 	if(!empty($sites)) {
 		foreach($sites as $site) {
 			$response = $siteBuilder->disable($params['username'], $site, $params['serverusername'], $params['serverpassword']);
@@ -1103,7 +1094,7 @@ function getSites($serviceID) {
 	try {
 		$sites = Capsule::table('sitePro_site')
 		->where('relid',$serviceID)
-		->value('name');
+		->select('name');
 	} catch (\Exception $e) {
 		logModuleCall(
 			'siteBuilder',
@@ -1114,6 +1105,15 @@ function getSites($serviceID) {
 			);
 		return 'Error: could fetch sites from database';
 	}
+
+	logModuleCall(
+		'siteBuilder',
+		__FUNCTION__,
+		$serviceID,
+		'Getsites',
+		$sites
+		);
+
 	return $sites;
 }