Commit @b4438a2f638805c81fb9bae0d8ba8aa633974778

임동욱 08-20

fix: 메뉴 등록이 500으로 죽던 값 두 가지

INSERT에 DEFAULT_YN이 들어가는데 화면에 없는 항목이라 아무도 보내지 않아 NULL이
박히고 있었다. 등록에는 'N'을 싣는다.

아이콘을 고르지 않았을 때 빈 문자열을 보내고 있었다. 파일 테이블을 가리키는 컬럼이라
등록에서는 값이 있을 때만 싣고, 수정에서는 지우기를 위해 빈 문자열을 유지한다.

실패하면 코드·메시지·보낸 파라미터를 서버 콘솔에 남긴다 — 백엔드 로그를 볼 수 없는
환경이라 이게 유일한 단서다.

Co-Authored-By: Claude Opus 5

@b4438a2f638805c81fb9bae0d8ba8aa633974778

3057371

b4438a2

lib/data/repositories/system-menu-repository.ts

--- lib/data/repositories/system-menu-repository.ts

+++ lib/data/repositories/system-menu-repository.ts


     topYn: values.isTopActive ? 'Y' : 'N',
     menuUrl: values.url,
     menuExplan: values.description,
-    // 빈 문자열을 그대로 보낸다 — UPDATE가 null만 건너뛰므로 이래야 아이콘을 지울 수 있다.
-    atchFileId: values.largeIconId,
-    smallAtchFileId: values.smallIconId,
     useYn: values.isActive ? 'Y' : 'N',
   };
+}
+
+function logFailure(
+  action: string,
+  result: { code: number; message: string },
+  params: Record<string, unknown>
+): void {
+  console.error(`[system-menu] ${action} 실패`, {
+    code: result.code,
+    message: result.message,
+    params,
+  });
 }
 
 export async function createSystemMenu(values: SystemMenuValues): Promise<void> {
   const accessToken = await getSessionAccessToken();
 
+  // 등록에는 지울 아이콘이 없다. 빈 문자열을 보내면 파일 테이블을 가리키는 값이 되어
+  // 제약에 걸릴 수 있으므로 고른 것이 있을 때만 싣는다.
+  const params = {
+    ...toWriteParams(values),
+    upMenuId: values.parentId,
+    menuIdPath: values.parentIdPath || undefined,
+    // INSERT에 DEFAULT_YN이 들어가는데 화면에는 없는 항목이다. 보내지 않으면 NULL이 박힌다.
+    defaultYn: 'N',
+    ...(values.largeIconId ? { atchFileId: values.largeIconId } : {}),
+    ...(values.smallIconId ? { smallAtchFileId: values.smallIconId } : {}),
+  };
+
   const result = await backendFetch<unknown>(MENU_PATH, {
     method: 'POST',
-    form: {
-      ...toWriteParams(values),
-      upMenuId: values.parentId,
-      menuIdPath: values.parentIdPath || undefined,
-    },
+    form: params,
     accessToken: accessToken ?? undefined,
     cache: 'no-store',
     canHaveNullData: true,

   });
 
   if (!result.ok) {
+    logFailure('등록', result, params);
     throw new BackendRequestError(result);
   }
 }

 export async function updateSystemMenu(values: SystemMenuValues): Promise<void> {
   const accessToken = await getSessionAccessToken();
 
+  // 수정은 빈 문자열을 그대로 보낸다 — UPDATE가 null만 건너뛰므로 이래야 아이콘을 지울 수 있다.
+  const params = {
+    ...toWriteParams(values),
+    atchFileId: values.largeIconId,
+    smallAtchFileId: values.smallIconId,
+  };
+
   const result = await backendFetch<unknown>(`${MENU_PATH}/${values.id}`, {
     method: 'PUT',
-    body: toWriteParams(values),
+    body: params,
     accessToken: accessToken ?? undefined,
     cache: 'no-store',
     canHaveNullData: true,

   });
 
   if (!result.ok) {
+    logFailure('수정', result, params);
     throw new BackendRequestError(result);
   }
 }

Add a comment

Open 0
Closed 0

List

...	...	@@ -108,23 +108,40 @@
108	108	topYn: values.isTopActive ? 'Y' : 'N',
109	109	menuUrl: values.url,
110	110	menuExplan: values.description,
111		- // 빈 문자열을 그대로 보낸다 — UPDATE가 null만 건너뛰므로 이래야 아이콘을 지울 수 있다.
112		- atchFileId: values.largeIconId,
113		- smallAtchFileId: values.smallIconId,
114	111	useYn: values.isActive ? 'Y' : 'N',
115	112	};
	113	+}
	114	+
	115	+function logFailure(
	116	+ action: string,
	117	+ result: { code: number; message: string },
	118	+ params: Record<string, unknown>
	119	+): void {
	120	+ console.error(`[system-menu] ${action} 실패`, {
	121	+ code: result.code,
	122	+ message: result.message,
	123	+ params,
	124	+ });
116	125	}
117	126
118	127	export async function createSystemMenu(values: SystemMenuValues): Promise<void> {
119	128	const accessToken = await getSessionAccessToken();
120	129
	130	+ // 등록에는 지울 아이콘이 없다. 빈 문자열을 보내면 파일 테이블을 가리키는 값이 되어
	131	+ // 제약에 걸릴 수 있으므로 고른 것이 있을 때만 싣는다.
	132	+ const params = {
	133	+ ...toWriteParams(values),
	134	+ upMenuId: values.parentId,
	135	+ menuIdPath: values.parentIdPath \|\| undefined,
	136	+ // INSERT에 DEFAULT_YN이 들어가는데 화면에는 없는 항목이다. 보내지 않으면 NULL이 박힌다.
	137	+ defaultYn: 'N',
	138	+ ...(values.largeIconId ? { atchFileId: values.largeIconId } : {}),
	139	+ ...(values.smallIconId ? { smallAtchFileId: values.smallIconId } : {}),
	140	+ };
	141	+
121	142	const result = await backendFetch<unknown>(MENU_PATH, {
122	143	method: 'POST',
123		- form: {
124		- ...toWriteParams(values),
125		- upMenuId: values.parentId,
126		- menuIdPath: values.parentIdPath \|\| undefined,
127		- },
	144	+ form: params,
128	145	accessToken: accessToken ?? undefined,
129	146	cache: 'no-store',
130	147	canHaveNullData: true,
...	...	@@ -132,6 +149,7 @@
132	149	});
133	150
134	151	if (!result.ok) {
	152	+ logFailure('등록', result, params);
135	153	throw new BackendRequestError(result);
136	154	}
137	155	}
...	...	@@ -139,9 +157,16 @@
139	157	export async function updateSystemMenu(values: SystemMenuValues): Promise<void> {
140	158	const accessToken = await getSessionAccessToken();
141	159
	160	+ // 수정은 빈 문자열을 그대로 보낸다 — UPDATE가 null만 건너뛰므로 이래야 아이콘을 지울 수 있다.
	161	+ const params = {
	162	+ ...toWriteParams(values),
	163	+ atchFileId: values.largeIconId,
	164	+ smallAtchFileId: values.smallIconId,
	165	+ };
	166	+
142	167	const result = await backendFetch<unknown>(`${MENU_PATH}/${values.id}`, {
143	168	method: 'PUT',
144		- body: toWriteParams(values),
	169	+ body: params,
145	170	accessToken: accessToken ?? undefined,
146	171	cache: 'no-store',
147	172	canHaveNullData: true,
...	...	@@ -149,6 +174,7 @@
149	174	});
150	175
151	176	if (!result.ok) {
	177	+ logFailure('수정', result, params);
152	178	throw new BackendRequestError(result);
153	179	}
154	180	}

Delete comment