diff --git a/api.hyungi.net/models/workerModel.js b/api.hyungi.net/models/workerModel.js index 0cd7635..e39d3c3 100644 --- a/api.hyungi.net/models/workerModel.js +++ b/api.hyungi.net/models/workerModel.js @@ -1,12 +1,21 @@ const { getDb } = require('../dbPool'); +// 날짜 형식 변환 헬퍼 함수 (ISO 8601 -> MySQL DATE) +const formatDate = (dateStr) => { + if (!dateStr) return null; + if (typeof dateStr === 'string' && dateStr.includes('T')) { + return dateStr.split('T')[0]; // ISO 8601 -> YYYY-MM-DD + } + return dateStr; +}; + // 1. 작업자 생성 const create = async (worker, callback) => { try { const db = await getDb(); - const { - worker_name, - job_type = 'worker', + const { + worker_name, + job_type = 'worker', phone_number = null, email = null, hire_date = null, @@ -16,10 +25,10 @@ const create = async (worker, callback) => { } = worker; const [result] = await db.query( - `INSERT INTO workers + `INSERT INTO workers (worker_name, job_type, phone_number, email, hire_date, department, notes, status) VALUES (?, ?, ?, ?, ?, ?, ?, ?)`, - [worker_name, job_type, phone_number, email, hire_date, department, notes, status] + [worker_name, job_type, phone_number, email, formatDate(hire_date), department, notes, status] ); callback(null, result.insertId); @@ -32,7 +41,13 @@ const create = async (worker, callback) => { const getAll = async (callback) => { try { const db = await getDb(); - const [rows] = await db.query(`SELECT * FROM workers ORDER BY worker_id DESC`); + const [rows] = await db.query(` + SELECT + *, + CASE WHEN status = 'active' THEN 1 ELSE 0 END AS is_active + FROM workers + ORDER BY worker_id DESC + `); callback(null, rows); } catch (err) { callback(err); @@ -43,7 +58,13 @@ const getAll = async (callback) => { const getById = async (worker_id, callback) => { try { const db = await getDb(); - const [rows] = await db.query(`SELECT * FROM workers WHERE worker_id = ?`, [worker_id]); + const [rows] = await db.query(` + SELECT + *, + CASE WHEN status = 'active' THEN 1 ELSE 0 END AS is_active + FROM workers + WHERE worker_id = ? + `, [worker_id]); callback(null, rows[0]); } catch (err) { callback(err); @@ -54,18 +75,43 @@ const getById = async (worker_id, callback) => { const update = async (worker, callback) => { try { const db = await getDb(); - const { worker_id, worker_name, join_date, job_type, salary, annual_leave, status } = worker; + const { + worker_id, + worker_name, + job_type, + join_date, + status, + phone_number, + email, + hire_date, + department, + notes + } = worker; const [result] = await db.query( `UPDATE workers SET worker_name = ?, - join_date = ?, job_type = ?, - salary = ?, - annual_leave = ?, - status = ? + join_date = ?, + status = ?, + phone_number = ?, + email = ?, + hire_date = ?, + department = ?, + notes = ? WHERE worker_id = ?`, - [worker_name, join_date, job_type, salary, annual_leave, status, worker_id] + [ + worker_name, + job_type, + formatDate(join_date), + status, + phone_number, + email, + formatDate(hire_date), + department, + notes, + worker_id + ] ); callback(null, result.affectedRows); diff --git a/web-ui/js/attendance.js b/web-ui/js/attendance.js index e2db96d..10ffcc7 100644 --- a/web-ui/js/attendance.js +++ b/web-ui/js/attendance.js @@ -38,7 +38,13 @@ function fillSelectOptions() { async function fetchWorkers() { try { const res = await fetch(`${API}/workers`, { headers: getAuthHeaders() }); - workers = await res.json(); + const allWorkers = await res.json(); + + // 활성화된 작업자만 필터링 + workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + workers.sort((a, b) => a.worker_id - b.worker_id); } catch (err) { alert('작업자 불러오기 실패'); diff --git a/web-ui/js/daily-issue-api.js b/web-ui/js/daily-issue-api.js index ef9bcec..8a0e95b 100644 --- a/web-ui/js/daily-issue-api.js +++ b/web-ui/js/daily-issue-api.js @@ -30,7 +30,7 @@ export async function getWorkersByDate(date) { // 현재는 기존 로직을 최대한 활용하여 구현합니다. let workers = []; const reports = await apiGet(`/daily-work-reports?date=${date}`); - + if (reports && reports.length > 0) { const workerMap = new Map(); reports.forEach(r => { @@ -41,7 +41,12 @@ export async function getWorkersByDate(date) { workers = Array.from(workerMap.values()); } else { // 보고서가 없으면 전체 작업자 목록을 가져옵니다. - workers = await apiGet('/workers'); + const allWorkers = await apiGet('/workers'); + + // 활성화된 작업자만 필터링 + workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); } return workers.sort((a, b) => a.worker_name.localeCompare(b.worker_name)); } catch (error) { diff --git a/web-ui/js/daily-work-report.js b/web-ui/js/daily-work-report.js index f9f13e2..894b59b 100644 --- a/web-ui/js/daily-work-report.js +++ b/web-ui/js/daily-work-report.js @@ -207,8 +207,14 @@ async function loadWorkers() { try { console.log('Workers API 호출 중... (통합 API 사용)'); const data = await window.apiCall(`${window.API}/workers`); - workers = Array.isArray(data) ? data : (data.data || data.workers || []); - console.log('✅ Workers 로드 성공:', workers.length); + const allWorkers = Array.isArray(data) ? data : (data.data || data.workers || []); + + // 활성화된 작업자만 필터링 + workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + + console.log(`✅ Workers 로드 성공: ${workers.length}명 (전체: ${allWorkers.length}명)`); } catch (error) { console.error('작업자 로딩 오류:', error); throw error; @@ -334,8 +340,8 @@ function addWorkEntry() { entryDiv.innerHTML = `
작업 항목 #${workEntryCounter}
-
@@ -477,10 +483,15 @@ function setupWorkEntryEvents(entryDiv) { // 작업 항목 제거 function removeWorkEntry(id) { - const entry = document.querySelector(`[data-id="${id}"]`); + console.log('🗑️ removeWorkEntry 호출됨, id:', id); + const entry = document.querySelector(`.work-entry[data-id="${id}"]`); + console.log('🗑️ 찾은 entry:', entry); if (entry) { entry.remove(); updateTotalHours(); + console.log('✅ 작업 항목 삭제 완료'); + } else { + console.log('❌ 작업 항목을 찾을 수 없음'); } } diff --git a/web-ui/js/manage-user.js b/web-ui/js/manage-user.js index 4f07386..e4edf55 100644 --- a/web-ui/js/manage-user.js +++ b/web-ui/js/manage-user.js @@ -246,12 +246,18 @@ async function loadWorkerOptions() { const res = await fetch(`${API}/workers`, { headers: getAuthHeaders() }); - + if (!res.ok) { throw new Error(`HTTP error! status: ${res.status}`); } - - const workers = await res.json(); + + const allWorkers = await res.json(); + + // 활성화된 작업자만 필터링 + const workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + if (Array.isArray(workers)) { workers.forEach(w => { const opt = document.createElement('option'); diff --git a/web-ui/js/management-dashboard.js b/web-ui/js/management-dashboard.js index bacb4bb..161aa78 100644 --- a/web-ui/js/management-dashboard.js +++ b/web-ui/js/management-dashboard.js @@ -127,8 +127,14 @@ async function loadWorkers() { try { console.log('작업자 데이터 로딩 중... (통합 API)'); const data = await apiCall(`${API}/workers`); - workers = Array.isArray(data) ? data : (data.data || data.workers || []); - console.log('✅ 작업자 로드 성공:', workers.length); + const allWorkers = Array.isArray(data) ? data : (data.data || data.workers || []); + + // 활성화된 작업자만 필터링 + workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + + console.log(`✅ 작업자 로드 성공: ${workers.length}명 (전체: ${allWorkers.length}명)`); } catch (error) { console.error('작업자 로딩 오류:', error); throw error; diff --git a/web-ui/js/modern-dashboard.js b/web-ui/js/modern-dashboard.js index ffecf97..c068618 100644 --- a/web-ui/js/modern-dashboard.js +++ b/web-ui/js/modern-dashboard.js @@ -216,8 +216,14 @@ async function loadWorkers() { try { console.log('👥 작업자 데이터 로딩...'); const response = await window.apiCall('/workers'); - workersData = Array.isArray(response) ? response : (response.data || []); - console.log(`✅ 작업자 ${workersData.length}명 로드 완료`); + const allWorkers = Array.isArray(response) ? response : (response.data || []); + + // 활성화된 작업자만 필터링 + workersData = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + + console.log(`✅ 작업자 ${workersData.length}명 로드 완료 (전체: ${allWorkers.length}명)`); return workersData; } catch (error) { console.error('작업자 데이터 로딩 오류:', error); diff --git a/web-ui/js/project-analysis-api.js b/web-ui/js/project-analysis-api.js index 2518597..4ee5910 100644 --- a/web-ui/js/project-analysis-api.js +++ b/web-ui/js/project-analysis-api.js @@ -8,11 +8,17 @@ import { apiGet } from './api-helper.js'; */ export async function getMasterData() { try { - const [workers, projects, tasks] = await Promise.all([ + const [allWorkers, projects, tasks] = await Promise.all([ apiGet('/workers'), apiGet('/projects'), apiGet('/tasks') ]); + + // 활성화된 작업자만 필터링 + const workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + return { workers, projects, tasks }; } catch (error) { console.error('마스터 데이터 로딩 실패:', error); diff --git a/web-ui/js/work-report-api.js b/web-ui/js/work-report-api.js index bb206ae..4adc18d 100644 --- a/web-ui/js/work-report-api.js +++ b/web-ui/js/work-report-api.js @@ -8,17 +8,22 @@ import { apiGet, apiPost } from './api-helper.js'; */ export async function getInitialData() { try { - const [workers, projects, tasks] = await Promise.all([ + const [allWorkers, projects, tasks] = await Promise.all([ apiGet('/workers'), apiGet('/projects'), apiGet('/tasks') ]); + // 활성화된 작업자만 필터링 + const workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + // 데이터 형식 검증 if (!Array.isArray(workers) || !Array.isArray(projects) || !Array.isArray(tasks)) { throw new Error('서버에서 받은 데이터 형식이 올바르지 않습니다.'); } - + // 작업자 목록은 ID 기준으로 정렬 workers.sort((a, b) => a.worker_id - b.worker_id); diff --git a/web-ui/js/work-report-calendar.js b/web-ui/js/work-report-calendar.js index eacc3e3..725e1ef 100644 --- a/web-ui/js/work-report-calendar.js +++ b/web-ui/js/work-report-calendar.js @@ -102,12 +102,18 @@ function setupEventListeners() { // 작업자 데이터 로드 (캐시) async function loadWorkersData() { if (allWorkers.length > 0) return allWorkers; - + try { console.log('👥 작업자 데이터 로딩...'); const response = await window.apiCall('/workers'); - allWorkers = Array.isArray(response) ? response : (response.data || []); - console.log(`✅ 작업자 ${allWorkers.length}명 로드 완료`); + const workers = Array.isArray(response) ? response : (response.data || []); + + // 활성화된 작업자만 필터링 + allWorkers = workers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + + console.log(`✅ 작업자 ${allWorkers.length}명 로드 완료 (전체: ${workers.length}명)`); return allWorkers; } catch (error) { console.error('작업자 데이터 로딩 오류:', error); diff --git a/web-ui/js/work-report-manage.js b/web-ui/js/work-report-manage.js index 0fe08e3..a628878 100644 --- a/web-ui/js/work-report-manage.js +++ b/web-ui/js/work-report-manage.js @@ -28,10 +28,15 @@ async function loadReports() { if (![wRes, pRes, rRes].every(res => res.ok)) throw new Error('불러오기 실패'); - const [workers, projects, reports] = await Promise.all([ + const [allWorkers, projects, reports] = await Promise.all([ wRes.json(), pRes.json(), rRes.json() ]); + // 활성화된 작업자만 필터링 + const workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + // 배열 체크 if (!Array.isArray(workers) || !Array.isArray(projects) || !Array.isArray(reports)) { throw new Error('잘못된 데이터 형식'); diff --git a/web-ui/js/work-report-review.js b/web-ui/js/work-report-review.js index 483f1a2..7bd3ff4 100644 --- a/web-ui/js/work-report-review.js +++ b/web-ui/js/work-report-review.js @@ -55,7 +55,13 @@ class WorkReportReviewManager { throw new Error(`API 응답 오류: Workers(${workersRes.status}), Projects(${projectsRes.status})`); } - this.workers = await workersRes.json(); + const allWorkers = await workersRes.json(); + + // 활성화된 작업자만 필터링 + this.workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + this.projects = await projectsRes.json(); // 새로운 API들 로드 (실패해도 기본값 사용) diff --git a/개발 log/2025-12-09.md b/개발 log/2025-12-09.md new file mode 100644 index 0000000..5b1bbf5 --- /dev/null +++ b/개발 log/2025-12-09.md @@ -0,0 +1,167 @@ +# 개발 로그 - 2025-12-09 + +## 타임라인 + +### 17:12 - 개발 로그 시스템 초기화 +- **작업**: 생성 +- **대상**: `개발 log/` 폴더 및 타임라인 시스템 +- **내용**: 코드 변경 이력을 추적하기 위한 개발 로그 시스템 구축 +- **설명**: 앞으로 모든 코드 수정/생성/삭제 작업이 이 파일에 시간순으로 기록됩니다 + +--- + +### 17:25 - 작업자 비활성화 버그 수정 +- **작업**: 수정 +- **대상**: `api.hyungi.net/models/workerModel.js` +- **문제**: 작업자를 페이지에서 비활성화해도 새로고침하면 다시 활성화 상태로 돌아오는 버그 +- **원인**: + - `workers` 테이블에는 `status` 필드만 존재 (active/inactive) + - 프론트엔드는 `is_active` 필드를 사용하는데 백엔드에서 제공하지 않음 + - 로컬 캐시는 업데이트되지만 서버에서 다시 불러오면 `is_active` 필드가 없어서 기본값으로 돌아감 +- **해결방법**: + - `getAll()` 함수: `CASE WHEN status = 'active' THEN 1 ELSE 0 END AS is_active` 추가 + - `getById()` 함수: 동일한 가상 필드 추가 +- **변경 내용**: + ```sql + -- 이전 + SELECT * FROM workers ORDER BY worker_id DESC + + -- 이후 + SELECT + *, + CASE WHEN status = 'active' THEN 1 ELSE 0 END AS is_active + FROM workers + ORDER BY worker_id DESC + ``` +- **파일**: api.hyungi.net/models/workerModel.js:32-46, 48-63 +- **테스트 필요**: API 서버 재시작 후 작업자 비활성화 테스트 + +--- + +### 17:30 - 날짜 형식 변환 오류 수정 (500 에러) +- **작업**: 수정 +- **대상**: `api.hyungi.net/models/workerModel.js` +- **문제**: 작업자 비활성화 시도 시 500 Internal Server Error 발생 +- **원인**: + - 프론트엔드에서 ISO 8601 형식(`2025-04-01T00:00:00.000Z`)으로 날짜 전송 + - MySQL은 `YYYY-MM-DD` 형식 요구 + - 날짜 변환 로직 없이 직접 INSERT/UPDATE 시도 +- **해결방법**: + - `formatDate()` 헬퍼 함수 추가 (ISO 8601 → MySQL DATE 변환) + - `create()` 함수: `hire_date` 변환 적용 + - `update()` 함수: `join_date`, `hire_date` 변환 적용 +- **변경 내용**: + ```javascript + const formatDate = (dateStr) => { + if (!dateStr) return null; + if (typeof dateStr === 'string' && dateStr.includes('T')) { + return dateStr.split('T')[0]; // ISO 8601 -> YYYY-MM-DD + } + return dateStr; + }; + ``` +- **파일**: api.hyungi.net/models/workerModel.js:4-10, 31, 89, 110 + +--- + +### 17:35 - 작업자 업데이트 필드 불일치 수정 +- **작업**: 수정 +- **대상**: `api.hyungi.net/models/workerModel.js` +- **문제**: 작업자 상태 변경이 DB에 저장되지 않음 +- **원인**: + - `update()` 함수가 존재하지 않는 필드 사용: `salary`, `annual_leave` + - 실제 테이블 필드와 불일치: `phone_number`, `email`, `hire_date`, `department`, `notes` 누락 +- **해결방법**: + - workers 테이블 스키마에 맞게 필드 수정 + - 모든 필드 올바르게 매핑 +- **변경 필드**: + - ❌ 제거: `salary`, `annual_leave` + - ✅ 추가: `phone_number`, `email`, `hire_date`, `department`, `notes` +- **파일**: api.hyungi.net/models/workerModel.js:74-121 + +--- + +### 17:40 - 프론트엔드 비활성 작업자 필터링 추가 +- **작업**: 수정 +- **대상**: 프론트엔드 JavaScript 파일들 +- **문제**: 대시보드 및 작업 보고서에서 비활성화된 작업자가 계속 표시됨 +- **원인**: API에서 모든 작업자를 불러온 후 활성/비활성 필터링을 하지 않음 +- **해결방법**: 각 페이지의 `loadWorkers()` 함수에 활성 작업자만 필터링하는 로직 추가 +- **변경 내용**: + ```javascript + // 활성화된 작업자만 필터링 + workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + ``` +- **수정 파일**: + 1. `web-ui/js/modern-dashboard.js:215-233` - 그룹 리더 대시보드 + 2. `web-ui/js/daily-work-report.js:206-222` - 일일 작업 보고서 + 3. `web-ui/js/work-report-calendar.js:103-123` - 작업 보고서 캘린더 +- **영향받는 페이지**: + - 그룹 리더 대시보드 (작업자 현황 카드) + - 일일 작업 보고서 (작업자 선택) + - 작업 보고서 캘린더 (작업자 필터) + +--- + +### 17:50 - 전체 페이지 비활성 작업자 필터링 추가 (대규모) +- **작업**: 수정 +- **대상**: 프론트엔드 JavaScript 파일들 (추가 8개 파일) +- **문제**: 대시보드 외 다른 페이지들에서도 비활성 작업자가 표시됨 +- **해결방법**: 작업자를 로드하는 모든 페이지에 활성화 필터링 로직 추가 +- **수정 파일 목록**: + 1. `web-ui/js/attendance.js:38-52` - 근태 관리 + 2. `web-ui/js/manage-user.js:241-272` - 사용자 관리 (작업자 드롭다운) + 3. `web-ui/js/management-dashboard.js:126-142` - 관리 대시보드 + 4. `web-ui/js/work-report-manage.js:29-50` - 작업 보고서 관리 + 5. `web-ui/js/work-report-review.js:54-65` - 작업 보고서 검토 + 6. `web-ui/js/work-report-api.js:9-36` - 작업 보고서 API (공통) + 7. `web-ui/js/project-analysis-api.js:9-27` - 프로젝트 분석 API (공통) + 8. `web-ui/js/daily-issue-api.js:26-56` - 일일 이슈 보고서 API +- **필터링 로직 (공통)**: + ```javascript + // 활성화된 작업자만 필터링 + const workers = allWorkers.filter(worker => { + return worker.status === 'active' || worker.is_active === 1 || worker.is_active === true; + }); + ``` +- **영향받는 모든 페이지**: + - ✅ 그룹 리더 대시보드 + - ✅ 관리 대시보드 + - ✅ 일일 작업 보고서 + - ✅ 작업 보고서 캘린더 + - ✅ 작업 보고서 관리 + - ✅ 작업 보고서 검토 + - ✅ 프로젝트 분석 + - ✅ 일일 이슈 보고서 + - ✅ 근태 관리 + - ✅ 사용자 관리 + +--- + +## 요약 + +### 백엔드 수정 (1개 파일) +- `api.hyungi.net/models/workerModel.js`: is_active 가상 필드, 날짜 변환, 필드 수정 + +### 프론트엔드 수정 (11개 파일) +- modern-dashboard.js +- daily-work-report.js +- work-report-calendar.js +- attendance.js +- manage-user.js +- management-dashboard.js +- work-report-manage.js +- work-report-review.js +- work-report-api.js +- project-analysis-api.js +- daily-issue-api.js + +### 테스트 완료 +- [x] 작업자 비활성화 저장 +- [x] 페이지 새로고침 후 상태 유지 +- [ ] 모든 페이지에서 비활성 작업자 숨김 확인 필요 + +--- +