From e42547a1ff8b5e7df4e8c022c5da74005d3ea542 Mon Sep 17 00:00:00 2001
From: Dane Everitt <dane@daneeveritt.com>
Date: Sat, 20 Feb 2016 15:59:37 -0500
Subject: [PATCH] add support for editing service options

---
 .../Controllers/Admin/ServiceController.php   |  14 ++-
 app/Repositories/ServiceRepository/Option.php |  24 +++++
 .../admin/services/options/view.blade.php     | 101 ++++++++++++++----
 3 files changed, 116 insertions(+), 23 deletions(-)

diff --git a/app/Http/Controllers/Admin/ServiceController.php b/app/Http/Controllers/Admin/ServiceController.php
index 981a8b326..ed475660a 100644
--- a/app/Http/Controllers/Admin/ServiceController.php
+++ b/app/Http/Controllers/Admin/ServiceController.php
@@ -107,7 +107,19 @@ class ServiceController extends Controller
 
     public function postOption(Request $request, $option)
     {
-        // editing option
+        try {
+            $repo = new ServiceRepository\Option;
+            $repo->update($option, $request->except([
+                '_token'
+            ]));
+            Alert::success('Option settings successfully updated.')->flash();
+        } catch (DisplayValidationException $ex) {
+            return redirect()->route('admin.services.option', $option)->withErrors(json_decode($ex->getMessage()))->withInput();
+        } catch (\Exception $ex) {
+            Log::error($ex);
+            Alert::danger('An error occured while attempting to modify this option.')->flash();
+        }
+        return redirect()->route('admin.services.option', $option)->withInput();
     }
 
     public function postOptionVariable(Request $request, $option, $variable)
diff --git a/app/Repositories/ServiceRepository/Option.php b/app/Repositories/ServiceRepository/Option.php
index 0b4d705c8..92c4dd369 100644
--- a/app/Repositories/ServiceRepository/Option.php
+++ b/app/Repositories/ServiceRepository/Option.php
@@ -42,7 +42,31 @@ class Option
 
     public function update($id, array $data)
     {
+        $option = Models\ServiceOptions::findOrFail($id);
 
+        $validator = Validator::make($data, [
+            'name' => 'sometimes|required|string|max:255',
+            'description' => 'sometimes|required|string|min:1',
+            'tag' => 'sometimes|required|string|max:255',
+            'executable' => 'sometimes|string|max:255',
+            'docker_image' => 'sometimes|required|string|max:255',
+            'startup' => 'sometimes|string'
+        ]);
+
+        if ($validator->fails()) {
+            throw new DisplayValidationException($validator->errors());
+        }
+
+        if (isset($data['executable']) && empty($data['executable'])) {
+            $data['executable'] = null;
+        }
+
+        if (isset($data['startup']) && empty($data['startup'])) {
+            $data['startup'] = null;
+        }
+
+        $option->fill($data);
+        $option->save();
     }
 
 }
diff --git a/resources/views/admin/services/options/view.blade.php b/resources/views/admin/services/options/view.blade.php
index 1eb51173f..aca4d83e1 100644
--- a/resources/views/admin/services/options/view.blade.php
+++ b/resources/views/admin/services/options/view.blade.php
@@ -31,28 +31,64 @@
         <li><a href="{{ route('admin.services.service', $service->id) }}">{{ $service->name }}</a></li>
         <li class="active">{{ $option->name }}</li>
     </ul>
-    <h3 class="nopad">Servers</h3>
-    <table class="table table-bordered table-hover">
-        <thead>
-            <tr>
-                <th>Name</th>
-                <th>Owner</th>
-                <th>Connection</th>
-                <th>Updated</th>
-            </tr>
-        </thead>
-        <tbody>
-            @foreach ($servers as $server)
-                <tr>
-                    <td><a href="{{ route('admin.servers.view', $server->id) }}">{{ $server->name }}</a></td>
-                    <td><a href="{{ route('admin.accounts.view', $server->owner) }}">{{ $server->a_ownerEmail }}</a></td>
-                    <td><code>{{ $server->ip }}:{{ $server->port }}</code></td>
-                    <td>{{ $server->updated_at }}</td>
-                </tr>
-            @endforeach
-        </tbody>
-    </table>
-    <h3 class="nopad">Option Variables</h3><hr />
+    <div class="alert alert-warning"><strong>Warning!</strong> This page contains advanced settings that the panel and daemon use to control servers. Modifying information on this page is not recommended unless you are absolutely sure of what you are doing.</div>
+    <h3>Settings</h3><hr />
+    <form action="{{ route('admin.services.option', $option->id) }}" method="POST">
+        <div class="row">
+            <div class="col-md-6 form-group">
+                <label class="control-label">Name:</label>
+                <div>
+                    <input type="text" name="name" value="{{ old('name', $option->name) }}" class="form-control" />
+                </div>
+            </div>
+            <div class="col-md-6 form-group">
+                <label class="control-label">Description:</label>
+                <div>
+                    <textarea name="description" class="form-control" rows="3">{{ old('description', $option->description) }}</textarea>
+                </div>
+            </div>
+        </div>
+        <div class="row">
+            <div class="col-md-3 form-group">
+                <label class="control-label">Tag:</label>
+                <div>
+                    <input type="text" name="tag" value="{{ old('tag', $option->tag) }}" class="form-control" />
+                </div>
+            </div>
+            <div class="col-md-3 form-group">
+                <label class="control-label">Executable:</label>
+                <div>
+                    <input type="text" name="executable" value="{{ old('executable', $option->executable) }}" class="form-control" />
+                    <p class="text-muted"><small>Leave blank to use parent executable.</small></p>
+                </div>
+            </div>
+            <div class="col-md-6 form-group">
+                <label class="control-label">Docker Image:</label>
+                <div>
+                    <input type="text" name="docker_image" value="{{ old('docker_image', $option->docker_image) }}" class="form-control" />
+                    <p class="text-muted"><small>Changing the docker image will only effect servers created or modified after this point.</small></p>
+                </div>
+            </div>
+        </div>
+        <div class="row">
+            <div class="col-md-12 form-group">
+                <label class="control-label">Default Startup Command:</label>
+                <div>
+                    <input type="text" name="startup" value="{{ old('startup', $option->startup) }}" class="form-control" />
+                    <p class="text-muted"><small>To use the default startup of the parent service simply leave this field blank.</small></p>
+                </div>
+            </div>
+        </div>
+        <div class="well well-sm">
+            <div class="row">
+                <div class="col-md-12">
+                    {!! csrf_field() !!}
+                    <input type="submit" class="btn btn-sm btn-primary" value="Update Service Option" />
+                </div>
+            </div>
+        </div>
+    </form>
+    <h3>Variables</h3><hr />
     @foreach($variables as $variable)
     <form action="{{ route('admin.services.option.variable', [ 'option' => $option->id, 'variable' => $variable->id ]) }}" method="POST">
         <div class="well">
@@ -128,6 +164,27 @@
         </div>
     </form>
     @endforeach
+    <h3>Servers</h3><hr />
+    <table class="table table-bordered table-hover">
+        <thead>
+            <tr>
+                <th>Name</th>
+                <th>Owner</th>
+                <th>Connection</th>
+                <th>Updated</th>
+            </tr>
+        </thead>
+        <tbody>
+            @foreach ($servers as $server)
+                <tr>
+                    <td><a href="{{ route('admin.servers.view', $server->id) }}">{{ $server->name }}</a></td>
+                    <td><a href="{{ route('admin.accounts.view', $server->owner) }}">{{ $server->a_ownerEmail }}</a></td>
+                    <td><code>{{ $server->ip }}:{{ $server->port }}</code></td>
+                    <td>{{ $server->updated_at }}</td>
+                </tr>
+            @endforeach
+        </tbody>
+    </table>
 </div>
 <script>
 $(document).ready(function () {