Skip to content

Commit 9106971

Browse files
committed
1 parent da003ef commit 9106971

File tree

2 files changed

+6
-12
lines changed

2 files changed

+6
-12
lines changed

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,12 @@ This project follows [Semantic Versioning](http://semver.org) guidelines.
66
## v0.6.0-pre.8 (Courageous Carniadactylus)
77
### Fixed
88
* `[pre.7]` — Fixes bug with subuser checkbox display.
9+
* `[pre.7]` — Fixes bug with injected JS that was causing `<!DOCTYPE html>` to be ignored in templates.
10+
* `[pre.7]` — Fixes exception thrown when trying to delete a node due to a misnamed model.
911

1012
### Changed
1113
* Subuser permissions are now stored in `Permission::list()` to make views way cleaner and make adding to views significantly cleaner.
14+
* `[pre.7]` — Sidebar for file manager now is a single link rather than a dropdown.
1215

1316
## v0.6.0-pre.7 (Courageous Carniadactylus)
1417
### Fixed

app/Repositories/NodeRepository.php

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -277,13 +277,9 @@ public function delete($id)
277277
throw new DisplayException('You cannot delete a node with servers currently attached to it.');
278278
}
279279

280-
DB::beginTransaction();
281-
282-
try {
280+
DB::transaction(function () use ($node) {
283281
// Unlink Database Servers
284-
Models\DatabaseServer::where('linked_node', $node->id)->update([
285-
'linked_node' => null,
286-
]);
282+
Models\DatabaseHost::where('node_id', $node->id)->update(['node_id' => null]);
287283

288284
// Delete Allocations
289285
Models\Allocation::where('node_id', $node->id)->delete();
@@ -293,11 +289,6 @@ public function delete($id)
293289

294290
// Delete Node
295291
$node->delete();
296-
297-
DB::commit();
298-
} catch (\Exception $ex) {
299-
DB::rollback();
300-
throw $ex;
301-
}
292+
});
302293
}
303294
}

0 commit comments

Comments
 (0)