Merge pull request #188 from whytea91/bugfix-correctly-fetch-current-databases-number

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

Author: sibprogrammer

src/Api/Operator/Database.php

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

tests/DatabaseTest.php

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