gidkom / php-openfire-restapi

src/Gidkom/OpenFireRestApi/OpenFireRestApi.php

Braces +3 added lines, -1 removed lines

@@ -25,7 +25,9 @@
     {
         $query = '';
         
-        if(isset($opts['search'])) $query .= '?search='.$opts['search'];
+        if(isset($opts['search'])) {
+            $query .= '?search='.$opts['search'];
+        }
         
     	$endpoint = '/users'.$query;        
     	return $this->doRequest('GET', $endpoint);

Doc Comments -8 removed lines

@@ -119,7 +119,6 @@ 
      * Add user to a group
      *
      * @param   string          $username   Username
-     * @param   string           $groups   Groups to add user
      * @return  json|false                 Json with data or error, or False when something went fully wrong
      */
     public function addToGroup($username, $groupName)
@@ -133,7 +132,6 @@ 
      * Delete user from a group
      *
      * @param   string          $username   Username
-     * @param   string           $groups   Groups to add user
      * @return  json|false                 Json with data or error, or False when something went fully wrong
      */
     public function deleteFromGroup($username, $groupName)
@@ -277,7 +275,6 @@ 
     /**
      * Delete a chat room
      *
-     * @param   string      $name               Name of the Group to delete
      * @return  json|false                       Json with data or error, or False when something went fully wrong
      */
     public function deleteChatRoom($roomName)
@@ -288,7 +285,6 @@ 
     /**
      * Get chat room participants
      *
-     * @param   string      $name               Name of the chatroom
      * @return  json|false                       Json with data or error, or False when something went fully wrong
      */
     public function getChatRoomParticipants($roomName)      
@@ -299,7 +295,6 @@ 
     /**
      * Get chat room occupants
      *
-     * @param   string      $name               Name of the chatroom
      * @return  json|false                       Json with data or error, or False when something went fully wrong
      */
     public function getChatRoomOccupants($roomName)      
@@ -356,7 +351,6 @@ 
     /**
      * Retrieve a system property
      *
-     * @param   string      $name                Name of property
      * @return  json|false                       Json with data or error, or False when something went fully wrong
      */
     public function getSystemProperty($propertyName)
@@ -393,7 +387,6 @@ 
     /**
      * Delete a system property
      *
-     * @param   array      $data                new property with value
      * @return  json|false                       Json with data or error, or False when something went fully wrong
      */
     public function deleteSystemProperty($propertyName)
@@ -514,7 +507,6 @@ 
     /**
      * Send a broadcast message to all online users
      *
-     * @param  string      $content               message to send
      * @return json|false       Json with data or error, or False when something went fully wrong
      */
     public function broadcastMessage($message = '')

Indentation +7 added lines, -7 removed lines

@@ -27,8 +27,8 @@ 
         
         if(isset($opts['search'])) $query .= '?search='.$opts['search'];
         
-    	$endpoint = '/users'.$query;        
-    	return $this->doRequest('GET', $endpoint);
+        $endpoint = '/users'.$query;        
+        return $this->doRequest('GET', $endpoint);
     }
 
 
@@ -413,11 +413,11 @@ 
     }
 
 
-     /**
-     * Get all groups
-     *
-     * @return  json|false      Json with data or error, or False when something went fully wrong
-     */
+        /**
+         * Get all groups
+         *
+         * @return  json|false      Json with data or error, or False when something went fully wrong
+         */
     public function getGroups()
     {
         $endpoint = '/groups';

Spacing +19 added lines, -19 removed lines

@@ -21,11 +21,11 @@ 
      *
      * @return json|false       Json with data or error, or False when something went fully wrong
      */
-    public function getUsers($opts = [])
+    public function getUsers($opts = [ ])
     {
         $query = '';
         
-        if(isset($opts['search'])) $query .= '?search='.$opts['search'];
+        if (isset($opts[ 'search' ])) $query .= '?search='.$opts[ 'search' ];
         
     	$endpoint = '/users'.$query;        
     	return $this->doRequest('GET', $endpoint);
@@ -54,10 +54,10 @@ 
      * @param   string[]|false  $groups     Groups  (Optional)
      * @return  json|false                 Json with data or error, or False when something went fully wrong
      */
-    public function addUser($username, $password, $name=false, $email=false, $groups=false)
+    public function addUser($username, $password, $name = false, $email = false, $groups = false)
     {
         $endpoint = '/users'; 
-        return $this->doRequest('POST', $endpoint, compact('username', 'password','name','email', 'groups'));
+        return $this->doRequest('POST', $endpoint, compact('username', 'password', 'name', 'email', 'groups'));
     }
 
 
@@ -83,10 +83,10 @@ 
      * @param   string[]|false  $groups     Groups (Optional)
      * @return  json|false                 Json with data or error, or False when something went fully wrong
      */
-    public function updateUser($username, $password, $name=false, $email=false, $groups=false)
+    public function updateUser($username, $password, $name = false, $email = false, $groups = false)
     {
         $endpoint = '/users/'.$username; 
-        return $this->doRequest('PUT', $endpoint, compact('username', 'password','name','email', 'groups'));
+        return $this->doRequest('PUT', $endpoint, compact('username', 'password', 'name', 'email', 'groups'));
     }
 
 
@@ -125,7 +125,7 @@ 
     public function addToGroup($username, $groupName)
     {
         $endpoint = '/users/'.$username.'/groups/'.$groupName; 
-        return $this->doRequest('POST', $endpoint );
+        return $this->doRequest('POST', $endpoint);
     }
 
 
@@ -139,7 +139,7 @@ 
     public function deleteFromGroup($username, $groupName)
     {
         $endpoint = '/users/'.$username.'/groups/'.$groupName; 
-        return $this->doRequest('DELETE', $endpoint );
+        return $this->doRequest('DELETE', $endpoint);
     }
 
 
@@ -178,7 +178,7 @@ 
     public function userRosters($username)
     {
         $endpoint = '/users/'.$username.'/roster';
-        return $this->doRequest('GET', $endpoint, compact('jid','name','subscriptionType'));
+        return $this->doRequest('GET', $endpoint, compact('jid', 'name', 'subscriptionType'));
     }
 
 
@@ -191,10 +191,10 @@ 
      * @param   int|false       $subscriptionType   	Subscription (Optional)
      * @return  json|false                     		Json with data or error, or False when something went fully wrong
      */
-    public function addToRoster($username, $jid, $name=false, $subscriptionType=false)
+    public function addToRoster($username, $jid, $name = false, $subscriptionType = false)
     {
         $endpoint = '/users/'.$username.'/roster';
-        return $this->doRequest('POST', $endpoint, compact('jid','name','subscriptionType'));
+        return $this->doRequest('POST', $endpoint, compact('jid', 'name', 'subscriptionType'));
     }
 
 
@@ -220,10 +220,10 @@ 
      * @param   int|false       $subscriptionType   Subscription (Optional)
      * @return  json|false                          Json with data or error, or False when something went fully wrong
      */
-    public function updateRoster($username, $jid, $nickname=false, $subscriptionType=false)
+    public function updateRoster($username, $jid, $nickname = false, $subscriptionType = false)
     {
         $endpoint = '/users/'.$username.'/roster/'.$jid;
-        return $this->doRequest('PUT', $endpoint, $jid, compact('jid','username','subscriptionType'));     
+        return $this->doRequest('PUT', $endpoint, $jid, compact('jid', 'username', 'subscriptionType'));     
     }
 
    
@@ -256,7 +256,7 @@ 
      * @param   string          $params        Params
      * @return  json|false                     Json with data or error, or False when something went fully wrong
      */
-    public function createChatRoom($params = [])
+    public function createChatRoom($params = [ ])
     {
         return $this->doRequest('POST', '/chatrooms', $params);
     }
@@ -268,7 +268,7 @@ 
      * @param   string          $params        Params
      * @return  json|false                     Json with data or error, or False when something went fully wrong
      */
-    public function updateChatRoom($roomName, $params = [])
+    public function updateChatRoom($roomName, $params = [ ])
     {
         return $this->doRequest('PUT', '/chatrooms/'.$roomName, $params);
     }
@@ -447,7 +447,7 @@ 
     public function createGroup($name, $description = false)
     {
         $endpoint = '/groups/';
-        return $this->doRequest('POST', $endpoint, compact('name','description'));
+        return $this->doRequest('POST', $endpoint, compact('name', 'description'));
     }
 
     /**
@@ -469,10 +469,10 @@ 
      * @param   string      $description        Some description of the group
      *
      */
-    public function updateGroup($name,  $description)
+    public function updateGroup($name, $description)
     {
         $endpoint = '/groups/'.$name;
-        return $this->doRequest('PUT', $endpoint, compact('name','description'));
+        return $this->doRequest('PUT', $endpoint, compact('name', 'description'));
     }
 
 
@@ -519,7 +519,7 @@ 
      */
     public function broadcastMessage($message = '')
     {
-        $content =['body'=> $message];
+        $content = [ 'body'=> $message ];
         $endpoint = '/messages/users';
         return $this->doRequest('POST', $endpoint, $content);
     }

src/Gidkom/OpenFireRestApi/RestClient.php

Braces +5 added lines, -4 removed lines

@@ -45,10 +45,11 @@
     	$base = ($this->useSSL) ? "https" : "http";
     	$url = $base . "://" . $this->host . ":" .$this->port.$this->plugin.$endpoint;
 	    
-		if ($this->useBasicAuth)
-            $auth = 'Basic ' . base64_encode($this->basicUser . ':' . $this->basicPwd);
-        else
-            $auth = $this->secret;
+		if ($this->useBasicAuth) {
+		            $auth = 'Basic ' . base64_encode($this->basicUser . ':' . $this->basicPwd);
+		} else {
+                    $auth = $this->secret;
+        }
 	    
     	$headers = array(
   			'Accept' => 'application/json',

Indentation +15 added lines, -15 removed lines

@@ -8,7 +8,7 @@ 
 */
 class RestClient
 {
-	public $host = 'localhost';
+    public $host = 'localhost';
     public $port = '9090';
     public $plugin = '/plugins/restapi/v1';
     public $secret = 'SuperSecret';
@@ -20,10 +20,10 @@ 
     public $basicUser = 'admin';
     public $basicPwd = '1234';
 
-	function __construct()
-	{
-		$this->client = new Client();
-	}
+    function __construct()
+    {
+        $this->client = new Client();
+    }
 
     /**
      * Make the request and analyze the result
@@ -33,28 +33,28 @@ 
      * @param   array           $params         Parameters
      * @return  array|false                     Array with data or error, or False when something went fully wrong
      */
-	protected function doRequest($type, $endpoint, $params=[])
+    protected function doRequest($type, $endpoint, $params=[])
     {
-    	$base = ($this->useSSL) ? "https" : "http";
-    	$url = $base . "://" . $this->host . ":" .$this->port.$this->plugin.$endpoint;
+        $base = ($this->useSSL) ? "https" : "http";
+        $url = $base . "://" . $this->host . ":" .$this->port.$this->plugin.$endpoint;
 	    
-		if ($this->useBasicAuth)
+        if ($this->useBasicAuth)
             $auth = 'Basic ' . base64_encode($this->basicUser . ':' . $this->basicPwd);
         else
             $auth = $this->secret;
 	    
-    	$headers = array(
-  			'Accept' => 'application/json',
-  			'Authorization' => $auth,
+        $headers = array(
+                'Accept' => 'application/json',
+                'Authorization' => $auth,
             'Content-Type'=>'application/json'
-  		);
+            );
 
         $body = json_encode($params);
 
         try {
-        	$result = $this->client->request($type, $url, compact('headers','body'));
+            $result = $this->client->request($type, $url, compact('headers','body'));
         } catch (\Exception $e) {
-        	return  ['status'=>false, 'data'=>['message'=>$e->getMessage()]];
+            return  ['status'=>false, 'data'=>['message'=>$e->getMessage()]];
         }
 	        
         

Spacing +5 added lines, -5 removed lines

@@ -33,13 +33,13 @@ 
      * @param   array           $params         Parameters
      * @return  array|false                     Array with data or error, or False when something went fully wrong
      */
-	protected function doRequest($type, $endpoint, $params=[])
+	protected function doRequest($type, $endpoint, $params = [ ])
     {
     	$base = ($this->useSSL) ? "https" : "http";
-    	$url = $base . "://" . $this->host . ":" .$this->port.$this->plugin.$endpoint;
+    	$url = $base."://".$this->host.":".$this->port.$this->plugin.$endpoint;
 	    
 		if ($this->useBasicAuth)
-            $auth = 'Basic ' . base64_encode($this->basicUser . ':' . $this->basicPwd);
+            $auth = 'Basic '.base64_encode($this->basicUser.':'.$this->basicPwd);
         else
             $auth = $this->secret;
 	    
@@ -52,9 +52,9 @@ 
         $body = json_encode($params);
 
         try {
-        	$result = $this->client->request($type, $url, compact('headers','body'));
+        	$result = $this->client->request($type, $url, compact('headers', 'body'));
         } catch (\Exception $e) {
-        	return  ['status'=>false, 'data'=>['message'=>$e->getMessage()]];
+        	return  [ 'status'=>false, 'data'=>[ 'message'=>$e->getMessage() ] ];
         }