From 0712f30a5141cc00d2299b4a718ce143cf82cb3f Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Sat, 14 Jul 2018 15:16:32 +0200
Subject: [PATCH] Fix bug in API: missing destroy budget limit method.

---
 app/Repositories/Budget/BudgetRepository.php       | 14 ++++++++++++++
 .../Budget/BudgetRepositoryInterface.php           |  7 +++++++
 2 files changed, 21 insertions(+)

diff --git a/app/Repositories/Budget/BudgetRepository.php b/app/Repositories/Budget/BudgetRepository.php
index 6cc8bd41eb..df207daba6 100644
--- a/app/Repositories/Budget/BudgetRepository.php
+++ b/app/Repositories/Budget/BudgetRepository.php
@@ -196,6 +196,20 @@ class BudgetRepository implements BudgetRepositoryInterface
         }
     }
 
+    /**
+     * Destroy a budget limit.
+     *
+     * @param BudgetLimit $budgetLimit
+     */
+    public function destroyBudgetLimit(BudgetLimit $budgetLimit): void
+    {
+        try {
+            $budgetLimit->delete();
+        } catch (Exception $e) {
+            Log::info(sprintf('Could not delete budget limit: %s', $e->getMessage()));
+        }
+    }
+
     /**
      * Filters entries from the result set generated by getBudgetPeriodReport.
      *
diff --git a/app/Repositories/Budget/BudgetRepositoryInterface.php b/app/Repositories/Budget/BudgetRepositoryInterface.php
index 229bee793a..68c94efcf3 100644
--- a/app/Repositories/Budget/BudgetRepositoryInterface.php
+++ b/app/Repositories/Budget/BudgetRepositoryInterface.php
@@ -46,6 +46,13 @@ interface BudgetRepositoryInterface
      */
     public function budgetedPerDay(Budget $budget): string;
 
+    /**
+     * Destroy a budget limit.
+     *
+     * @param BudgetLimit $budgetLimit
+     */
+    public function destroyBudgetLimit(BudgetLimit $budgetLimit): void;
+
     /**
      * @return bool
      */