App\Http\Controllers\Settings\MailController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	3	1
A	update()	0	17	1

<?php

namespace App\Http\Controllers\Settings;

use App\Http\Controllers\Controller;
use Brotzka\DotenvEditor\DotenvEditor;
use App\Http\Requests\Settings\MailSettingsUpdateRequest;

/**
 * Class MailController.
 */
class MailController extends Controller
{
    /**
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function index()
    {
        return view('settings.mail');
    }

    /**
     * @param MailSettingsUpdateRequest $request
     * @return \Illuminate\Http\RedirectResponse
     * @throws \Brotzka\DotenvEditor\Exceptions\DotEnvException
     */
    public function update(MailSettingsUpdateRequest $request)
    {
        $env = new DotenvEditor;
        $env->changeEnv([
            'MAIL_DRIVER' => $request->input('MAIL_DRIVER'),
            'MAIL_PORT' => $request->input('MAIL_PORT'),
            'MAIL_HOST' => $request->input('MAIL_HOST'),
            'MAIL_USERNAME' => $request->input('MAIL_USERNAME'),
            'MAIL_PASSWORD' => $request->input('MAIL_PASSWORD'),
        ]);

        notify()->flash(trans('general.woohoo'), 'success', [

It seems like trans('general.woohoo') can also be of type array; however, parameter $message of Codecourse\Notify\Notifier::flash() does only seem to accept string, maybe add an additional type check?

            'timer' => 2000,
            'text' => trans('general.success.update'),
        ]);

        return redirect()->route('settings.mail');
    }
}