From c7f070a2d1f914ff32610f0e572a74ac8bedc108 Mon Sep 17 00:00:00 2001 From: James Cole Date: Sun, 24 Aug 2014 17:15:08 +0200 Subject: [PATCH] Remove reminders when removing piggy bank. --- .../Trigger/Piggybanks/EloquentPiggybankTrigger.php | 7 +++++++ app/models/Piggybank.php | 8 ++++++++ 2 files changed, 15 insertions(+) diff --git a/app/lib/Firefly/Trigger/Piggybanks/EloquentPiggybankTrigger.php b/app/lib/Firefly/Trigger/Piggybanks/EloquentPiggybankTrigger.php index a0eef0425f..cb5442842a 100644 --- a/app/lib/Firefly/Trigger/Piggybanks/EloquentPiggybankTrigger.php +++ b/app/lib/Firefly/Trigger/Piggybanks/EloquentPiggybankTrigger.php @@ -218,6 +218,13 @@ class EloquentPiggybankTrigger */ public function destroy(\Piggybank $piggyBank) { + $reminders = $piggyBank->piggybankreminders()->get(); + /** @var \PiggybankReminder $reminder */ + foreach ($reminders as $reminder) { + $reminder->delete(); + + } + return true; } diff --git a/app/models/Piggybank.php b/app/models/Piggybank.php index 826841ebc3..9dfbd8b149 100644 --- a/app/models/Piggybank.php +++ b/app/models/Piggybank.php @@ -221,6 +221,14 @@ class Piggybank extends Ardent return $this->hasMany('PiggybankEvent'); } + /** + * @return \Illuminate\Database\Eloquent\Relations\HasMany + */ + public function piggybankreminders() + { + return $this->hasMany('PiggybankReminder'); + } + /** * Same but for specific date. *