SamProject/sam-api
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix : 결과 전달시 두번 래핑되는 부분 수정

Browse Source 
- 컨트롤러와 서비스에서 각각 래핑 후 결과 전달됨으로 이중 래핑되고 있음
  -> 서비스에서 래핑하는 부분을 컨트롤러로 옮겨서 컨트롤러에서만 한번 래핑하는 걸로 수정
This commit is contained in:
권혁성
2025-08-19 12:41:17 +09:00
parent aa190bf48d
commit 43e4c507a7
18 changed files with 299 additions and 277 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
app/Services/Authz/RolePermissionService.php
View File

@@ -82,7 +82,7 @@ public static function list(int $roleId)
$role = self::loadRoleOrError($roleId, $tenantId);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
self::setTeam($tenantId);
@@ -93,7 +93,7 @@ public static function list(int $roleId)
->orderBy('name')
->get(['id','tenant_id','name','guard_name','created_at','updated_at']);
return ApiResponse::response('result', $perms);
return $perms;
}
/** 부여 (중복 무시) */
@@ -103,7 +103,7 @@ public static function grant(int $roleId, array $params = [])
$role = self::loadRoleOrError($roleId, $tenantId);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
// 유효성: 두 방식 중 하나만 요구하진 않지만, 최소 하나는 있어야 함
@@ -118,23 +118,23 @@ public static function grant(int $roleId, array $params = [])
],
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['permission_names']) && (empty($params['menus']) || empty($params['actions']))) {
return ApiResponse::error('permission_names 또는 menus+actions 중 하나는 필요합니다.', 422);
return ['error' => 'permission_names 또는 menus+actions 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
$names = self::resolvePermissionNames($tenantId, $params);
if (empty($names)) {
return ApiResponse::error('유효한 퍼미션이 없습니다.', 422);
return ['error' => '유효한 퍼미션이 없습니다.', 'code' => 422];
}
// Spatie: 이름 배열 부여 OK (teams 컨텍스트 적용됨)
$role->givePermissionTo($names);
return ApiResponse::response('success');
return 'success';
}
/** 회수 (없는 건 무시) */
@@ -144,7 +144,7 @@ public static function revoke(int $roleId, array $params = [])
$role = self::loadRoleOrError($roleId, $tenantId);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -158,22 +158,22 @@ public static function revoke(int $roleId, array $params = [])
],
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['permission_names']) && (empty($params['menus']) || empty($params['actions']))) {
return ApiResponse::error('permission_names 또는 menus+actions 중 하나는 필요합니다.', 422);
return ['error' => 'permission_names 또는 menus+actions 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
$names = self::resolvePermissionNames($tenantId, $params);
if (empty($names)) {
return ApiResponse::error('유효한 퍼미션이 없습니다.', 422);
return ['error' => '유효한 퍼미션이 없습니다.', 'code' => 422];
}
$role->revokePermissionTo($names);
return ApiResponse::response('success');
return 'success';
}
/** 동기화(완전 교체) */
@@ -183,7 +183,7 @@ public static function sync(int $roleId, array $params = [])
$role = self::loadRoleOrError($roleId, $tenantId);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -197,10 +197,10 @@ public static function sync(int $roleId, array $params = [])
],
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['permission_names']) && (empty($params['menus']) || empty($params['actions']))) {
return ApiResponse::error('permission_names 또는 menus+actions 중 하나는 필요합니다.', 422);
return ['error' => 'permission_names 또는 menus+actions 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
@@ -209,6 +209,6 @@ public static function sync(int $roleId, array $params = [])
// 동기화
$role->syncPermissions($names);
return ApiResponse::response('success');
return 'success';
}
}

20
app/Services/Authz/RoleService.php
View File

@@ -35,7 +35,7 @@ public static function index(array $params = [])
$list = $query->orderByDesc('id')
->paginate($size, ['*'], 'page', $page);
return ApiResponse::response('result', $list);
return $list;
}
/** 생성 */
@@ -54,7 +54,7 @@ public static function store(array $params = [])
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
// Spatie 팀(테넌트) 컨텍스트
@@ -67,7 +67,7 @@ public static function store(array $params = [])
'description' => $params['description'] ?? null,
]);
return ApiResponse::response('result', $role);
return $role;
}
/** 단건 */
@@ -80,10 +80,10 @@ public static function show(int $id)
->find($id);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
return ApiResponse::response('result', $role);
return $role;
}
/** 수정 */
@@ -96,7 +96,7 @@ public static function update(int $id, array $params = [])
->find($id);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -110,12 +110,12 @@ public static function update(int $id, array $params = [])
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
$role->fill($v->validated())->save();
return ApiResponse::response('result', $role->fresh());
return $role->fresh();
}
/** 삭제 (하드삭제) */
@@ -128,13 +128,13 @@ public static function destroy(int $id)
->find($id);
if (!$role) {
return ApiResponse::error('역할을 찾을 수 없습니다.', 404);
return ['error' => '역할을 찾을 수 없습니다.', 'code' => 404];
}
DB::transaction(function () use ($role) {
$role->delete(); // Spatie Role 기본: soft delete 없음
});
return ApiResponse::response('success');
return 'success';
}
}

34
app/Services/Authz/UserRoleService.php
View File

@@ -74,7 +74,7 @@ public static function list(int $userId)
$user = self::loadUserOrError($userId);
if (!$user) {
return ApiResponse::error('사용자를 찾을 수 없습니다.', 404);
return ['error' => '사용자를 찾을 수 없습니다.', 'code' => 404];
}
self::setTeam($tenantId);
@@ -86,7 +86,7 @@ public static function list(int $userId)
->select(['roles.id','roles.tenant_id','roles.name','roles.description','roles.guard_name','roles.created_at','roles.updated_at'])
->orderBy('roles.id', 'desc');
return ApiResponse::response('get', $builder);
return $builder->get();
}
/** 부여 (중복 무시) */
@@ -96,7 +96,7 @@ public static function grant(int $userId, array $params = [])
$user = self::loadUserOrError($userId);
if (!$user) {
return ApiResponse::error('사용자를 찾을 수 없습니다.', 404);
return ['error' => '사용자를 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -106,21 +106,21 @@ public static function grant(int $userId, array $params = [])
'role_ids.*' => 'integer|min:1',
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['role_names']) && empty($params['role_ids'])) {
return ApiResponse::error('role_names 또는 role_ids 중 하나는 필요합니다.', 422);
return ['error' => 'role_names 또는 role_ids 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
$names = self::resolveRoleNames($tenantId, $params);
if (empty($names)) {
return ApiResponse::error('유효한 역할이 없습니다.', 422);
return ['error' => '유효한 역할이 없습니다.', 'code' => 422];
}
$user->assignRole($names); // teams 컨텍스트 적용됨
return ApiResponse::response('success');
return 'success';
}
/** 회수 (없는 건 무시) */
@@ -130,7 +130,7 @@ public static function revoke(int $userId, array $params = [])
$user = self::loadUserOrError($userId);
if (!$user) {
return ApiResponse::error('사용자를 찾을 수 없습니다.', 404);
return ['error' => '사용자를 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -140,21 +140,21 @@ public static function revoke(int $userId, array $params = [])
'role_ids.*' => 'integer|min:1',
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['role_names']) && empty($params['role_ids'])) {
return ApiResponse::error('role_names 또는 role_ids 중 하나는 필요합니다.', 422);
return ['error' => 'role_names 또는 role_ids 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
$names = self::resolveRoleNames($tenantId, $params);
if (empty($names)) {
return ApiResponse::error('유효한 역할이 없습니다.', 422);
return ['error' => '유효한 역할이 없습니다.', 'code' => 422];
}
$user->removeRole($names); // 배열 허용
return ApiResponse::response('success');
return 'success';
}
/** 동기화(완전 교체) */
@@ -164,7 +164,7 @@ public static function sync(int $userId, array $params = [])
$user = self::loadUserOrError($userId);
if (!$user) {
return ApiResponse::error('사용자를 찾을 수 없습니다.', 404);
return ['error' => '사용자를 찾을 수 없습니다.', 'code' => 404];
}
$v = Validator::make($params, [
@@ -174,10 +174,10 @@ public static function sync(int $userId, array $params = [])
'role_ids.*' => 'integer|min:1',
]);
if ($v->fails()) {
return ApiResponse::error($v->errors()->first(), 422);
return ['error' => $v->errors()->first(), 'code' => 422];
}
if (empty($params['role_names']) && empty($params['role_ids'])) {
return ApiResponse::error('role_names 또는 role_ids 중 하나는 필요합니다.', 422);
return ['error' => 'role_names 또는 role_ids 중 하나는 필요합니다.', 'code' => 422];
}
self::setTeam($tenantId);
@@ -185,10 +185,10 @@ public static function sync(int $userId, array $params = [])
$names = self::resolveRoleNames($tenantId, $params);
if (empty($names)) {
// 정책상 빈 목록 sync 허용 시: $user->syncRoles([]) 로 전부 제거 가능
return ApiResponse::error('유효한 역할이 없습니다.', 422);
return ['error' => '유효한 역할이 없습니다.', 'code' => 422];
}
$user->syncRoles($names);
return ApiResponse::response('success');
return 'success';
}
}