App\Http\Controllers\RestoreUserController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	userReActivate()	0	31	3

<?php

namespace App\Http\Controllers;

use App\Models\User;
use Carbon\Carbon;

class RestoreUserController extends ProfilesController
{
    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('web');
    }

    /**
     * User Account Restore.
     *
     * @param string $token
     *
     * @return \Illuminate\Http\Response
     */
    public function userReActivate($token)
    {
        $userKeys = new ProfilesController();
        $sepKey = $userKeys->getSeperationKey();
        $userIdKey = $userKeys->getIdMultiKey();
        $restoreKey = config('settings.restoreKey');
        $encrypter = config('settings.restoreUserEncType');
        $level5 = base64_decode($token);
        $level4 = openssl_decrypt($level5, $encrypter, $restoreKey);
        $level3 = base64_decode($level4);
        $level2 = urldecode($level3);
        $level1[] = explode($sepKey, $level2);

$level1 was never initialized. Although not strictly required by PHP, it is generally a good practice to add $level1 = array(); before regardless.

        $userId = $level1[0][1] / $userIdKey;
        $user = SoftDeletesController::getDeletedUser($userId);

        if (!is_object($user)) {
            abort(500);
        }

        $deletedDate = $user->deleted_at;

The property deleted_at does not seem to exist on Illuminate\Http\Response.

        $currentDate = Carbon::now();
        $daysDeleted = $currentDate->diffInDays($deletedDate);
        $cutoffDays = config('settings.restoreUserCutoff');

        if ($daysDeleted >= $cutoffDays) {
            return redirect('/login')->with('error', trans('profile.errorRestoreUserTime'));

The expression return redirect('/login')->with('error', trans('profile.errorRestoreUserTime')) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

        }

        $user->restore();

        return redirect('/login')->with('success', trans('profile.successUserRestore', ['username' => $user->name]));

The expression return redirect('/login')->with('success', trans('profile.successUserRestore', array('username' => $user->name))) returns the type Illuminate\Http\RedirectResponse which is incompatible with the documented return type Illuminate\Http\Response.

The property name does not seem to exist on Illuminate\Http\Response.

    }
}