Fix bug listing allocations when making a new server.

closes pterodactyl#730

Author: DaneEveritt

CHANGELOG.md

@@ -9,6 +9,7 @@ This project follows [Semantic Versioning](http://semver.org) guidelines.
 * `[beta.1]` — Fixes bug that would prevent root admins from accessing servers they were not set as the owner of.
 * `[beta.1]` — Fixes wrong URL redirect being provided when creating a subuser.
 * `[beta.1]` — Fixes missing check in environment setup that would leave the Hashids salt empty.
+* `[beta.1]` — Fixes bug preventing loading of allocations when trying to create a new server.
 
 ## v0.7.0-beta.1 (Derelict Dermodactylus)
 ### Added

app/Contracts/Repository/NodeRepositoryInterface.php

@@ -60,10 +60,9 @@ public function getNodeAllocations($id);
     public function getNodeServers($id);
 
     /**
-     * Return a collection of nodes beloning to a specific location for use on frontend display.
+     * Return a collection of nodes for all locations to use in server creation UI.
      *
-     * @param int $location
      * @return mixed
      */
-    public function getNodesForLocation($location);
+    public function getNodesForServerCreation();
 }

app/Http/Controllers/Admin/ServersController.php

@@ -231,6 +231,7 @@ public function create()
         $nests = $this->nestRepository->getWithEggs();
 
         Javascript::put([
+            'nodeData' => $this->nodeRepository->getNodesForServerCreation(),
             'nests' => $nests->map(function ($item) {
                 return array_merge($item->toArray(), [
                     'eggs' => $item->eggs->keyBy('id')->toArray(),
@@ -262,17 +263,6 @@ public function store(ServerFormRequest $request)
         return redirect()->route('admin.servers.view', $server->id);
     }
 
-    /**
-     * Returns a tree of all avaliable nodes in a given location.
-     *
-     * @param \Illuminate\Http\Request $request
-     * @return \Illuminate\Support\Collection
-     */
-    public function nodes(Request $request)
-    {
-        return $this->nodeRepository->getNodesForLocation($request->input('location'));
-    }
-
     /**
      * Display the index when viewing a specific server.
      *

app/Repositories/Eloquent/NodeRepository.php

@@ -129,9 +129,9 @@ public function getNodeServers($id)
     /**
      * {@inheritdoc}
      */
-    public function getNodesForLocation($location)
+    public function getNodesForServerCreation()
     {
-        $instance = $this->getBuilder()->with('allocations')->where('location_id', $location)->get();
+        $instance = $this->getBuilder()->with('allocations')->get();
 
         return $instance->map(function ($item) {
             $filtered = $item->allocations->where('server_id', null)->map(function ($map) {

public/themes/pterodactyl/js/admin/new-server.js

@@ -29,7 +29,7 @@ $(document).ready(function() {
     });
     $('#pNodeId').select2({
         placeholder: 'Select a Node',
-    });
+    }).change();
     $('#pAllocation').select2({
         placeholder: 'Select a Default Allocation',
     });
@@ -79,14 +79,6 @@ $(document).ready(function() {
     });
 });
 
-function hideLoader() {
-    $('#allocationLoader').hide();
-}
-
-function showLoader() {
-    $('#allocationLoader').show();
-}
-
 var lastActiveBox = null;
 $(document).on('click', function (event) {
     if (lastActiveBox !== null) {
@@ -97,12 +89,9 @@ $(document).on('click', function (event) {
     lastActiveBox.addClass('box-primary');
 });
 
-var curentNode = null;
-var NodeData = [];
-
-$('#pNodeId').on('change', function (event) {
+$('#pNodeId').on('change', function () {
     currentNode = $(this).val();
-    $.each(NodeData, function (i, v) {
+    $.each(Pterodactyl.nodeData, function (i, v) {
         if (v.id == currentNode) {
             $('#pAllocation').html('').select2({
                 data: v.allocations,

routes/admin.php

@@ -95,7 +95,6 @@
     Route::get('/view/{server}/delete', 'ServersController@viewDelete')->name('admin.servers.view.delete');
 
     Route::post('/new', 'ServersController@store');
-    Route::post('/new/nodes', 'ServersController@nodes')->name('admin.servers.new.nodes');
     Route::post('/view/{server}/build', 'ServersController@updateBuild');
     Route::post('/view/{server}/startup', 'ServersController@saveStartup');
     Route::post('/view/{server}/database', 'ServersController@newDatabase');