From 4f3629fc4c0992a9a94beaf3daf260c5256923b1 Mon Sep 17 00:00:00 2001
From: Dane Everitt <dane@daneeveritt.com>
Date: Sun, 17 Jan 2016 22:54:03 -0500
Subject: [PATCH] Fixes issue with special characters in URL for file manager

closes #21
---
 app/Repositories/Daemon/FileRepository.php  |  2 +-
 resources/views/server/files/edit.blade.php |  4 ++--
 resources/views/server/files/list.blade.php | 12 ++++++------
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/app/Repositories/Daemon/FileRepository.php b/app/Repositories/Daemon/FileRepository.php
index 9f32029bdf..b34b533498 100644
--- a/app/Repositories/Daemon/FileRepository.php
+++ b/app/Repositories/Daemon/FileRepository.php
@@ -143,7 +143,7 @@ public function returnDirectoryListing($directory)
             throw new Exception('A valid directory must be specified in order to list its contents.');
         }
 
-        $res = $this->client->request('GET', '/server/directory/' . $directory, [
+        $res = $this->client->request('GET', '/server/directory/' . rawurlencode($directory), [
             'headers' => $this->headers
         ]);
 
diff --git a/resources/views/server/files/edit.blade.php b/resources/views/server/files/edit.blade.php
index bc412789bf..e78375599d 100644
--- a/resources/views/server/files/edit.blade.php
+++ b/resources/views/server/files/edit.blade.php
@@ -29,8 +29,8 @@
                 <div>
                     <input type="hidden" name="file" value="{{ $file }}" />
                     {!! csrf_field() !!}
-                    <button class="btn btn-primary btn-sm" id="save_file">{{ trans('strings.save') }}</button>
-                    <button class="btn btn-default btn-sm" onclick="window.location='/server/{{ $server->uuidShort }}/files?dir={{ urlencode($directory) }}';return false;">{{ trans('server.files.back') }}</button>
+                    <button class="btn btn-primary btn-sm" id="save_file" type="submit">{{ trans('strings.save') }}</button>
+                    <a href="/server/{{ $server->uuidShort }}/files?dir={{ rawurlencode($directory) }}"><button class="btn btn-default btn-sm">{{ trans('server.files.back') }}</button></a>
                 </div>
             </div>
         @endcan
diff --git a/resources/views/server/files/list.blade.php b/resources/views/server/files/list.blade.php
index 5877e65ffa..873a80fbb5 100644
--- a/resources/views/server/files/list.blade.php
+++ b/resources/views/server/files/list.blade.php
@@ -22,7 +22,7 @@
         @if (isset($directory['show']) && $directory['show'] === true)
             <tr>
                 <td><i class="fa fa-folder-open" style="margin-left: 0.859px;"></i></td>
-                <td><a href="/server/{{ $server->uuidShort }}/files?dir={{ $directory['link'] }}" class="load_new">&larr; {{ $directory['link_show'] }}</a></a></td>
+                <td><a href="/server/{{ $server->uuidShort }}/files?dir={{ rawurlencode($directory['link']) }}" class="load_new">&larr; {{ $directory['link_show'] }}</a></a></td>
                 <td></td>
                 <td></td>
                 <td></td>
@@ -31,7 +31,7 @@
         @foreach ($folders as $folder)
             <tr>
                 <td><i class="fa fa-folder-open" style="margin-left: 0.859px;"></i></td>
-                <td><a href="/server/{{ $server->uuidShort }}/files?dir=/@if($folder['directory'] !== ''){{ $folder['directory'] }}/@endif{{ $folder['entry'] }}" class="load_new">{{ $folder['entry'] }}</a></td>
+                <td><a href="/server/{{ $server->uuidShort }}/files?dir=/@if($folder['directory'] !== ''){{ rawurlencode($folder['directory']) }}/@endif{{ rawurlencode($folder['entry']) }}" class="load_new">{{ $folder['entry'] }}</a></td>
                 <td>{{ $folder['size'] }}</td>
                 <td>{{ date('m/d/y H:i:s', $folder['date']) }}</td>
                 <td style="text-align:center;">
@@ -41,7 +41,7 @@
                         </div>
                         <div class="col-md-3">
                             @can('delete-files', $server)
-                                <a href="@if($folder['directory'] !== ''){{ $folder['directory'] }}/@endif{{ $folder['entry'] }}" class="delete_file"><span class="badge label-danger"><i class="fa fa-trash-o"></i></span></a>
+                                <a href="@if($folder['directory'] !== ''){{ rawurlencode($folder['directory']) }}/@endif{{ rawurlencode($folder['entry']) }}" class="delete_file"><span class="badge label-danger"><i class="fa fa-trash-o"></i></span></a>
                             @endcan
                         </div>
                     </div>
@@ -54,7 +54,7 @@
                 <td>
                     @if(in_array($file['extension'], $extensions))
                         @can('edit-files', $server)
-                            <a href="/server/{{ $server->uuidShort }}/files/edit/@if($file['directory'] !== ''){{ $file['directory'] }}/@endif{{ $file['entry'] }}" class="edit_file">{{ $file['entry'] }}</a>
+                            <a href="/server/{{ $server->uuidShort }}/files/edit/@if($file['directory'] !== ''){{ rawurlencode($file['directory']) }}/@endif{{ rawurlencode($file['entry']) }}" class="edit_file">{{ $file['entry'] }}</a>
                         @else
                             {{ $file['entry'] }}
                         @endcan
@@ -70,12 +70,12 @@
                         </div>
                         <div class="col-md-3 hidden-xs hidden-sm">
                             @can('download-files', $server)
-                                <a href="/server/{{ $server->uuidShort }}/files/download/@if($file['directory'] !== ''){{ $file['directory'] }}/@endif{{ $file['entry'] }}"><span class="badge"><i class="fa fa-download"></i></span></a>
+                                <a href="/server/{{ $server->uuidShort }}/files/download/@if($file['directory'] !== ''){{ rawurlencode($file['directory']) }}/@endif{{ rawurlencode($file['entry']) }}"><span class="badge"><i class="fa fa-download"></i></span></a>
                             @endcan
                         </div>
                         <div class="col-md-3">
                             @can('delete-files', $server)
-                                <a href="@if($file['directory'] !== ''){{ $file['directory'] }}/@endif{{ $file['entry'] }}" class="delete_file"><span class="badge label-danger"><i class="fa fa-trash-o"></i></span>
+                                <a href="@if($file['directory'] !== ''){{ rawurlencode($file['directory']) }}/@endif{{ rawurlencode($file['entry']) }}" class="delete_file"><span class="badge label-danger"><i class="fa fa-trash-o"></i></span>
                             @endcan
                         </div>
                     </div>