misc bugfixes (#3)

Author: algirdasci

modules/servers/hostinger/hostinger.php

@@ -105,32 +105,42 @@ function hostinger_AdminCustomButtonArray(): array
  */
 function hostinger_UsageUpdate(array $params): void
 {
-    try {
-        $apiClient = new VPSVirtualMachineApi(config: getHostingerApiConfig($params));
-        $response = $apiClient->getMetricsV1(
-            virtualMachineId: getHostingerVirtualMachineId($params),
-            dateFrom: new DateTime('-1 day'),
-            dateTo: new DateTime(),
-        );
-
-        $diskUsageArray = (array) $response->getDiskSpace()->getUsage();
-        $incomingTrafficArray = (array) $response->getIncomingTraffic()->getUsage();
-        $outgoingTrafficArray = (array) $response->getOutgoingTraffic()->getUsage();
-        $latestDiskUsage = array_pop($diskUsageArray);
-        $latestIncomingTraffic = array_pop($incomingTrafficArray);
-        $latestOutgoingTraffic = array_pop($outgoingTrafficArray);
-
-        Capsule::table('tblhosting')
-            ->where('server', $params['serverid'])
-            ->update([
-                'diskusage' => $latestDiskUsage / 1024 / 1024 ?? $params['diskusage'],
-                'bwusage' => ($latestIncomingTraffic + $latestOutgoingTraffic) / 1024 / 1024 ?? $params['bwusage'],
-                'lastupdate' => Capsule::raw('now()'),
-            ]);
-    } catch (Throwable $e) {
-        logModuleCall('hostinger', __FUNCTION__, $params, $e->getMessage(), $e->getTraceAsString());
-
-        throw $e;
+    $apiClient = new VPSVirtualMachineApi(config: getHostingerApiConfig($params));
+
+    $servers = Capsule::table('tblhosting')
+        ->where('server', $params['serverid'])
+        ->get();
+
+    foreach ($servers as $server) {
+        $serverId = (int) $server->subscriptionid ?? null;
+        if (!$serverId) {
+            continue;
+        }
+
+        try {
+            $response = $apiClient->getMetricsV1(
+                virtualMachineId: $serverId,
+                dateFrom: new DateTime('-1 day'),
+                dateTo: new DateTime(),
+            );
+
+            $diskUsageArray = (array)$response->getDiskSpace()->getUsage();
+            $incomingTrafficArray = (array)$response->getIncomingTraffic()->getUsage();
+            $outgoingTrafficArray = (array)$response->getOutgoingTraffic()->getUsage();
+            $latestDiskUsage = array_pop($diskUsageArray);
+            $latestIncomingTraffic = array_pop($incomingTrafficArray);
+            $latestOutgoingTraffic = array_pop($outgoingTrafficArray);
+
+            Capsule::table('tblhosting')
+                ->where('server', $params['serverid'])
+                ->update([
+                    'diskusage'  => $latestDiskUsage / 1024 / 1024 ?? $params['diskusage'],
+                    'bwusage'    => ($latestIncomingTraffic + $latestOutgoingTraffic) / 1024 / 1024 ?? $params['bwusage'],
+                    'lastupdate' => Capsule::raw('now()'),
+                ]);
+        } catch (Throwable $e) {
+            logModuleCall('hostinger', __FUNCTION__, $params, $e->getMessage(), $e->getTraceAsString());
+        }
     }
 }
 

modules/servers/hostinger/include/clientarea.php

@@ -149,8 +149,8 @@ function hostinger_ClientAreaSetNameservers(array $params, array $details = []):
 {
     $error = null;
 
-    if (!empty($_POST)) {
-        $error = hostinger_SetNameservers($params, $_POST['ns1'], $_POST['ns2']);
+    if (!empty($_POST['ns1'])) {
+        $error = hostinger_SetNameservers($params, $_POST['ns1'], $_POST['ns2'] ?? null);
         if ($error === 'success') {
             markHostingerServerDetailsObsolete($params);
             makeRedirect(makeActionLink($params, 'SetNameservers'));
@@ -171,7 +171,7 @@ function hostinger_ClientAreaChangeRootPassword(array $params, array $details =
 {
     $error = null;
 
-    if (!empty($_POST)) {
+    if (!empty($_POST['password'])) {
         $error = hostinger_ChangeRootPassword($params, $_POST['password']);
         if ($error === 'success') {
             markHostingerServerDetailsObsolete($params);
@@ -191,7 +191,7 @@ function hostinger_ClientAreaChangePanelPassword(array $params, array $details =
 {
     $error = null;
 
-    if (!empty($_POST)) {
+    if (!empty($_POST['password'])) {
         $error = hostinger_ChangePanelPassword($params, $_POST['password']);
         if ($error === 'success') {
             markHostingerServerDetailsObsolete($params);