BUGFIX EXTPLESK-8950 Fix case when getAll returns 1 database instead of 0

Author: whytea91

src/Api/Operator/Database.php

@@ -78,9 +78,15 @@ public function getAll(?string $field, $value): array
         $response = $this->getBy('get-db', $field, $value);
         $items = [];
         foreach ((array) $response->xpath('//result') as $xmlResult) {
-            if ($xmlResult) {
-                $items[] = new Struct\Info($xmlResult);
+            if (!$xmlResult) {
+                continue;
+            }
+
+            if (!isset($xmlResult->id) || (int) $xmlResult->id <= 0) {
+                continue;
             }
+
+            $items[] = new Struct\Info($xmlResult);
         }
 
         return $items;

tests/DatabaseTest.php

@@ -225,4 +225,15 @@ private function createUser(array $params): \PleskX\Api\Struct\Database\UserInfo
 
         return $user;
     }
+
+    public function testGetAllForWebspaceWithNoDatabases()
+    {
+        $webspace = static::createWebspace();
+        $databases = static::$client->database()->getAll('webspace-id', $webspace->id);
+
+        $this->assertIsArray($databases);
+        $this->assertEmpty($databases);
+
+        static::$client->webspace()->delete('id', $webspace->id);
+    }
 }