영업관리 PartSelect·JOIN을 part_mng 전용 테이블로 분리 (item_info 혼재 데이터 해소)
배경: item_info에 wace 마이그레이션(numeric id 8,179건) + RPS 자체 등록(UUID 21k건)이 섞여 있어 PartSelect/그리드 JOIN에서 잡 데이터(-20260126-, 00 등)와 RPS 자체 등록 데이터가 노출됨. wace 도메인 메뉴는 part_mng만 참조하도록 분리. 개발관리 등 다른 wace 메뉴도 동일 테이블 활용. 변경: - DDL: docs/migration/sales/ddl-extracted/105_create_part_mng.sql (item_info의 numeric id 데이터를 part_mng로 컬럼 매핑 INSERT, 8,176건 적재) - backend service JOIN 6곳 교체: item_info IT → part_mng PM (PM.objid::varchar = CI.part_objid) · salesEstimateService.ts (getList LATERAL JOIN, getById 라인 조회 — 2곳) · salesOrderMgmtService.ts (getList, getById, getOrderFormView, createProjectsFromContract — 4곳) - /sales/parts endpoint: part_mng SELECT + status active/release/활성 필터. 반환 키는 기존 호환을 위해 id/item_number/item_name으로 alias (objid::varchar/part_no/part_name). - 자동 검증: scripts/verify-part-mng.sql (카운트·JOIN·그리드 SQL·잡 데이터 제거 검증) 영향: - item_info는 그대로 — RPS 자체 메뉴(/sales/sales-item 등) 전용으로 유지 - ecrMngService, wacePlmDataImportService 등 RPS 다른 모듈은 이미 part_mng 사용 가정 → 자연 활성화 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -33,17 +33,23 @@ router.get("/codes/:groupId", async (req: Request, res: Response) => {
|
||||
}
|
||||
});
|
||||
|
||||
/** GET /api/sales/parts → [{id, item_number, item_name}] (영업관리 풀-옵션용) */
|
||||
/** GET /api/sales/parts → [{id, item_number, item_name}] (영업관리 풀-옵션용)
|
||||
* 데이터 출처: wace 품목 마스터 전용 테이블 part_mng (8,176건).
|
||||
* 반환 키는 기존 호환을 위해 id/item_number/item_name으로 alias.
|
||||
* status는 active/release/활성만 (wace 운영 값).
|
||||
*/
|
||||
router.get("/parts", async (_req: Request, res: Response) => {
|
||||
try {
|
||||
const pool = getPool();
|
||||
// wace 이식 데이터(company_code 빈 값/별표) 우선, COMPANY_16 데이터 추가
|
||||
const result = await pool.query(
|
||||
`SELECT id, item_number, item_name
|
||||
FROM item_info
|
||||
WHERE COALESCE(company_code, '') IN ('', '*', 'COMPANY_16')
|
||||
AND (item_number IS NOT NULL OR item_name IS NOT NULL)
|
||||
ORDER BY item_number NULLS LAST, item_name NULLS LAST`,
|
||||
`SELECT objid::varchar AS id,
|
||||
part_no AS item_number,
|
||||
part_name AS item_name
|
||||
FROM part_mng
|
||||
WHERE LOWER(COALESCE(status, '')) IN ('active', 'release', '활성')
|
||||
AND part_no IS NOT NULL AND part_no <> ''
|
||||
AND part_name IS NOT NULL AND part_name <> ''
|
||||
ORDER BY part_no NULLS LAST, part_name NULLS LAST`,
|
||||
);
|
||||
res.json({ success: true, data: result.rows });
|
||||
} catch (err) {
|
||||
|
||||
@@ -297,14 +297,14 @@ export async function getList(filter: EstimateListFilter) {
|
||||
SELECT
|
||||
CI.contract_objid,
|
||||
COUNT(*) AS item_count,
|
||||
(array_agg(COALESCE(IT.item_name, CI.part_name) ORDER BY CI.seq))[1] AS first_part_name,
|
||||
(array_agg(COALESCE(IT.item_number, CI.part_no) ORDER BY CI.seq))[1] AS first_part_no,
|
||||
(array_agg(COALESCE(PM.part_name, CI.part_name) ORDER BY CI.seq))[1] AS first_part_name,
|
||||
(array_agg(COALESCE(PM.part_no, CI.part_no) ORDER BY CI.seq))[1] AS first_part_no,
|
||||
MIN(CASE WHEN CI.due_date IS NOT NULL AND CI.due_date != '' THEN CI.due_date END) AS earliest_due_date,
|
||||
GREATEST(COUNT(CASE WHEN CI.due_date IS NOT NULL AND CI.due_date != '' THEN 1 END) - 1, 0) AS other_due_date_count,
|
||||
STRING_AGG(DISTINCT CC_RR.code_name, ', ') FILTER (WHERE CC_RR.code_name IS NOT NULL) AS return_reason_summary,
|
||||
STRING_AGG(DISTINCT CC_PRDI.code_name, ', ') FILTER (WHERE CC_PRDI.code_name IS NOT NULL) AS product_summary
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
LEFT JOIN comm_code CC_RR ON CC_RR.code_id = CI.return_reason AND CC_RR.status='active'
|
||||
LEFT JOIN comm_code CC_PRDI ON CC_PRDI.code_id = CI.product AND CC_PRDI.status='active'
|
||||
WHERE CI.status = 'ACTIVE'
|
||||
@@ -365,10 +365,10 @@ export async function getById(objid: string) {
|
||||
|
||||
const itemsRes = await pool.query(
|
||||
`SELECT CI.*,
|
||||
COALESCE(IT.item_number, CI.part_no) AS master_part_no,
|
||||
COALESCE(IT.item_name, CI.part_name) AS master_part_name
|
||||
COALESCE(PM.part_no, CI.part_no) AS master_part_no,
|
||||
COALESCE(PM.part_name, CI.part_name) AS master_part_name
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
WHERE CI.contract_objid = $1
|
||||
AND CI.status = 'ACTIVE'
|
||||
ORDER BY CI.seq`,
|
||||
|
||||
@@ -197,8 +197,8 @@ export async function getList(filter: OrderListFilter) {
|
||||
SELECT
|
||||
CI.contract_objid,
|
||||
COUNT(*) AS item_count,
|
||||
(array_agg(COALESCE(IT.item_name, CI.part_name) ORDER BY CI.seq))[1] AS first_part_name,
|
||||
(array_agg(COALESCE(IT.item_number, CI.part_no) ORDER BY CI.seq))[1] AS first_part_no,
|
||||
(array_agg(COALESCE(PM.part_name, CI.part_name) ORDER BY CI.seq))[1] AS first_part_name,
|
||||
(array_agg(COALESCE(PM.part_no, CI.part_no) ORDER BY CI.seq))[1] AS first_part_no,
|
||||
MIN(CASE WHEN CI.due_date IS NOT NULL AND CI.due_date != '' THEN CI.due_date END) AS earliest_due_date,
|
||||
GREATEST(COUNT(CASE WHEN CI.due_date IS NOT NULL AND CI.due_date != '' THEN 1 END) - 1, 0) AS other_due_date_count,
|
||||
COALESCE(SUM(CAST(REPLACE(NULLIF(CI.order_quantity, ''), ',', '') AS NUMERIC)), 0) AS order_quantity_sum,
|
||||
@@ -207,7 +207,7 @@ export async function getList(filter: OrderListFilter) {
|
||||
COUNT(CASE WHEN CI.order_quantity IS NOT NULL AND CI.order_quantity != '' AND CI.order_quantity != '0' THEN 1 END) AS has_order_data,
|
||||
STRING_AGG(DISTINCT CC_PRDI.code_name, ', ') FILTER (WHERE CC_PRDI.code_name IS NOT NULL) AS product_summary
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
LEFT JOIN comm_code CC_PRDI ON CC_PRDI.code_id = CI.product AND CC_PRDI.status='active'
|
||||
WHERE CI.status='ACTIVE'
|
||||
GROUP BY CI.contract_objid
|
||||
@@ -246,9 +246,9 @@ export async function getById(objid: string) {
|
||||
if (headerRes.rowCount === 0) return null;
|
||||
|
||||
const itemsRes = await pool.query(
|
||||
`SELECT CI.*, IT.item_name AS master_item_name
|
||||
`SELECT CI.*, PM.part_name AS master_item_name
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
WHERE CI.contract_objid = $1 AND CI.status = 'ACTIVE'
|
||||
ORDER BY CI.seq`, [objid],
|
||||
);
|
||||
@@ -323,8 +323,8 @@ export async function getOrderFormView(objid: string) {
|
||||
const itemsSql = `
|
||||
SELECT
|
||||
CI.seq,
|
||||
COALESCE(IT.item_number, CI.part_no) AS part_no,
|
||||
COALESCE(IT.item_name, CI.part_name) AS part_name,
|
||||
COALESCE(PM.part_no, CI.part_no) AS part_no,
|
||||
COALESCE(PM.part_name, CI.part_name) AS part_name,
|
||||
IT.size AS spec,
|
||||
CC_UNIT.code_name AS unit_name,
|
||||
IT.unit AS unit_code,
|
||||
@@ -335,7 +335,7 @@ export async function getOrderFormView(objid: string) {
|
||||
CI.order_vat AS order_vat,
|
||||
CI.order_total_amount AS order_total_amount
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
LEFT JOIN comm_code CC_UNIT ON CC_UNIT.code_id = IT.unit AND CC_UNIT.status='active'
|
||||
WHERE CI.contract_objid = $1 AND CI.status = 'ACTIVE'
|
||||
ORDER BY CI.seq
|
||||
@@ -670,11 +670,11 @@ async function createProjectsFromContract(client: any, contractObjid: string, us
|
||||
const itemsRes = await client.query(
|
||||
`SELECT
|
||||
CI.objid, CI.part_objid,
|
||||
COALESCE(IT.item_number, CI.part_no) AS part_no,
|
||||
COALESCE(IT.item_name, CI.part_name) AS part_name,
|
||||
COALESCE(PM.part_no, CI.part_no) AS part_no,
|
||||
COALESCE(PM.part_name, CI.part_name) AS part_name,
|
||||
CI.quantity, CI.order_quantity, CI.due_date, CI.product
|
||||
FROM contract_item CI
|
||||
LEFT JOIN item_info IT ON IT.id = CI.part_objid
|
||||
LEFT JOIN part_mng PM ON PM.objid::varchar = CI.part_objid
|
||||
WHERE CI.contract_objid = $1 AND CI.status = 'ACTIVE'
|
||||
ORDER BY CI.seq`,
|
||||
[contractObjid],
|
||||
|
||||
Reference in New Issue
Block a user