From 9e5417ab1c11b9ab25fb8f816b1a247734476976 Mon Sep 17 00:00:00 2001
From: David Bomba <turbo124@gmail.com>
Date: Fri, 13 Jan 2023 20:16:17 +1100
Subject: [PATCH] Define the Client Statement Class

---
 app/DataMapper/Schedule/ClientStatement.php   | 96 +++++++++++++++++++
 .../TaskScheduler/StoreSchedulerRequest.php   |  7 +-
 app/Models/Scheduler.php                      | 12 +--
 tests/Feature/Scheduler/SchedulerTest.php     | 15 ++-
 4 files changed, 121 insertions(+), 9 deletions(-)
 create mode 100644 app/DataMapper/Schedule/ClientStatement.php

diff --git a/app/DataMapper/Schedule/ClientStatement.php b/app/DataMapper/Schedule/ClientStatement.php
new file mode 100644
index 000000000000..e5f51a0051e4
--- /dev/null
+++ b/app/DataMapper/Schedule/ClientStatement.php
@@ -0,0 +1,96 @@
+<?php
+/**
+ * Invoice Ninja (https://invoiceninja.com).
+ *
+ * @link https://github.com/invoiceninja/invoiceninja source repository
+ *
+ * @copyright Copyright (c) 2022. Invoice Ninja LLC (https://invoiceninja.com)
+ *
+ * @license https://www.elastic.co/licensing/elastic-license
+ */
+
+namespace App\DataMapper;
+
+use App\Models\Client;
+use stdClass;
+
+class ClientStatement
+{
+
+    /**
+     * Defines the template name
+     * 
+     * @var string
+     */
+    public string $template = 'client_statement';
+
+    /**
+     * An array of clients hashed_ids
+     *
+     * Leave blank if this action should apply to all clients
+     * 
+     * @var array
+     */
+    public array $clients = [];
+
+    /**
+     * The consts to be used to define the date_range variable of the statement
+     */
+    public const THIS_MONTH = 'this_month';
+    public const THIS_QUARTER = 'this_quarter';
+    public const THIS_YEAR = 'this_year';
+    public const PREVIOUS_MONTH = 'previous_month';
+    public const PREVIOUS_QUARTER = 'previous_quarter';
+    public const PREVIOUS_YEAR = 'previous_year';
+    public const CUSTOM_RANGE = "custom_range";
+
+    /**
+     * The date range the statement should include
+     * 
+     * @var string
+     */
+    public string $date_range = 'this_month';
+
+    /**
+     * If a custom range is select for the date range then
+     * the start_date should be supplied in Y-m-d format
+     * 
+     * @var string
+     */
+    public string $start_date = '';
+
+
+    /**
+     * If a custom range is select for the date range then
+     * the end_date should be supplied in Y-m-d format
+     * 
+     * @var string
+     */
+    public string $end_date = '';
+
+    /**
+     * Flag which allows the payment table
+     * to be shown
+     *
+     * @var boolean
+     */
+    public bool $show_payments_table = true;
+
+    /**
+     * Flag which allows the aging table
+     * to be shown
+     *
+     * @var boolean
+     */
+    public bool $show_aging_table = true;
+
+    /**
+     * String const which defines whether
+     * the invoices to be shown are either
+     * paid or unpaid
+     * 
+     * @var string
+     */
+    public string $status = 'paid'; // paid | unpaid
+
+}
\ No newline at end of file
diff --git a/app/Http/Requests/TaskScheduler/StoreSchedulerRequest.php b/app/Http/Requests/TaskScheduler/StoreSchedulerRequest.php
index 0c63f95a9542..5e9711340386 100644
--- a/app/Http/Requests/TaskScheduler/StoreSchedulerRequest.php
+++ b/app/Http/Requests/TaskScheduler/StoreSchedulerRequest.php
@@ -28,9 +28,8 @@ class StoreSchedulerRequest extends Request
 
     public function rules()
     {
-
+nlog($this->all());
         $rules = [
-            'name' => ['bail', 'required', Rule::unique('schedulers')->where('company_id', auth()->user()->company()->id)],
             'is_paused' => 'bail|sometimes|boolean',
             'frequency_id' => 'bail|required|integer|digits_between:1,12',
             'next_run' => 'bail|required|date:Y-m-d',
@@ -38,6 +37,10 @@ class StoreSchedulerRequest extends Request
             'parameters' => 'bail|array',
         ];
 
+        $rules['name'] = ['bail', 'required', Rule::unique('schedulers')->where('company_id', auth()->user()->company()->id)];
+
+nlog($rules);
+
         return $rules;
 
     }
diff --git a/app/Models/Scheduler.php b/app/Models/Scheduler.php
index 09732f875ae9..93d50a42657e 100644
--- a/app/Models/Scheduler.php
+++ b/app/Models/Scheduler.php
@@ -36,12 +36,12 @@ class Scheduler extends BaseModel
     use SoftDeletes;
 
     protected $fillable = [
-        'start_from',
-        'is_paused',
-        'repeat_every',
+        'name',
+        'frequency_id',
+        'next_run',
         'scheduled_run',
-        'action_class',
-        'action_name',
+        'template',
+        'is_paused',
         'parameters',
     ];
 
@@ -50,7 +50,7 @@ class Scheduler extends BaseModel
         'created_at' => 'timestamp',
         'updated_at' => 'timestamp',
         'deleted_at' => 'timestamp',
-        'paused' => 'boolean',
+        'is_paused' => 'boolean',
         'is_deleted' => 'boolean',
         'parameters' => 'array',
     ];
diff --git a/tests/Feature/Scheduler/SchedulerTest.php b/tests/Feature/Scheduler/SchedulerTest.php
index e5b40a153a0b..13c2c84c0f29 100644
--- a/tests/Feature/Scheduler/SchedulerTest.php
+++ b/tests/Feature/Scheduler/SchedulerTest.php
@@ -69,8 +69,21 @@ class SchedulerTest extends TestCase
             'next_run' => '2023-01-31',
             'template' => 'client_statement',
             'clients' => [],
-
+            'parameters' => [
+                'date_range' => 'last_month',
+                'show_payments_table' => true,
+                'show_aging_table' => true,
+                'status' => 'paid'
+            ],
         ];
+
+        $response = $this->withHeaders([
+            'X-API-SECRET' => config('ninja.api_secret'),
+            'X-API-TOKEN' => $this->token,
+        ])->postJson('/api/v1/task_schedulers', $data);
+
+        $response->assertStatus(200);
+        
     }
 
     public function testDeleteSchedule()