使用Spatie角色权限更新时触发PermissionDoesNotExist错误排查

错误原因 #

1. PermissionDoesNotExist异常触发点：Spatie的hasPermissionTo方法在传入的权限名称/对象不存在于数据库时，会直接抛出PermissionDoesNotExist异常，而非返回false。你的循环中直接调用该方法，未做前置校验或异常捕获，导致报错。
2. 逻辑错误：方法内的判断逻辑完全颠倒——当角色没有对应权限时，你把$hasPermission设为true，这会导致无论角色是否拥有所有权限，最终都返回true，完全失去了校验意义。

修复方案 #

方案1：先校验权限是否存在，再检查角色关联 #

修改方法，先确保传入的所有权限都存在，再逐个检查角色是否拥有：

public static function roleHasPermissions($role, $permissions)
{
    $hasPermission = true;

    foreach ($permissions as $permission) {
        // 先检查权限是否存在
        $existingPermission = \Spatie\Permission\Models\Permission::findByName($permission);
        
        if (!$existingPermission || !$role->hasPermissionTo($existingPermission)) {
            $hasPermission = false;
            // 一旦发现无权限，可提前跳出循环优化性能
            break;
        }
    }

    return $hasPermission;
}

方案2：捕获异常处理不存在的权限 #

如果允许部分权限不存在的场景（比如传入了错误的权限名称），可以通过捕获异常来处理：

public static function roleHasPermissions($role, $permissions)
{
    $hasPermission = true;

    foreach ($permissions as $permission) {
        try {
            if (!$role->hasPermissionTo($permission)) {
                $hasPermission = false;
                break;
            }
        } catch (\Spatie\Permission\Exceptions\PermissionDoesNotExist $e) {
            // 权限不存在时直接标记为不满足
            $hasPermission = false;
            break;
        }
    }

    return $hasPermission;
}

额外优化建议 #

• 如果$permissions是批量传入的，也可以用Spatie提供的批量检查方法hasAllPermissions简化代码，但同样要注意权限不存在的异常问题，比如先过滤出合法权限再调用。
• 建议在调用这个方法之前，先对传入的$permissions进行过滤，确保所有权限名称都是合法存在的，减少后续异常处理的复杂度。

内容的提问来源于stack exchange，提问作者Yeabsra Habtu