Don't require auto-allocation settings if not enabled; closes pterodactyl#3085

Author: DaneEveritt

app/Http/Requests/Admin/Settings/AdvancedSettingsFormRequest.php

@@ -20,8 +20,19 @@ public function rules()
             'pterodactyl:guzzle:timeout' => 'required|integer|between:1,60',
             'pterodactyl:guzzle:connect_timeout' => 'required|integer|between:1,60',
             'pterodactyl:client_features:allocations:enabled' => 'required|in:true,false',
-            'pterodactyl:client_features:allocations:range_start' => 'required|integer|between:1024,65535',
-            'pterodactyl:client_features:allocations:range_end' => 'required|integer|between:1024,65535',
+            'pterodactyl:client_features:allocations:range_start' => [
+                'nullable',
+                'required_if:pterodactyl:client_features:allocations:enabled,true',
+                'integer',
+                'between:1024,65535',
+            ],
+            'pterodactyl:client_features:allocations:range_end' => [
+                'nullable',
+                'required_if:pterodactyl:client_features:allocations:enabled,true',
+                'integer',
+                'between:1024,65535',
+                'gt:pterodactyl:client_features:allocations:range_start',
+            ],
         ];
     }
 

resources/views/admin/settings/advanced.blade.php

@@ -101,14 +101,14 @@
                             <div class="form-group col-md-4">
                                 <label class="control-label">Starting Port</label>
                                 <div>
-                                    <input type="number" required class="form-control" name="pterodactyl:client_features:allocations:range_start" value="{{ old('pterodactyl:client_features:allocations:range_start', config('pterodactyl.client_features.allocations.range_start')) }}">
+                                    <input type="number" class="form-control" name="pterodactyl:client_features:allocations:range_start" value="{{ old('pterodactyl:client_features:allocations:range_start', config('pterodactyl.client_features.allocations.range_start')) }}">
                                     <p class="text-muted small">The starting port in the range that can be automatically allocated.</p>
                                 </div>
                             </div>
                             <div class="form-group col-md-4">
                                 <label class="control-label">Ending Port</label>
                                 <div>
-                                    <input type="number" required class="form-control" name="pterodactyl:client_features:allocations:range_end" value="{{ old('pterodactyl:client_features:allocations:range_end', config('pterodactyl.client_features.allocations.range_end')) }}">
+                                    <input type="number" class="form-control" name="pterodactyl:client_features:allocations:range_end" value="{{ old('pterodactyl:client_features:allocations:range_end', config('pterodactyl.client_features.allocations.range_end')) }}">
                                     <p class="text-muted small">The ending port in the range that can be automatically allocated.</p>
                                 </div>
                             </div>