Laravel-Backpack / BackupManager

src/app/Http/Controllers/BackupController.php

Unused Use Statements +2 added lines, -6 removed lines

@@ -1,14 +1,10 @@
 <?php namespace Backpack\BackupManager\app\Http\Controllers;
 
-use App\Http\Requests;
 use App\Http\Controllers\Controller;
-use Illuminate\Http\Request;
-use Auth;
-use App;
-use Storage;
-use Carbon\Carbon;
 use Artisan;
+use Illuminate\Http\Request;
 use Log;
+use Storage;
 
 class BackupController extends Controller {
 

Indentation +97 added lines, -97 removed lines

@@ -12,101 +12,101 @@
 
 class BackupController extends Controller {
 
-	public function index()
-	{
-		if (!count(config('laravel-backup.backup.destination.disks'))) {
-			dd(trans('backpack::backup.no_disks_configured'));
-		}
-
-		$this->data['backups'] = [];
-
-		foreach (config('laravel-backup.backup.destination.disks') as $disk_name) {
-			$disk = Storage::disk($disk_name);
-			$adapter = $disk->getDriver()->getAdapter();
-			$files = $disk->files();
-
-			// make an array of backup files, with their filesize and creation date
-			foreach ($files as $k => $f) {
-				// only take the zip files into account
-				if (substr($f, -4) == '.zip' && $disk->exists($f)) {
-					$this->data['backups'][] = [
-						'file_path' => $f,
-						'file_name' => str_replace('backups/', '', $f),
-						'file_size' => $disk->size($f),
-						'last_modified' => $disk->lastModified($f),
-						'disk' => $disk_name,
-						'download' => ($adapter instanceof \League\Flysystem\Adapter\Local)?true:false
-						];
-				}
-			}
-		}
-
-		// reverse the backups, so the newest one would be on top
-		$this->data['backups'] = array_reverse($this->data['backups']);
-		$this->data['title'] = 'Backups';
-
-		return view("backupmanager::backup", $this->data);
-	}
-
-	public function create()
-	{
-	    try {
-	      ini_set('max_execution_time', 300);
-	      // start the backup process
-	      Artisan::call('backup:run');
-	      $output = Artisan::output();
-
-	      // log the results
-	      Log::info("Backpack\BackupManager -- new backup started from admin interface \r\n".$output);
-	      // return the results as a response to the ajax call
-	      echo($output);
-	    } catch (Exception $e) {
-	      Response::make($e->getMessage(), 500);
-	    }
-
-	    return 'success';
-	}
-
-	/**
-	 * Downloads a backup zip file.
-	 */
-	public function download($file_name)
-	{
-		$disk = Storage::disk(\Request::input('disk'));
-		$adapter = $disk->getDriver()->getAdapter();
-
-		if ($adapter instanceof \League\Flysystem\Adapter\Local) {
-			$storage_path = $disk->getDriver()->getAdapter()->getPathPrefix();
-
-			if ($disk->exists($file_name)) {
-				return response()->download($storage_path.$file_name);
-			}
-			else
-			{
-				abort(404, trans('backpack::backup.backup_doesnt_exist'));
-			}
-		}
-		else
-		{
-			abort(404, trans('backpack::backup.only_local_downloads_supported'));
-		}
-	}
-
-	/**
-	 * Deletes a backup file.
-	 */
-	public function delete($file_name)
-	{
-		$disk = Storage::disk(\Request::input('disk'));
-
-		if ($disk->exists($file_name)) {
-			$disk->delete($file_name);
-
-			return 'success';
-		}
-		else
-		{
-			abort(404, trans('backpack::backup.backup_doesnt_exist'));
-		}
-	}
+    public function index()
+    {
+        if (!count(config('laravel-backup.backup.destination.disks'))) {
+            dd(trans('backpack::backup.no_disks_configured'));
+        }
+
+        $this->data['backups'] = [];
+
+        foreach (config('laravel-backup.backup.destination.disks') as $disk_name) {
+            $disk = Storage::disk($disk_name);
+            $adapter = $disk->getDriver()->getAdapter();
+            $files = $disk->files();
+
+            // make an array of backup files, with their filesize and creation date
+            foreach ($files as $k => $f) {
+                // only take the zip files into account
+                if (substr($f, -4) == '.zip' && $disk->exists($f)) {
+                    $this->data['backups'][] = [
+                        'file_path' => $f,
+                        'file_name' => str_replace('backups/', '', $f),
+                        'file_size' => $disk->size($f),
+                        'last_modified' => $disk->lastModified($f),
+                        'disk' => $disk_name,
+                        'download' => ($adapter instanceof \League\Flysystem\Adapter\Local)?true:false
+                        ];
+                }
+            }
+        }
+
+        // reverse the backups, so the newest one would be on top
+        $this->data['backups'] = array_reverse($this->data['backups']);
+        $this->data['title'] = 'Backups';
+
+        return view("backupmanager::backup", $this->data);
+    }
+
+    public function create()
+    {
+        try {
+            ini_set('max_execution_time', 300);
+            // start the backup process
+            Artisan::call('backup:run');
+            $output = Artisan::output();
+
+            // log the results
+            Log::info("Backpack\BackupManager -- new backup started from admin interface \r\n".$output);
+            // return the results as a response to the ajax call
+            echo($output);
+        } catch (Exception $e) {
+            Response::make($e->getMessage(), 500);
+        }
+
+        return 'success';
+    }
+
+    /**
+     * Downloads a backup zip file.
+     */
+    public function download($file_name)
+    {
+        $disk = Storage::disk(\Request::input('disk'));
+        $adapter = $disk->getDriver()->getAdapter();
+
+        if ($adapter instanceof \League\Flysystem\Adapter\Local) {
+            $storage_path = $disk->getDriver()->getAdapter()->getPathPrefix();
+
+            if ($disk->exists($file_name)) {
+                return response()->download($storage_path.$file_name);
+            }
+            else
+            {
+                abort(404, trans('backpack::backup.backup_doesnt_exist'));
+            }
+        }
+        else
+        {
+            abort(404, trans('backpack::backup.only_local_downloads_supported'));
+        }
+    }
+
+    /**
+     * Deletes a backup file.
+     */
+    public function delete($file_name)
+    {
+        $disk = Storage::disk(\Request::input('disk'));
+
+        if ($disk->exists($file_name)) {
+            $disk->delete($file_name);
+
+            return 'success';
+        }
+        else
+        {
+            abort(404, trans('backpack::backup.backup_doesnt_exist'));
+        }
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 						'file_size' => $disk->size($f),
 						'last_modified' => $disk->lastModified($f),
 						'disk' => $disk_name,
-						'download' => ($adapter instanceof \League\Flysystem\Adapter\Local)?true:false
+						'download' => ($adapter instanceof \League\Flysystem\Adapter\Local) ? true : false
 						];
 				}
 			}

Braces +3 added lines, -6 removed lines

@@ -80,13 +80,11 @@ 
 
 			if ($disk->exists($file_name)) {
 				return response()->download($storage_path.$file_name);
-			}
-			else
+			} else
 			{
 				abort(404, trans('backpack::backup.backup_doesnt_exist'));
 			}
-		}
-		else
+		} else
 		{
 			abort(404, trans('backpack::backup.only_local_downloads_supported'));
 		}
@@ -103,8 +101,7 @@ 
 			$disk->delete($file_name);
 
 			return 'success';
-		}
-		else
+		} else
 		{
 			abort(404, trans('backpack::backup.backup_doesnt_exist'));
 		}

src/resources/lang/ro/backup.php

Indentation +1 added lines, -1 removed lines

@@ -33,6 +33,6 @@
     'create_error_message'             => 'Copia de siguranță NU a putut fi creată.',
     'create_warning_title'             => 'Eroare necunoscută',
     'create_warning_message'           => 'Copia de siguranță e posibil să NU fi fost creată. Verificați fișierele de log pentru detalii.',
-	'location' 						   => 'Locație',
+    'location' 						   => 'Locație',
     'no_disks_configured'              => 'Nu există niciun disc in config/laravel-backups.php',
 ];

src/resources/lang/en/backup.php

Indentation +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     'create_error_message'             => 'The backup was file could NOT be created.',
     'create_warning_title'             => 'Unknown error',
     'create_warning_message'           => 'Your backup may NOT have been created. Please check log files for details.',
-	'location' 						   => 'Location',
+    'location' 						   => 'Location',
     'no_disks_configured'              => 'No backup disks configured in config/laravel-backup.php',
     'backup_doesnt_exist'              => "The backup file doesn't exist.",
     'only_local_downloads_supported'   => "Only downloads from the Local filesystem are supported.",