chpark/vexplor
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: update production controller to use AuthenticatedRequest

Browse Source 
- Modified the production controller to replace the generic Request type with AuthenticatedRequest for better type safety and to ensure user authentication is handled correctly.
- This change enhances the security and clarity of the API endpoints related to production plan management, ensuring that user-specific data is accessed appropriately.

Made-with: Cursor
This commit is contained in:
kjs
2026-03-16 09:35:23 +09:00
parent ba39ebf341
commit 79e3420ff3
5 changed files with 60 additions and 59 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
backend-node/src/controllers/productionController.ts
View File

@@ -2,13 +2,14 @@
* 생산계획 컨트롤러
*/
import { Request, Response } from "express";
import { Response } from "express";
import { AuthenticatedRequest } from "../types/auth";
import * as productionService from "../services/productionPlanService";
import { logger } from "../utils/logger";
// ─── 수주 데이터 조회 (품목별 그룹핑) ───
export async function getOrderSummary(req: Request, res: Response) {
export async function getOrderSummary(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const { excludePlanned, itemCode, itemName } = req.query;
@@ -28,7 +29,7 @@ export async function getOrderSummary(req: Request, res: Response) {
// ─── 안전재고 부족분 조회 ───
export async function getStockShortage(req: Request, res: Response) {
export async function getStockShortage(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const data = await productionService.getStockShortage(companyCode);
@@ -41,7 +42,7 @@ export async function getStockShortage(req: Request, res: Response) {
// ─── 생산계획 상세 조회 ───
export async function getPlanById(req: Request, res: Response) {
export async function getPlanById(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const planId = parseInt(req.params.id, 10);
@@ -59,7 +60,7 @@ export async function getPlanById(req: Request, res: Response) {
// ─── 생산계획 수정 ───
export async function updatePlan(req: Request, res: Response) {
export async function updatePlan(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const planId = parseInt(req.params.id, 10);
@@ -78,7 +79,7 @@ export async function updatePlan(req: Request, res: Response) {
// ─── 생산계획 삭제 ───
export async function deletePlan(req: Request, res: Response) {
export async function deletePlan(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const planId = parseInt(req.params.id, 10);
@@ -96,7 +97,7 @@ export async function deletePlan(req: Request, res: Response) {
// ─── 자동 스케줄 생성 ───
export async function generateSchedule(req: Request, res: Response) {
export async function generateSchedule(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const createdBy = req.user!.userId;
@@ -116,7 +117,7 @@ export async function generateSchedule(req: Request, res: Response) {
// ─── 스케줄 병합 ───
export async function mergeSchedules(req: Request, res: Response) {
export async function mergeSchedules(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const mergedBy = req.user!.userId;
@@ -142,7 +143,7 @@ export async function mergeSchedules(req: Request, res: Response) {
// ─── 반제품 계획 자동 생성 ───
export async function generateSemiSchedule(req: Request, res: Response) {
export async function generateSemiSchedule(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const createdBy = req.user!.userId;
@@ -167,7 +168,7 @@ export async function generateSemiSchedule(req: Request, res: Response) {
// ─── 스케줄 분할 ───
export async function splitSchedule(req: Request, res: Response) {
export async function splitSchedule(req: AuthenticatedRequest, res: Response) {
try {
const companyCode = req.user!.companyCode;
const splitBy = req.user!.userId;