Hi, what about improving the getRecords method to add default values for page and per_page params and the fields parameter like the one below? It should work and shouldn't affect backward compatibility.

It would be great to have better documentation of the SDK and several examples.

2. public function getRecords($cvId = null, $sortByField = null, $sortOrder = null, $page = 1, $perPage = 200, $customHeaders = null, $fields= null)
3. /*
4.  * Optional parameters:
5.  * cvId = custom view id (integer)
6.  * sortByField = Field API Name (string)
7.  * sortOrder = asc - ascending order || desc - descending order
8.  * page = Positive Integer values only default is 1
9.  * per_page = Positive Integer values only default is 200
10.  * customHeaders = string
11.  * fields = Multiple field API names, comma separated. For example: Last_Name,Email (string)
12.  * 
13.  * I put fields as last to avoid breaking backward compatibility. "converted" and "approved" might be added too.
14.  */
15.  {
16.     try {
17.         $this->urlPath = $this->module->getAPIName();
18.         $this->requestMethod = APIConstants::REQUEST_METHOD_GET;
19.         $this->addHeader("Content-Type", "application/json");
20.         if ($customHeaders != null) {
21.             foreach ($customHeaders as $key => $value) {
22.                 $this->addHeader($key, $value);
23.             }
24.         }
25.         if ($fields != null) {
26.             $this->addParam("fields", (string) $fields);
27.         }
28.         if ($cvId != null) {
29.             $this->addParam("cvid", (integer) $cvId);
30.         }
31.         if ($sortByField != null) {
32.             $this->addParam("sort_by", $sortByField);
33.         }
34.         if ($sortOrder != null) {
35.             $this->addParam("sort_order", $sortOrder);
36.         }
37. 
    
38.         $this->addParam("page", (integer) $page);
39. 
    
40.         $this->addParam("per_page", (integer) $perPage);
41. 
    
42. 
    
43.         $responseInstance = APIRequest::getInstance($this)->getBulkAPIResponse();
44.         $responseJSON = $responseInstance->getResponseJSON();
45.         $records = $responseJSON["data"];
46.         $recordsList = array();
47.         foreach ($records as $record) {
48.             $recordInstance = ZCRMRecord::getInstance($this->module->getAPIName(), $record["id"]);
49.             EntityAPIHandler::getInstance($recordInstance)->setRecordProperties($record);
50.             array_push($recordsList, $recordInstance);
51.         }
52. 
    
53.         $responseInstance->setData($recordsList);
54. 
    
55.         return $responseInstance;
56.     } catch (ZCRMException $exception) {
57.         APIExceptionHandler::logException($exception);
58.         throw $exception;
59.     }
60. }