diff options
| author | Teddy Wing | 2017-08-01 21:05:44 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-08-01 21:05:44 +0200 |
| commit | 327975ede5e0c400887152fd0f8112a40aafa005 (patch) | |
| tree | 55d8116bd23bd1b9efef0b9ee56c9e9c377d0e39 /app/controllers/errors_controller.rb | |
| parent | 3dd23ffba6d0ffcc77d010a489e7582532db96e3 (diff) | |
| download | chouette-core-327975ede5e0c400887152fd0f8112a40aafa005.tar.bz2 | |
Import#child_change: Fix bug when parent status already set
Don't update the parent status if it's already been set to a finished
status. This protects against situations like this:
child_import_1 status: :failed
#child_change => parent_import status: :failed
child_import_2 status: :successful
#child_change => parent_import status: :successful
In that scenario, even though one of the parent's children had failed,
it still gets a successful status at the end. To prevent that situation,
don't update the parent's status if it's already been set to a finished
status.
Refs #3509, #3511
Diffstat (limited to 'app/controllers/errors_controller.rb')
0 files changed, 0 insertions, 0 deletions
