Webelightdev\LaravelDbBackup\Commands\DbBackupCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	handle()	0	11	1

<?php

namespace Webelightdev\LaravelDbBackup\Commands;

use Illuminate\Console\Command;
use Webelightdev\LaravelDbBackup\BackupStorage\Backup;
use Webelightdev\LaravelDbBackup\BackupStorage\GoogleDrive\Storage;

class DbBackupCommand extends Command
{
    protected $backup;
    protected $storage;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'dbbackup:run';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Create Database backup.';

    /**
     * Create a new command instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(Backup $backup, Storage $storage)
    {
        parent::__construct();
        $this->backup = $backup;
        $this->storage = $storage;
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $backup_file = $this->backup->createBackup();
        $this->storage->uploadFile($backup_file);
        
        // \Mail::send('emails.db_backup_email', compact('current_date'), function ($message) use ($current_date, $dumpfname) {

64% of this comment could be valid code. Did you maybe forget this after debugging?

        //     $message->from(env('MAIL_ADDRESS'), env('MAIL_NAME'));

71% of this comment could be valid code. Did you maybe forget this after debugging?

        //     $message->to(env('MAIL_TO'))->subject('ERP System DB Backup-'.$current_date)->attach($dumpfname);

74% of this comment could be valid code. Did you maybe forget this after debugging?

        // });
        unlink($backup_file);
    }
}