hyungi/TK-FB-Project
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 작업자 비활성화 기능 완전 수정

Browse Source 
작업자 퇴사 시 비활성화 기능이 제대로 작동하지 않던 문제 해결

백엔드 수정:
- is_active 가상 필드 추가 (status 기반 자동 생성)
- ISO 8601 날짜 형식을 MySQL DATE 형식으로 변환
- 작업자 업데이트 필드 오류 수정 (salary, annual_leave 제거)

프론트엔드 수정 (11개 파일):
- 모든 페이지에서 비활성 작업자 필터링 로직 추가
- 대시보드, 작업보고서, 근태관리, 사용자관리 등 전체 페이지 적용

영향받는 기능:
- 작업자 관리: 비활성화 상태가 DB에 저장되고 새로고침 후에도 유지
- 모든 페이지: 비활성화된 작업자가 선택 목록에서 제외됨

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Hyungi Ahn
2025-12-09 17:45:57 +09:00
parent a2669e08c4
commit 5c8f553f87
13 changed files with 317 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

72
api.hyungi.net/models/workerModel.js
View File

@@ -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);

8
web-ui/js/attendance.js
View File

@@ -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('작업자 불러오기 실패');

9
web-ui/js/daily-issue-api.js
View File

@@ -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) {

21
web-ui/js/daily-work-report.js
View File

@@ -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 = `
<div class="work-entry-header">
<div class="work-entry-title">작업 항목 #${workEntryCounter}</div>
<button type="button" class="remove-work-btn" onclick="removeWorkEntry(${workEntryCounter})" title="이 작업 삭제">
✕
<button type="button" class="remove-work-btn" onclick="event.stopPropagation(); removeWorkEntry(${workEntryCounter})" title="이 작업 삭제">
🗑️ 삭제
</button>
</div>
@@ -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('❌ 작업 항목을 찾을 수 없음');
}
}

12
web-ui/js/manage-user.js
View File

@@ -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');

10
web-ui/js/management-dashboard.js
View File

@@ -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;

10
web-ui/js/modern-dashboard.js
View File

@@ -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);

8
web-ui/js/project-analysis-api.js
View File

@@ -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);

9
web-ui/js/work-report-api.js
View File

@@ -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);

12
web-ui/js/work-report-calendar.js
View File

@@ -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);

7
web-ui/js/work-report-manage.js
View File

@@ -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('잘못된 데이터 형식');

8
web-ui/js/work-report-review.js
View File

@@ -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들 로드 (실패해도 기본값 사용)

167
개발 log/2025-12-09.md Normal file
View File

@@ -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] 페이지 새로고침 후 상태 유지
- [ ] 모든 페이지에서 비활성 작업자 숨김 확인 필요
---