ryanwinchester / hubspot-php

src/Resources/Deals.php

Unused Use Statements -2 removed lines

@@ -1,8 +1,6 @@
 <?php
 namespace SevenShores\Hubspot\Resources;
 
-use SevenShores\Hubspot\Exceptions\HubspotException;
-
 class Deals extends Resource
 {
     /**

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
      * @return \Psr\Http\Message\ResponseInterface|\SevenShores\Hubspot\Http\Response
      * @throws \SevenShores\Hubspot\Exceptions\BadRequest
      */
-    function getAll(array $params = []){
+    function getAll(array $params = []) {
         $endpoint = "https://api.hubapi.com/deals/v1/deal/paged";
 
         $queryString = build_query_string($params);
@@ -103,7 +103,7 @@ 
     {
         $endpoint = "https://api.hubapi.com/deals/v1/deal/{$dealId}/associations/COMPANY";
 
-        $queryString = build_query_string(['id' => (array)$companyIds]);
+        $queryString = build_query_string(['id' => (array) $companyIds]);
 
         return $this->client->request('put', $endpoint, [], $queryString);
     }
@@ -117,7 +117,7 @@ 
     {
         $endpoint = "https://api.hubapi.com/deals/v1/deal/{$dealId}/associations/COMPANY";
 
-        $queryString = build_query_string(['id' => (array)$companyIds]);
+        $queryString = build_query_string(['id' => (array) $companyIds]);
 
         return $this->client->request('delete', $endpoint, [], $queryString);
     }
@@ -131,7 +131,7 @@ 
     {
         $endpoint = "https://api.hubapi.com/deals/v1/deal/{$dealId}/associations/CONTACT";
 
-        $queryString = build_query_string(['id' => (array)$contactIds]);
+        $queryString = build_query_string(['id' => (array) $contactIds]);
 
         return $this->client->request('put', $endpoint, [], $queryString);
     }
@@ -156,7 +156,7 @@ 
     {
         $endpoint = "https://api.hubapi.com/deals/v1/deal/{$dealId}/associations/CONTACT";
 
-        $queryString = build_query_string(['id' => (array)$contactIds]);
+        $queryString = build_query_string(['id' => (array) $contactIds]);
 
         return $this->client->request('delete', $endpoint, [], $queryString);
     }

Doc Comments +11 added lines, -11 removed lines

@@ -7,7 +7,7 @@ 
 {
     /**
      * @param array $deal Array of deal properties.
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      * @throws HubSpotException
      */
     function create(array $deal)
@@ -22,7 +22,7 @@ 
     /**
      * @param int $id The deal id.
      * @param array $deal The deal properties to update.
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function update($id, array $deal)
     {
@@ -50,7 +50,7 @@ 
 
     /**
      * @param int $id
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function delete($id)
     {
@@ -61,7 +61,7 @@ 
 
     /**
      * @param array $params Optional parameters ['limit', 'offset']
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function getRecentlyModified(array $params = [])
     {
@@ -73,7 +73,7 @@ 
 
     /**
      * @param array $params Optional parameters ['limit', 'offset']
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function getRecentlyCreated(array $params = [])
     {
@@ -85,7 +85,7 @@ 
 
     /**
      * @param int $id
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function getById($id)
     {
@@ -97,7 +97,7 @@ 
     /**
      * @param int $dealId
      * @param int|int[] $companyIds
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function associateWithCompany($dealId, $companyIds)
     {
@@ -111,7 +111,7 @@ 
     /**
      * @param int $dealId
      * @param int|int[] $companyIds
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function disassociateFromCompany($dealId, $companyIds)
     {
@@ -125,7 +125,7 @@ 
     /**
      * @param int $dealId
      * @param int|int[] $contactIds
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function associateWithContact($dealId, $contactIds)
     {
@@ -139,7 +139,7 @@ 
     /**
      * @param int $contactId
      * @param array $params Optional parameters ['limit', 'offset']
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function associatedWithContact($contactId, $params = [])
     {
@@ -153,7 +153,7 @@ 
     /**
      * @param int $dealId
      * @param int|int[] $contactIds
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      */
     function disassociateFromContact($dealId, $contactIds)
     {

src/Resources/Files.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
             [
                 'name' => 'file_names',
                 'contents' => isset($params['file_names']) ? $params['file_names'] : null
-            ],[
+            ], [
                 'name' => 'folder_paths',
                 'contents' => isset($params['folder_paths']) ? $params['folder_paths'] : null
             ]

src/Resources/SingleEmail.php

Indentation +4 added lines, -4 removed lines

@@ -20,10 +20,10 @@
         $endpoint = "https://api.hubapi.com/email/public/v1/singleEmail/send";
 
         $options['json'] = [
-          'emailId'           => $id,
-          'message'           => $message,
-          'contactProperties' => $contactProperties,
-          'customProperties'  => $customProperties
+            'emailId'           => $id,
+            'message'           => $message,
+            'contactProperties' => $contactProperties,
+            'customProperties'  => $customProperties
         ];
 
         return $this->client->request('post', $endpoint, $options);

src/Http/Client.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         $options["headers"]["User-Agent"] = $this->user_agent;
 
         if ($this->oauth2) {
-            $options["headers"]["Authorization"] = "Bearer " . $this->key;
+            $options["headers"]["Authorization"] = "Bearer ".$this->key;
         }
 
         try {

src/Resources/CompanyProperties.php

Spacing +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
     {
         $endpoint = 'https://api.hubapi.com/companies/v2/groups/';
 
-        if($includeProperties){
+        if ($includeProperties) {
             $queryString = build_query_string(['includeProperties' => 'true']);
 
             return $this->client->request('get', $endpoint, [], $queryString);

src/helpers.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('build_query_string')) {
+if (!function_exists('build_query_string')) {
     /**
      * Generate a query string.
      *
@@ -19,7 +19,7 @@ 
             if (is_array($value)) {
                 $query .= build_batch_query_string($key, $value, $encoding);
             } elseif (!empty($value)) {
-                $query .= '&' . url_encode($key, $encoding) . '=' . url_encode($value, $encoding);
+                $query .= '&'.url_encode($key, $encoding).'='.url_encode($value, $encoding);
             }
         }
 
@@ -27,7 +27,7 @@ 
     }
 }
 
-if (! function_exists('build_batch_query_string')) {
+if (!function_exists('build_batch_query_string')) {
     /**
      * Generate a query string for batch requests.
      *
@@ -38,13 +38,13 @@ 
      */
     function build_batch_query_string($key, $items, $encoding = PHP_QUERY_RFC3986)
     {
-        return array_reduce($items, function ($query, $item) use ($key, $encoding) {
-            return $query . "&" . url_encode($key, $encoding) . '=' . url_encode($item, $encoding);
+        return array_reduce($items, function($query, $item) use ($key, $encoding) {
+            return $query."&".url_encode($key, $encoding).'='.url_encode($item, $encoding);
         }, '');
     }
 }
 
-if (! function_exists('url_encode')) {
+if (!function_exists('url_encode')) {
     /**
      * Url encode a string.
      *
@@ -67,7 +67,7 @@ 
     }
 }
 
-if (! function_exists('ms_timestamp')) {
+if (!function_exists('ms_timestamp')) {
     /**
      * Get a millisecond timestamp from a date or time.
      *

src/Resources/Events.php

Indentation +3 added lines, -3 removed lines

@@ -32,9 +32,9 @@
         $required['email'] = $contactEmail;
 
         $parameters = array_merge(
-          $required,
-          ['_m' => $contactRevenue],
-          $contactProperties
+            $required,
+            ['_m' => $contactRevenue],
+            $contactProperties
         );
 
         $query_string = build_query_string($parameters);

src/Resources/Timeline.php

Doc Comments +10 added lines, -10 removed lines

@@ -17,7 +17,7 @@ 
      * @param mixed       $timestamp
      * @param array       $eventTypeData
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/create-or-update-event
      */
@@ -53,7 +53,7 @@ 
      * @param  int      $appId
      * @param  array    $events
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see https://developers.hubspot.com/docs/methods/timeline/batch-create-or-update-events
      */
@@ -71,7 +71,7 @@ 
      *
      * @param int $appId
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/get-event-types
      */
@@ -90,7 +90,7 @@ 
      * @param string|null $detailTemplate
      * @param string|null $objectType
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/create-event-type
      */
@@ -124,7 +124,7 @@ 
      * @param string|null $detailTemplate
      * @param string|null $objectType
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/update-event-type
      */
@@ -155,7 +155,7 @@ 
      * @param int $appId
      * @param int $eventTypeId
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/delete-event-type
      */
@@ -171,7 +171,7 @@ 
      * @param int $appId
      * @param int $eventTypeId
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/get-timeline-event-type-properties
      */
@@ -192,7 +192,7 @@ 
      * @param string|null $objectProperty
      * @param array       $options
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/create-timeline-event-type-property
      */
@@ -229,7 +229,7 @@ 
      * @param string     $propertyType
      * @param array|null $options
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/udpate-timeline-event-type-property
      */
@@ -265,7 +265,7 @@ 
      * @param int $eventTypeId
      * @param int $eventTypePropertyId
      *
-     * @return mixed
+     * @return \Psr\Http\Message\ResponseInterface
      *
      * @see http://developers.hubspot.com/docs/methods/timeline/delete-timeline-event-type-property
      */

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      *
      * @see https://developers.hubspot.com/docs/methods/timeline/batch-create-or-update-events
      */
-    public function createOrUpdateBatch($appId,$events=[])
+    public function createOrUpdateBatch($appId, $events = [])
     {
         $endpoint = "https://api.hubapi.com/integrations/v1/{$appId}/timeline/event/batch";
 

src/Resources/DealProperties.php

Spacing +3 added lines, -3 removed lines

@@ -73,7 +73,7 @@ 
      */
     public function delete($name)
     {
-        $endpoint = 'https://api.hubapi.com/properties/v1/deals/properties/named/' . $name;
+        $endpoint = 'https://api.hubapi.com/properties/v1/deals/properties/named/'.$name;
         return $this->client->request('delete', $endpoint);
     }
 
@@ -153,7 +153,7 @@ 
     {
         $endpoint = "https://api.hubapi.com/properties/v1/deals/groups/named/{$groupName}";
 
-        if($includeProperties){
+        if ($includeProperties) {
             $queryString = build_query_string(['includeProperties' => 'true']);
 
             return $this->client->request('get', $endpoint, [], $queryString);
@@ -174,7 +174,7 @@ 
     {
         $endpoint = 'https://api.hubapi.com/properties/v1/deals/groups';
 
-        if($includeProperties){
+        if ($includeProperties) {
             $queryString = build_query_string(['includeProperties' => 'true']);
 
             return $this->client->request('get', $endpoint, [], $queryString);