Update: Prevent transaction dates at the beginning of the first day o…

…f the reporting period

src/Exceptions/InvalidTransactionDate.php

@@ -0,0 +1,41 @@
+<?php
+
+/**
+ * Eloquent IFRS Accounting
+ *
+ * @author    Edward Mungai
+ * @copyright Edward Mungai, 2021, Germany
+ * @license   MIT
+ */
+
+namespace IFRS\Exceptions;
+
+use Carbon\Carbon;
+
+use Illuminate\Support\Facades\Log;
+use Illuminate\Support\Facades\Auth;
+
+class InvalidTransactionDate extends IFRSException
+{
+
+    /**
+     * Invalid Transaction Date Exception
+     *
+     * @param string $message
+     * @param int    $code
+     */
+    public function __construct(string $message = null, int $code = null)
+    {
+        $error = "Transaction date cannot be at the beginning of the first day of the Reporting Period. Use a Balance object instead ";
+
+        Log::notice(
+            $error . $message,
+            [
+                'user_id' => Auth::user()->id,
+                'time' => Carbon::now(),
+            ]
+        );
+
+        parent::__construct($error . ' ' . $message, $code);
+    }
+}

src/Models/Transaction.php

@@ -33,6 +33,7 @@
 use IFRS\Exceptions\UnpostedAssignment;
 use IFRS\Exceptions\RedundantTransaction;
 use IFRS\Exceptions\ClosedReportingPeriod;
+use IFRS\Exceptions\InvalidTransactionDate;
 use IFRS\Exceptions\AdjustingReportingPeriod;
 
 /**
@@ -586,6 +587,10 @@ public function save(array $options = []): bool
     {
         $period = ReportingPeriod::getPeriod(Carbon::parse($this->transaction_date));
 
+        if (ReportingPeriod::periodStart($this->transaction_date)->eq(Carbon::parse($this->transaction_date))) {
+            throw new InvalidTransactionDate();
+        }
+
         if ($period->status == ReportingPeriod::CLOSED) {
             throw new ClosedReportingPeriod($period->calendar_year);
         }

tests/Unit/TransactionTest.php

@@ -4,13 +4,16 @@
 
 use Carbon\Carbon;
 use IFRS\User;
+
 use IFRS\Exceptions\AdjustingReportingPeriod;
 use IFRS\Exceptions\ClosedReportingPeriod;
 use IFRS\Exceptions\HangingClearances;
 use IFRS\Exceptions\MissingLineItem;
 use IFRS\Exceptions\PostedTransaction;
 use IFRS\Exceptions\RedundantTransaction;
 use IFRS\Exceptions\UnpostedAssignment;
+use IFRS\Exceptions\InvalidTransactionDate;
+
 use IFRS\Models\Account;
 use IFRS\Models\Assignment;
 use IFRS\Models\Currency;
@@ -21,9 +24,11 @@
 use IFRS\Models\ReportingPeriod;
 use IFRS\Models\Transaction;
 use IFRS\Models\Vat;
+
 use IFRS\Tests\TestCase;
 use IFRS\Transactions\ClientInvoice;
 use IFRS\Transactions\JournalEntry;
+
 use Illuminate\Support\Facades\DB;
 
 class TransactionTest extends TestCase
@@ -1004,4 +1009,20 @@ public function testTransactionPredicates()
         $this->assertFalse($cleared->assignable);
         $this->assertTrue($cleared->clearable);
     }
+
+    /**
+     * Test Invalid Transaction Date.
+     *
+     * @return void
+     */
+    public function testInvalidTransactionDate()
+    {
+
+        $this->expectException(InvalidTransactionDate::class);
+        $this->expectExceptionMessage('Transaction date cannot be at the beginning of the first day of the Reporting Period. Use a Balance object instead');
+
+        Transaction::create([
+            'transaction_date' => Carbon::parse(date('Y').'-01-01'),
+        ]);
+    }
 }