diff options
| author | Teddy Wing | 2017-08-30 11:19:06 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-08-30 11:23:22 +0200 |
| commit | 36c65586265bee358b37e824acde91adc7a61a34 (patch) | |
| tree | 694a79145ec3c0eb12e4a534acc5753299103753 | |
| parent | abe1d0d346e9059d46caf62f3bde67df75a07540 (diff) | |
| download | chouette-core-36c65586265bee358b37e824acde91adc7a61a34.tar.bz2 | |
Import#child_change: Remove `child` argument
It isn't used any more so get rid of it. I had forgotten to remove the
argument from the place it was actually called in the code when I did
this previously in 61817b2e7828455021bcec66e2d6da70f879e1c4. Actually,
Alban hadn't modified the method signature in the code he gave me, I
shouldn't have committed that change as him, my bad.
The argument was added back in because it caused `#notify_parent` to
fail. Update that method so it makes the correct method call.
| -rw-r--r-- | app/models/import.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/import.rb b/app/models/import.rb index cdda3d0dc..eb2428b2b 100644 --- a/app/models/import.rb +++ b/app/models/import.rb @@ -29,11 +29,11 @@ class Import < ActiveRecord::Base end def notify_parent - parent.child_change(self) + parent.child_change update(notified_parent_at: DateTime.now) end - def child_change(child) + def child_change return if self.class.finished_statuses.include?(status) update_status |