1635: Cleaned up

itk-dev · Jul 5, 2024 · db33c0c · db33c0c
1 parent e33f84f
commit db33c0c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/src/Controller/InvoiceEntryController.php b/src/Controller/InvoiceEntryController.php
@@ -12,7 +12,6 @@
 use App\Service\BillingService;
 use App\Service\ClientHelper;
 use App\Service\InvoiceHelper;
-use App\Service\ViewService;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;

diff --git a/src/Service/InvoiceHelper.php b/src/Service/InvoiceHelper.php
@@ -248,7 +248,7 @@ private function resolveOptions(array $options): array
             ->setAllowedTypes('invoice_issue_far_past_cutoff_date', ['null', 'string'])
             ->setAllowedValues('invoice_issue_far_past_cutoff_date', static function (?string $value) {
                 try {
-                    new \DateTimeImmutable($value);
+                    new \DateTimeImmutable($value ?: '0000-00-00');
 
                     return true;
                 } catch (Exception) {

diff --git a/src/Service/ProjectBillingService.php b/src/Service/ProjectBillingService.php
@@ -39,15 +39,19 @@ public function __construct(
 
     public function getIssuesNotIncludedInProjectBillingFromTheFarPast(ProjectBilling $projectBilling): array
     {
-        $cutoffDate = $this->invoiceHelper->getIssueFarPastCutoffDate();
+        $issues = null;
 
-        $projectBilling = clone $projectBilling;
-        $projectBilling->setPeriodStart(new \DateTimeImmutable('0001-01-01'));
-        $projectBilling->setPeriodEnd($cutoffDate);
+        if ($cutoffDate = $this->invoiceHelper->getIssueFarPastCutoffDate()) {
+            $projectBilling = clone $projectBilling;
+            $projectBilling->setPeriodStart(new \DateTimeImmutable('0001-01-01'));
+            $projectBilling->setPeriodEnd($cutoffDate);
+
+            $issues = $this->getIssuesNotIncludedInProjectBilling($projectBilling);
+        }
 
         return [
             'cutoff_date' => $cutoffDate,
-            'issues' => $this->getIssuesNotIncludedInProjectBilling($projectBilling),
+            'issues' => $issues,
         ];
     }