salaros / vtwsclib-php

src/WSClient.php

Doc Comments +1 added lines, -1 removed lines

@@ -258,7 +258,7 @@
     /**
      * Gets last operation error, if any
      * @access public
-     * @return WSClientError The error object
+     * @return boolean The error object
      */
     public function getLastError()
     {

Spacing +38 added lines, -38 removed lines

@@ -77,7 +77,7 @@ 
         if (!preg_match('/^https?:\/\//i', $url)) {
             $url = sprintf('http://%s', $url);
         }
-        if (strripos($url, '/') != (strlen($url)-1)) {
+        if (strripos($url, '/') != (strlen($url) - 1)) {
             $url .= '/';
         }
 
@@ -95,14 +95,14 @@ 
      */
     private function checkForError(array $jsonResult) // TODO move checkForError body to sendHttpRequest method's body
     {
-        if (isset($jsonResult['success']) && (bool)$jsonResult['success'] === true) {
+        if (isset($jsonResult[ 'success' ]) && (bool) $jsonResult[ 'success' ] === true) {
             $this->lastErrorMessage = null;
             return false;
         }
 
         $this->lastErrorMessage = new WSClientError(
-            $jsonResult['error']['message'],
-            $jsonResult['error']['code']
+            $jsonResult[ 'error' ][ 'message' ],
+            $jsonResult[ 'error' ][ 'code' ]
         );
 
         return true;
@@ -132,10 +132,10 @@ 
         try {
             switch ($method) {
                 case 'GET':
-                    $response = $this->httpClient->get($this->serviceBaseURL, ['query' => $requestData]);
+                    $response = $this->httpClient->get($this->serviceBaseURL, [ 'query' => $requestData ]);
                     break;
                 case 'POST':
-                    $response = $this->httpClient->post($this->serviceBaseURL, ['form_params' => $requestData]);
+                    $response = $this->httpClient->post($this->serviceBaseURL, [ 'form_params' => $requestData ]);
                     break;
                 default:
                     $this->lastErrorMessage = new WSClientError("Unknown request type {$method}");
@@ -154,7 +154,7 @@ 
 
         return (!is_array($jsonObj) || $this->checkForError($jsonObj))
             ? null
-            : $jsonObj['result'];
+            : $jsonObj[ 'result' ];
     }
 
     /**
@@ -171,13 +171,13 @@ 
         ];
         $result = $this->sendHttpRequest($getdata, 'GET');
         
-        if (!is_array($result) || !isset($result['token'])) {
+        if (!is_array($result) || !isset($result[ 'token' ])) {
             return false;
         }
 
-        $this->serviceServerTime = $result['serverTime'];
-        $this->serviceExpireTime = $result['expireTime'];
-        $this->serviceToken = $result['token'];
+        $this->serviceServerTime = $result[ 'serverTime' ];
+        $this->serviceExpireTime = $result[ 'expireTime' ];
+        $this->serviceToken = $result[ 'token' ];
 
         return true;
     }
@@ -212,12 +212,12 @@ 
         $this->accessKey = $accessKey;
 
         // Session data
-        $this->sessionName = $result['sessionName'];
-        $this->userID = $result['userId'];
+        $this->sessionName = $result[ 'sessionName' ];
+        $this->userID = $result[ 'userId' ];
 
         // Vtiger CRM and WebServices API version
-        $this->apiVersion = $result['version'];
-        $this->vtigerVersion = $result['vtigerVersion'];
+        $this->apiVersion = $result[ 'version' ];
+        $this->vtigerVersion = $result[ 'vtigerVersion' ];
 
         return true;
     }
@@ -249,8 +249,8 @@ 
         }
 
         $accessKey = array_key_exists('accesskey', $result)
-            ? $result['accesskey']
-            : $result[0];
+            ? $result[ 'accesskey' ]
+            : $result[ 0 ];
 
         return $this->login($username, $accessKey);
     }
@@ -292,11 +292,11 @@ 
         ];
 
         $result = $this->sendHttpRequest($getdata, 'GET');
-        $modules = $result['types'];
+        $modules = $result[ 'types' ];
 
         $result = array();
         foreach ($modules as $moduleName) {
-            $result[$moduleName] = ['name' => $moduleName];
+            $result[ $moduleName ] = [ 'name' => $moduleName ];
         }
         return $result;
     }
@@ -330,7 +330,7 @@ 
      */
     private function getTypedID($moduleName, $entityID)
     {
-        if (stripos((string)$entityID, 'x') !== false) {
+        if (stripos((string) $entityID, 'x') !== false) {
             return $entityID;
         }
 
@@ -341,7 +341,7 @@ 
             return false;
         }
 
-        return "{$type['idPrefix']}x{$entityID}";
+        return "{$type[ 'idPrefix' ]}x{$entityID}";
     }
 
     /**
@@ -387,7 +387,7 @@ 
         $this->checkLogin();
 
         // Make sure the query ends with ;
-        $query = (strripos($query, ';') != strlen($query)-1)
+        $query = (strripos($query, ';') != strlen($query) - 1)
             ? trim($query .= ';')
             : trim($query);
 
@@ -442,23 +442,23 @@ 
 
         // Build the query
         $criteria = array();
-        $query="SELECT id FROM $moduleName WHERE ";
+        $query = "SELECT id FROM $moduleName WHERE ";
         foreach ($params as $param => $value) {
-            $criteria[] = "{$param} LIKE '{$value}'";
+            $criteria[ ] = "{$param} LIKE '{$value}'";
         }
 
-        $query.=implode(" AND ", $criteria);
-        $query.=" LIMIT 1";
+        $query .= implode(" AND ", $criteria);
+        $query .= " LIMIT 1";
 
         $records = $this->query($query);
         if (!$records || !is_array($records) || (count($records) !== 1)) {
             return false;
         }
 
-        $entityID = $records[0]['id'];
+        $entityID = $records[ 0 ][ 'id' ];
         $entityIDParts = explode('x', $entityID, 2);
         return (is_array($entityIDParts) && count($entityIDParts) === 2)
-            ? $entityIDParts[1]
+            ? $entityIDParts[ 1 ]
             : -1;
     }
 
@@ -474,8 +474,8 @@ 
         $this->checkLogin();
 
         // Assign record to logged in user if not specified
-        if (!isset($params['assigned_user_id'])) {
-            $params['assigned_user_id'] = $this->userID;
+        if (!isset($params[ 'assigned_user_id' ])) {
+            $params[ 'assigned_user_id' ] = $this->userID;
         }
 
         $postdata = [
@@ -500,19 +500,19 @@ 
         $this->checkLogin();
 
         // Assign record to logged in user if not specified
-        if (!isset($params['assigned_user_id'])) {
-            $params['assigned_user_id'] = $this->userID;
+        if (!isset($params[ 'assigned_user_id' ])) {
+            $params[ 'assigned_user_id' ] = $this->userID;
         }
 
         // TODO implement the case when no ID is given
         if (array_key_exists('id', $params)) {
-            $data = $this->entityRetrieveByID($moduleName, $params['id']);
+            $data = $this->entityRetrieveByID($moduleName, $params[ 'id' ]);
             if ($data !== false && is_array($data)) {
-                $entityID = $data['id'];
+                $entityID = $data[ 'id' ];
                 $params = array_merge(
-                    $data,      // needed to provide mandatory field values
-                    $params,    // updated data override
-                    ['id'=>$entityID] // fixing id, might be useful when non <moduleid>x<id> one was specified
+                    $data, // needed to provide mandatory field values
+                    $params, // updated data override
+                    [ 'id'=>$entityID ] // fixing id, might be useful when non <moduleid>x<id> one was specified
                 );
             }
         }
@@ -572,7 +572,7 @@ 
         ];
 
         if (!empty($moduleName)) {
-            $requestData['elementType'] = $moduleName;
+            $requestData[ 'elementType' ] = $moduleName;
         }
 
         return $this->sendHttpRequest($requestData, true);