App\Http\Controllers\Profile\WizardProfileController::processWizard()

Complexity

Conditions	10
Paths	256

Size

Total Lines	28
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function processWizard(Request $request, FillWikiUserProfile $fillWikiUserProfile)
    {
        $role = $request->input('role') !== null ? $request->input('role') : '';
        $firstname = $request->input('firstname') !== null ? $request->input('firstname') : '';
        $lastname = $request->input('lastname') !== null ? $request->input('lastname') : '';
        $postalCode = $request->input('postal_code') !== null ? $request->input('postal_code') : '';
        $email = $request->input('email') !== null ? $request->input('email') : '';
        $farmingType = $request->input('farming_type') !== null ? $request->input('farming_type') : [];

        $fillWikiUserProfile->fill(Auth::user()->uuid, $role, $firstname, $lastname, $email, $postalCode, $farmingType);

        if(Auth::user()->hasVerifiedEmail()) {
            $user = Auth::user();
            if ($request->session()->has('wiki_callback')) {
                $user->wiki_token = $request->session()->get('wiki_token');
                $user->save();
                $callback = urldecode($request->session()->get('wiki_callback'));

It seems like $request->session()->get('wiki_callback') can also be of type null; however, parameter $string of urldecode() does only seem to accept string, maybe add an additional type check?

                return redirect($callback);
            }

            if($request->session()->has('sso')){
                $sso = $request->session()->get('sso');
                $sig = $request->session()->get('sig');
                return redirect('discourse/sso?sso='.$sso.'&sig='.$sig);
            }
        }

        return redirect()->route('verification.notice');
    }