Merge branch 'hi' into 'main'

Hi

See merge request !15

controllers/TripController.js

@@ -66,24 +66,38 @@ const addCollaborator = async (req, res) => {
     try {
         const { collaboratorEmail } = req.body;
         
-        // Verify if the collaboratorEmail corresponds to a valid user
-        const user = await User.findOne({ email: collaboratorEmail });
-        if (!user) {
-            return res.status(404).json({ message: 'User not found' });
+        const collaborator = await User.findOne({ email: collaboratorEmail });
+        if (!collaborator) {
+            return res.status(404).json({ message: '해당 이메일의 사용자를 찾을 수 없습니다.' });
         }
 
-        const trip = await Trip.findById(req.params.id);
-        if (!trip) return res.status(404).json({ message: 'Trip not found' });
+        const trip = await Trip.findById(req.params.id).populate('create_by');
+        if (!trip) {
+            return res.status(404).json({ message: '여행을 찾을 수 없습니다.' });
+        }
+
+        const isAlreadyCollaborator = trip.collaborators.some(
+            collab => collab.toString() === collaborator._id.toString()
+        );
         
-        // Check if the user is already a collaborator
-        if (trip.collaborators.includes(user._id)) {
-            return res.status(400).json({ message: 'User is already a collaborator' });
+        if (isAlreadyCollaborator) {
+            return res.status(400).json({ message: '이미 공동작업자로 등록된 사용자입니다.' });
         }
 
-        trip.collaborators.push(user._id);
+        if (trip.create_by._id.toString() === collaborator._id.toString()) {
+            return res.status(400).json({ message: '생성자는 공동작업자로 추가할 수 없습니다.' });
+        }
+
+        trip.collaborators.push(collaborator._id);
         await trip.save();
-        res.status(200).json(trip);
+        
+        const updatedTrip = await Trip.findById(trip._id)
+            .populate('create_by')
+            .populate('collaborators');
+            
+        res.status(200).json(updatedTrip);
     } catch (err) {
+        console.error('공동작업자 추가 오류:', err);
         res.status(500).json({ error: err.message });
     }
 };

models/trips.js

@@ -26,7 +26,10 @@ const tripSchema = new mongoose.Schema({
     end_date: { type: Date, required: true },
     location: { type: String, required: true },
     create_by: { type: mongoose.Schema.Types.ObjectId, ref: 'User', required: true },
-    collaborators: [{ type: mongoose.Schema.Types.ObjectId, ref: 'User' }],
+    collaborators: [{
+        type: mongoose.Schema.Types.ObjectId,
+        ref: 'User'
+    }],
     plans: {
         type: Map, // Object 대신 Map 사용
         of: {

route/tripRoute.js

@@ -3,243 +3,28 @@ const router = express.Router();
 const Trip = require('../models/trips'); // Trip 스키마 참조
 const User = require('../models/user'); // User 스키마 참조
 
-/*
-테스트 방법:
 
-1. 전체 여행 목록 조회 테스트 (GET /api/trips/trips)
-- GET http://localhost:3000/api/trips/trips 요청
-- 응답으로 전체 여행 목록이 반환되는지 확인
-
-2. 특정 여행 조회 테스트 (GET /api/trips/trips/:id) 
-- GET http://localhost:3000/api/trips/trips/{여행ID} 요청
-- 해당 ID의 여행 정보가 반환되는지 확인
-
-3. 새 여행 생성 테스트 (POST /api/trips/trips)
-- POST http://localhost:3000/api/trips/trips
-- Body: {
-    "name": "테스트 여행",
-    "start_date": "2023-12-01",
-    "end_date": "2023-12-03", 
-    "create_by": "사용자ID",
-    "collaborators": ["협업자ID1", "협업자ID2"]
-  }
-
-4. 여행 수정 테스트 (PUT /api/trips/trips/:id)
-- PUT http://localhost:3000/api/trips/trips/{여행ID}
-- Body: {
-    "name": "수정된 여행명",
-    "start_date": "2023-12-02",
-    "end_date": "2023-12-04",
-    "collaborators": ["협업자ID1"]
-  }
-
-5. 여행 삭제 테스트 (DELETE /api/trips/trips/:id)
-- DELETE http://localhost:3000/api/trips/trips/{여행ID}
-- 삭제 성공 메시지 확인
-
-6. 공동 작업자 추가 테스트 (POST /api/trips/trips/:id/collaborators)
-- POST http://localhost:3000/api/trips/trips/{여행ID}/collaborators
-- Body: {
-    "collaboratorId": "새로운협업자ID"
-  }
-
-7. 하루 계획 추가 테스트 (POST /api/trips/trips/:id/plans)
-- POST http://localhost:3000/api/trips/trips/{여행ID}/plans
-- Body: {
-    "dayKey": "day1",
-    "places": [
-      {
-        "id": 1,
-        "name": "장소1",
-        "location": "위치1",
-        "coordinates": {
-          "lat": 37.5665,
-          "lng": 126.9780
-        }
-      }
-    ],
-    "route": [1]
-  }
-
-테스트 도구: Postman 또는 curl 사용 권장
-*/
+const tripController = require('../controllers/TripController');
 
 // **GET**: 전체 여행 목록 조회
-router.get('/trips', async (req, res) => {
-    try {
-        const trips = await Trip.find().populate('create_by collaborators');
-        res.status(200).json(trips);
-        console.log('여행 목록 조회 내역', trips);
-    } catch (err) {
-        console.error('여행 계획 조회 실패', err);
-        res.status(500).json({ error: err.message });
-    }
-});
+router.get('/trips', tripController.getAllTrips);
 
 // **GET**: 특정 여행 계획 조회
-router.get('/trips/:id', async (req, res) => {
-    try {
-        const trip = await Trip.findById(req.params.id).populate('create_by collaborators');
-        if (!trip) return res.status(404).json({ message: 'Trip not found' });
-        res.status(200).json(trip);
-    } catch (err) {
-        console.error('여행 계획 조회 실패', err);
-        res.status(500).json({ error: err.message });
-    }
-});
+router.get('/trips/:id', tripController.getTripById);
 
 // **POST**: 새로운 여행 계획 생성
-router.post('/trips', async (req, res) => {
-    try {
-        const { name, start_date, end_date, create_by, location, collaborators } = req.body;
-        const trip = new Trip({
-            name,
-            start_date,
-            end_date,
-            create_by,
-            location,
-            collaborators
-        });
-        const savedTrip = await trip.save();
-        res.status(201).json(savedTrip);
-    } catch (err) {
-        console.error('여행 계획 작성 실패', err);
-        res.status(500).json({ error: err.message });
-    }
-});
-
-// // **PUT**: 특정 여행 계획 수정
-// router.put('/trips/:id', async (req, res) => {
-//     try {
-//         const { name, start_date, end_date, location, collaborators } = req.body;
-//         const updatedTrip = await Trip.findByIdAndUpdate(
-//             req.params.id,
-//             { name, start_date, end_date, location, collaborators },
-//             { new: true }
-//         );
-//         if (!updatedTrip) return res.status(404).json({ message: 'Trip not found' });
-//         res.status(200).json(updatedTrip);
-//     } catch (err) {
-//         console.err('여행계획 수정 실패', err);
-//         res.status(500).json({ error: err.message });
-//     }
-// });
+router.post('/trips', tripController.createTrip);
 
 // **PUT**: 특정 여행 계획 수정(협업자도 수정할 수 있게)
-router.put('/trips/:id', async (req, res) => {
-    try {
-        const { name, start_date, end_date, location, collaborators } = req.body;
-
-        // 여행 계획 조회
-        const trip = await Trip.findById(req.params.id);
-        if (!trip) {
-            return res.status(404).json({ message: 'Trip not found' });
-        }
-
-        // 권한 확인: 생성자나 협업자인 경우에만 수정 허용
-        const userId = req.user._id; // Passport 세션을 통해 인증된 사용자 ID
-        const isAuthorized =
-            trip.create_by.toString() === userId.toString() || 
-            trip.collaborators.includes(userId.toString());
-
-        if (!isAuthorized) {
-            return res.status(403).json({ message: '수정 권한이 없습니다.' });
-        }
-
-        // MongoDB의 `findByIdAndUpdate` 함수로 업데이트
-        const updatedTrip = await Trip.findByIdAndUpdate(
-            req.params.id,
-            { name, start_date, end_date, location, collaborators },
-            { new: true } // 업데이트 후 새로운 문서를 반환
-        );
-
-        res.status(200).json(updatedTrip);
-    } catch (err) {
-        console.error('여행 계획 수정 실패:', err);
-        res.status(500).json({ error: err.message });
-    }
-});
-
-
+router.put('/trips/:id', tripController.updateTrip);
 
 // **DELETE**: 특정 여행 계획 삭제
-router.delete('/trips/:id', async (req, res) => {
-    try {
-        const deletedTrip = await Trip.findById(req.params.id);
-        if (!deletedTrip) return res.status(404).json({ message: 'Trip not found' });
-
-        // 요청한 사용자가 여행의 생성자인지 확인
-        const userId = req.user._id; // Passport 세션을 통해 인증된 사용자 ID
-        if (trip.create_by.toString() !== userId.toString()) {
-            console.log('삭제 권한이 없습니다');
-            return res.status(403).json({ message: '삭제 권한이 없습니다.' });
-        }
-
-        await Trip.findByIdAndDelete(req.params.id);
-        res.status(200).json({ message: 'Trip deleted successfully' });
-    } catch (err) {
-        console.error('여행 계획 삭제 실패: ', err);
-        res.status(500).json({ error: err.message });
-    }
-});
+router.delete('/trips/:id', tripController.deleteTrip);
 
 // **POST**: 공동 작업자 추가
-router.post('/trips/:id/collaborators', async (req, res) => {
-    try {
-        const { collaboratorId } = req.body;
-        const trip = await Trip.findById(req.params.id);
-        if (!trip) return res.status(404).json({ message: 'Trip not found' });
-        trip.collaborators.push(collaboratorId);
-        await trip.save();
-        res.status(200).json(trip);
-    } catch (err) {
-        console.error('공동 작업자 추가 실패: ',err);
-        res.status(500).json({ error: err.message });
-    }
-});
-
-router.put('/trips/:id/plans/day', async (req, res) => {
-    try {
-        const { dayKey, places, route } = req.body; // 요청 데이터
-        const trip = await Trip.findById(req.params.id); // Trip 찾기
-
-        if (!trip) {
-            return res.status(404).json({ message: 'Trip not found' });
-        }
-
-        // `plans` 디버깅
-        console.log('Existing Plans:', JSON.stringify(trip.plans, null, 2));
-
-        // `Map`을 사용하여 dayKey 접근
-        const currentDay = trip.plans.get(dayKey);
-        if (!currentDay) {
-            return res.status(404).json({ message: `Day ${dayKey} not found in trip plans` });
-        }
-
-        // 데이터 업데이트
-        if (places) currentDay.places = places;
-        if (route) currentDay.route = route;
-
-        // `Map`에 업데이트된 day 데이터 설정
-        trip.plans.set(dayKey, currentDay);
-
-        // Mongoose에 변경 사항 알림
-        trip.markModified('plans');
-
-        console.log('Before Save:', JSON.stringify(trip.plans.get(dayKey), null, 2));
-
-        // 변경 사항 저장
-        const updatedTrip = await trip.save();
-
-        console.log('After Save:', JSON.stringify(updatedTrip.plans.get(dayKey), null, 2));
-
-        res.status(200).json(updatedTrip.plans.get(dayKey)); // 수정된 day 데이터 반환
-    } catch (err) {
-        console.error('Error updating day:', err);
-        res.status(500).json({ error: err.message });
-    }
-});
-
+router.post('/trips/:id/collaborators', tripController.addCollaborator);
 
+// **PUT**: 하루 계획 추가
+router.put('/trips/:id/plans/day', tripController.addDayPlan);
 
 module.exports = router;
\ No newline at end of file