Changes for Zapier CLI

app/Http/Controllers/AccountApiController.php

@@ -36,8 +36,13 @@ class AccountApiController extends BaseAPIController
     {
         $headers = Utils::getApiHeaders();
 
+        // Legacy support for Zapier 
+        if (request()->v2) {
+            return $this->response(RESULT_SUCCESS);
+        } else {
             return Response::make(RESULT_SUCCESS, 200, $headers);
         }
+    }
 
     public function register(RegisterRequest $request)
     {

app/Http/Controllers/IntegrationController.php

@@ -11,7 +11,7 @@ use Utils;
 /**
  * Class IntegrationController.
  */
-class IntegrationController extends Controller
+class IntegrationController extends BaseAPIController
 {
     /**
      * @return \Illuminate\Http\JsonResponse
@@ -33,6 +33,14 @@ class IntegrationController extends Controller
             return Response::json('Failed to create subscription', 500);
         }
 
-        return Response::json(['id' => $subscription->id], 201);
+        return Response::json(['id' => $subscription->public_id], 201);
+    }
+
+    public function unsubscribe($publicId)
+    {
+        $subscription = Subscription::scope($publicId)->firstOrFail();
+        $subscription->delete();
+
+        return $this->response(RESULT_SUCCESS);
     }
 }

app/Http/Controllers/InvoiceApiController.php

@@ -73,6 +73,10 @@ class InvoiceApiController extends BaseAPIController
             $invoices->where('invoice_status_id', '>=', $statusId);
         }
 
+        if (request()->has('is_recurring')) {
+            $invoices->where('is_recurring', '=', request()->is_recurring);
+        }
+
         return $this->listResponse($invoices);
     }
 
@@ -241,8 +245,6 @@ class InvoiceApiController extends BaseAPIController
             'po_number' => '',
             'invoice_design_id' => $account->invoice_design_id,
             'invoice_items' => [],
-            'custom_value1' => 0,
-            'custom_value2' => 0,
             'custom_taxes1' => false,
             'custom_taxes2' => false,
             'tax_name1' => '',
@@ -274,7 +276,7 @@ class InvoiceApiController extends BaseAPIController
         }
 
         // initialize the line items
-        if (isset($data['product_key']) || isset($data['cost']) || isset($data['notes']) || isset($data['qty'])) {
+        if (! isset($data['invoice_items']) && (isset($data['product_key']) || isset($data['cost']) || isset($data['notes']) || isset($data['qty']))) {
             $data['invoice_items'] = [self::prepareItem($data)];
             // make sure the tax isn't applied twice (for the invoice and the line item)
             unset($data['invoice_items'][0]['tax_name1']);
@@ -337,6 +339,13 @@ class InvoiceApiController extends BaseAPIController
             }
         }
 
+        // Workaround to support line item taxes w/Zapier
+        foreach (['tax_rate1', 'tax_name1', 'tax_rate2', 'tax_name2'] as $field) {
+            if (! empty($item['item_' . $field])) {
+                $item[$field] = $item['item_' . $field];
+            }
+        }
+
         return $item;
     }
 

app/Http/Controllers/TaskApiController.php

@@ -110,6 +110,24 @@ class TaskApiController extends BaseAPIController
             $data['client'] = $data['client_id'];
         }
 
+        if (! empty($data['time_details'])) {
+            $timeLog = [];
+            foreach ($data['time_details'] as $detail) {
+                $startTime = strtotime($detail['start_datetime']);
+                $endTime = false;
+                if (! empty($detail['end_datetime'])) {
+                    $endTime = strtotime($detail['end_datetime']);
+                } elseif (! empty($detail['duration_minutes'])) {
+                    $endTime = $startTime + ($detail['duration_minutes'] * 60);
+                }
+                $timeLog[] = [$startTime, $endTime];
+                if (! $endTime) {
+                    $data['is_running'] = true;
+                }
+            }
+            $data['time_log'] = json_encode($timeLog);
+        }
+
         $task = $this->taskRepo->save($taskId, $data);
         $task = Task::scope($task->public_id)->with('client')->first();
 

app/Models/Client.php

@@ -340,9 +340,17 @@ class Client extends EntityModel
      */
     public function getPrimaryContact()
     {
-        return $this->contacts()
-                    ->whereIsPrimary(true)
-                    ->first();
+        if (! $this->relationLoaded('contacts')) {
+            $this->load('contacts');
+        }
+
+        foreach ($this->contacts as $contact) {
+            if ($contact->is_primary) {
+                return $contact;
+            }
+        }
+
+        return false;
     }
 
     /**
@@ -352,16 +360,10 @@ class Client extends EntityModel
     {
         if ($this->name) {
             return $this->name;
-        }
-
-        if (! $this->contacts->count()) {
-            return '';
-        }
-
-        $contact = $this->contacts[0];
-
+        } else if ($contact = $this->getPrimaryContact()) {
             return $contact->getDisplayName();
         }
+    }
 
     /**
      * @return string

app/Ninja/Repositories/ClientRepository.php

@@ -124,6 +124,17 @@ class ClientRepository extends BaseRepository
             }
         }
 
+        // convert shipping country code to id
+        if (isset($data['shipping_country_code'])) {
+            $countryCode = strtolower($data['shipping_country_code']);
+            $country = Cache::get('countries')->filter(function ($item) use ($countryCode) {
+                return strtolower($item->iso_3166_2) == $countryCode || strtolower($item->iso_3166_3) == $countryCode;
+            })->first();
+            if ($country) {
+                $data['shipping_country_id'] = $country->id;
+            }
+        }
+
         // set default payment terms
         if (auth()->check() && ! isset($data['payment_terms'])) {
             $data['payment_terms'] = auth()->user()->account->payment_terms;

app/Ninja/Repositories/TaskRepository.php

@@ -161,14 +161,16 @@ class TaskRepository extends BaseRepository
 
         $task->fill($data);
 
+        if (! empty($data['project_id'])) {
+            $project = Project::scope($data['project_id'])->firstOrFail();
+            $task->project_id = $project->id;
+            $task->client_id = $project->client_id;
+        } else {
             if (isset($data['client'])) {
                 $task->client_id = $data['client'] ? Client::getPrivateId($data['client']) : null;
             } elseif (isset($data['client_id'])) {
                 $task->client_id = $data['client_id'] ? Client::getPrivateId($data['client_id']) : null;
             }
-
-        if (isset($data['project_id'])) {
-            $task->project_id = $data['project_id'] ? Project::getPrivateId($data['project_id']) : null;
         }
 
         if (isset($data['description'])) {

app/Ninja/Transformers/ClientTransformer.php

@@ -118,6 +118,7 @@ class ClientTransformer extends EntityTransformer
         return array_merge($this->getDefaults($client), [
             'id' => (int) $client->public_id,
             'name' => $client->name,
+            'display_name' => $client->getDisplayName(),
             'balance' => (float) $client->balance,
             'paid_to_date' => (float) $client->paid_to_date,
             'updated_at' => $this->getTimestamp($client->updated_at),

routes/api.php

@@ -17,6 +17,7 @@ Route::resource('payments', 'PaymentApiController');
 Route::resource('tasks', 'TaskApiController');
 Route::resource('credits', 'CreditApiController');
 Route::post('hooks', 'IntegrationController@subscribe');
+Route::delete('hooks/{subscription_id}', 'IntegrationController@unsubscribe');
 Route::post('email_invoice', 'InvoiceApiController@emailInvoice');
 Route::get('user_accounts', 'AccountApiController@getUserAccounts');
 Route::resource('products', 'ProductApiController');