Skip to content

Commit d9f1a7f

Browse files
committed
allow setting variable options to "0", closes pterodactyl#87
1 parent 812b869 commit d9f1a7f

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

app/Http/Controllers/Admin/ServiceController.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,11 @@ public function postOptionVariable(Request $request, $service, $option, $variabl
181181

182182
// Because of the way old() works on the display side we prefix all of the variables with thier ID
183183
// We need to remove that prefix here since the repo doesn't want it.
184-
$data = [];
184+
$data = [
185+
'user_viewable' => '0',
186+
'user_editable' => '0',
187+
'required' => '0'
188+
];
185189
foreach($request->except(['_token']) as $id => $val) {
186190
$data[str_replace($variable.'_', '', $id)] = $val;
187191
}

app/Repositories/ServiceRepository/Variable.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,9 @@ public function update($id, array $data)
101101
'description' => 'sometimes|required|string',
102102
'env_variable' => 'sometimes|required|regex:/^[\w]{1,255}$/',
103103
'default_value' => 'sometimes|string|max:255',
104-
'user_viewable' => 'sometimes|required|numeric|size:1',
105-
'user_editable' => 'sometimes|required|numeric|size:1',
106-
'required' => 'sometimes|required|numeric|size:1',
104+
'user_viewable' => 'sometimes|required|numeric|boolean',
105+
'user_editable' => 'sometimes|required|numeric|boolean',
106+
'required' => 'sometimes|required|numeric|boolean',
107107
'regex' => 'sometimes|required|string|min:1'
108108
]);
109109

0 commit comments

Comments
 (0)