Backpack\BackupManager\app\Http\Controllers\BackupController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	100
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	7
Bugs	2	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
C	index()	0	35	7
A	create()	0	18	2
A	download()	0	21	3
A	delete()	0	14	2

<?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 Log;

class BackupController extends Controller {

	public function index()

The return type could not be reliably inferred; please add a @return annotation.

	{
		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) {

The class Backpack\BackupManager\app\Http\Controllers\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

	      Response::make($e->getMessage(), 500);
	    }

	    return 'success';
	}

	/**
	 * Downloads a backup zip file.
	 */
	public function download($file_name)

The return type could not be reliably inferred; please add a @return annotation.

The parameter $file_name is not named in camelCase.

	{
		$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)

The parameter $file_name is not named in camelCase.

	{
		$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'));
		}
	}
}