Make getAll() return the list of users without wrapping it under a 'list' item.

[irc/unrealircd/unrealircd-rpc-php.git] / lib / ServerBan.php

diff --git a/lib/ServerBan.php b/lib/ServerBan.php
index 2a7eaabbc84e7ebdc35d96ba68972ba0f0941c80..126692ad5e3fb341dcd7f7a80611834a6dab4288 100644 (file)
--- a/lib/ServerBan.php
+++ b/lib/ServerBan.php
@@ -5,7 +5,7 @@ namespace UnrealIRCd;
 use Exception;
 use stdClass;
 
-class ServerBan implements Contracts\ServerBan
+class ServerBan
 {
 
     public Connection $connection;
@@ -19,10 +19,10 @@ class ServerBan implements Contracts\ServerBan
      * Add a ban.
      *
      * @param  string  $user
-     * @return stdClass
+     * @return stdClass|array|bool
      * @throws Exception
      */
-    public function add(string $name, string $type, string $duration, string $reason): stdClass
+    public function add(string $name, string $type, string $duration, string $reason): stdClass|array|bool
     {
         return $this->connection->query('server_ban.add', [
             'name' => $name,
@@ -36,10 +36,10 @@ class ServerBan implements Contracts\ServerBan
      * Delete a ban.
      *
      * @param  string  $name
-     * @return stdClass
+     * @return stdClass|array|bool
      * @throws Exception
      */
-    public function delete(string $name, string $type): stdClass
+    public function delete(string $name, string $type): stdClass|array|bool
     {
         return $this->connection->query('server_ban.del', [
             'name' => $name,
@@ -50,15 +50,15 @@ class ServerBan implements Contracts\ServerBan
     /**
      * Return a list of all bans.
      *
-     * @return stdClass
+     * @return stdClass|array|bool
      * @throws Exception
      */
-    public function getAll(): stdClass
+    public function getAll(): stdClass|array|bool
     {
         $response = $this->connection->query('server_ban.list');
 
         if (!is_bool($response)) {
-            return $response;
+            return $response->list;
         }
 
         throw new Exception('Invalid JSON Response from UnrealIRCd RPC.');
@@ -67,15 +67,14 @@ class ServerBan implements Contracts\ServerBan
     /**
      * Get a specific ban.
      *
-     * @param  array  $params
-     * @return stdClass
+     * @return stdClass|array|bool
      * @throws Exception
      */
-    public function get(array $params): stdClass
+    public function get(string $name, string $type): stdClass|array|bool
     {
         $response = $this->connection->query('server_ban.get', [
-            'name' => $params['name'],
-            'type' => $params['type']
+            'name' => $name,
+            'type' => $type
         ]);
 
         if (!is_bool($response)) {