Migrating users from old database, need help with associating roles and permission

[vipertecpro]

I have old database in which i have 3000 + users, i need to migrate those users into new database with their roles and permission, this is small api i created but it's taking too long and it not a good practice.

According to our estimate there will be 3 lakh records going to be inserted into model_has_permission table.
I have 3 roles and each role have minimum of 40-50 permission.

    /**
     * Assign role and permission to users
     *
     * */
    public function sortAllUsers(): array
    {
        $noRoleAssignedUsers = [];
        $getAllUsers = User::all()->chunk(300);
        foreach ($getAllUsers as $user){
           foreach ($user as $u){
               if((int)$u->role === 1){ //For Admin
                   $u->assignRole('admin'); // 30
               }elseif((int)$u->role === 2){ //For Ar Admin
                   $u->assignRole('ar admin'); // 49
               }elseif((int)$u->role === 3){ // For User
                   $u->assignRole('user'); // 30
               }else{
                   $u->assignRole('user');
                   $noRoleAssignedUsers[] = [
                       'id'    => $u->id,
                       'name'    => $u->name,
                       'email'    => $u->email,
                   ];
               }
               $u->syncPermissions($u->getPermissionsViaRoles()); // THIS IS CAUSING SERVER CRASHING 
           }
        }
        return [
            'message'              => 'All users roles and permission assigned',
            '$noRoleAssignedUsers' => $noRoleAssignedUsers
        ];
    }

What is the best way to migrate those users into new database along with their roles and permissions ?

[erikn69]

Why $u->syncPermissions($u->getPermissionsViaRoles());?

That permissions already assigned to roles, so, that is an unnecessary duplicity

Also you could do a custom manually sync

[vipertecpro]

I am using meteoric theme in which i got this plugin per-installed and setup,
Now based on my project requirement, there are some permissions that i need to assign to specific user + user -> role -> permission .
I am unable to fulfill this combination requirement.

Take a look at this sample code in which we are filtering menu based on user direct permission AND user role has permission.

/**
     * Filter menu item based on the user permission using Spatie plugin
     *
     * @param $array
     */
    public static function filterMenuPermissions(&$array)
    {
        if (!is_array($array)) {
            return;
        }

        $user = auth()->user();

        $checkPermission = $checkRole = false;
        if (auth()->check()) {
            // check if the spatie plugin functions exist
            $checkPermission = method_exists($user, 'hasAnyPermission');
            $checkRole       = method_exists($user, 'hasAnyRole');
        }

        foreach ($array as $key => &$value) {
            if (is_callable($value)) {
                continue;
            }

            if ($checkPermission && isset($value['permission'])) {
                foreach ($value['permission'] as $permission){
                    if($user->hasDirectPermission($permission) === false || $user->hasAnyPermission($permission) === false){ // This is not working as i am expecting.
                        unset($array[$key]);
                    }
                }
            }

            if ($checkRole && isset($value['role']) && !$user->hasAnyRole((array) $value['role'])) {
                unset($array[$key]);
            }

            if (is_array($value)) {
                self::filterMenuPermissions($value);
            }
        }
    }

If i can solve this issue i don't even have to do that bulk insert into database.

My actual problem is that i need to check and give "User direct permission + User role has permission".
Because of that issue every time i create new user, i am doing more nonsense like this

           $userData = [
                'name'               => $getFullName,
                'email'              => $request->get('email'),
                'password'           => bcrypt($password),
                'remember_token'     => date('Y-m-d h:i:s')
            ];
            $user = User::create($userData);
            $user->assignRole('user'); // i know this is enough 
            $user->syncPermissions($user->getPermissionsViaRoles());  .// BUT I AM ALSO DOING THIS :( 

For direct permission we have made separate UI

This is when we submit the code

 /**
     * Update user role.
     *
     * @param Request $request
     * @param User $user
     * @return JsonResponse
     */
    public function updateUserRole(Request $request, User $user): JsonResponse
    {
        try{
            $user->syncRoles([$request->get('role')]);
            $user->syncPermissions($user->getPermissionsViaRoles());
            return response()->json([
                "status"       => 'success',
                "message"      => 'User role updated successfully.',
                'redirectUrl'   => route('users.userPermissions',$user->id)
            ]);
        }catch (Exception $exception){
            Log::info('---------------------------------------------------------------------------------------');
            Log::info('****************************************');
            Log::info('CLASS : '.__CLASS__);
            Log::info('METHOD : '.__METHOD__);
            Log::info('FUNCTION : '.__FUNCTION__);
            Log::info('DIRECTORY : '.__DIR__);
            Log::info('****************************************');
            Log::info('EXCEPTION',[
                'status' => 'error',
                'curlUrl' => '',
                'message' => $exception->getMessage(),
                'content' => '',
                'header' => '',
                'code'    => $exception->getCode(),
                'file'    => $exception->getFile(),
                'line'    => $exception->getLine(),
                'trace'   => $exception->getTrace(),
            ]);
            return response()->json([
                "status" => 'error',
                "message" => $exception->getMessage(),
                'code'    => $exception->getCode(),
                'file'    => $exception->getFile(),
                'line'    => $exception->getLine(),
                'trace'   => $exception->getTrace(),
            ], 401);
        }
    }

[vipertecpro]

I got it all fixed, thx alot,