From 3ca7e4d578342b0dde8aba53c08c314696b6e869 Mon Sep 17 00:00:00 2001 From: Dane Everitt Date: Wed, 31 Aug 2016 16:09:23 -0400 Subject: [PATCH] prevent daemon call until confirming DB has been updated closes #71 --- app/Repositories/ServerRepository.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/Repositories/ServerRepository.php b/app/Repositories/ServerRepository.php index 8a815958c..21be6597a 100644 --- a/app/Repositories/ServerRepository.php +++ b/app/Repositories/ServerRepository.php @@ -504,6 +504,10 @@ class ServerRepository $server->io = $data['io']; } + // Try save() here so if it fails we haven't contacted the daemon + // This won't be committed unless the HTTP request succeedes anyways + $server->save(); + $node = Models\Node::getByID($server->node); $client = Models\Node::guzzleRequest($server->node); @@ -527,7 +531,7 @@ class ServerRepository ] ] ]); - $server->save(); + DB::commit(); return true; } catch (\GuzzleHttp\Exception\TransferException $ex) {