fixed scheduling

backend/src/services/SchedulingService.ts

@@ -52,6 +52,55 @@ export class SchedulingService {
     const shifts = this.prepareShifts(shiftPlan);
     const workerAvailabilities = this.prepareAvailabilities(availabilities, shiftPlan);
     
+    // 🆕 ENHANCED DATA VALIDATION
+    console.log('\n🔍 ===== ENHANCED DATA VALIDATION =====');
+    console.log(`Shift Plan: ${shiftPlan.name} (${shiftPlan.id})`);
+    console.log(`Template: ${shiftPlan.isTemplate}`);
+    console.log(`Generated shifts: ${shifts.length}`);
+    console.log(`Input availabilities: ${availabilities.length}`);
+    console.log(`Mapped availabilities: ${workerAvailabilities.length}`);
+    
+    // Check shift ID patterns
+    const shiftIdsFromShifts = shifts.map(s => s.id);
+    const shiftIdsFromAvailabilities = [...new Set(workerAvailabilities.map(a => a.shiftId))];
+    
+    console.log(`Shift IDs in generated shifts: ${shiftIdsFromShifts.length}`);
+    console.log(`Unique shift IDs in availabilities: ${shiftIdsFromAvailabilities.length}`);
+    
+    // Find matching shift IDs
+    const matchingShiftIds = shiftIdsFromAvailabilities.filter(availId => 
+      shiftIdsFromShifts.includes(availId)
+    );
+    
+    console.log(`✅ Matching shift IDs: ${matchingShiftIds.length}/${shiftIdsFromAvailabilities.length}`);
+    
+    // Show first few matches for verification
+    if (matchingShiftIds.length > 0) {
+      console.log('🔍 FIRST 5 MATCHING SHIFT IDs:');
+      matchingShiftIds.slice(0, 5).forEach(id => {
+        const shift = shifts.find(s => s.id === id);
+        const availCount = workerAvailabilities.filter(a => a.shiftId === id).length;
+        console.log(`   - ${id}: ${availCount} availabilities, Date: ${shift?.date}, TimeSlot: ${shift?.timeSlotId}`);
+      });
+    }
+    
+    // Show unmatched availabilities for debugging
+    const unmatchedAvailabilities = workerAvailabilities.filter(avail => 
+      !shiftIdsFromShifts.includes(avail.shiftId)
+    );
+    
+    if (unmatchedAvailabilities.length > 0) {
+      console.log('❌ UNMATCHED AVAILABILITIES:');
+      const uniqueUnmatched = [...new Set(unmatchedAvailabilities.map(a => a.shiftId))];
+      uniqueUnmatched.slice(0, 5).forEach(shiftId => {
+        const count = unmatchedAvailabilities.filter(a => a.shiftId === shiftId).length;
+        console.log(`   - ${shiftId}: ${count} availabilities`);
+      });
+      if (uniqueUnmatched.length > 5) console.log(`   ... and ${uniqueUnmatched.length - 5} more unique unmatched shift IDs`);
+    }
+    
+    console.log('===== END ENHANCED DATA VALIDATION =====\n');
+    
     return {
       shiftPlan: {
         id: shiftPlan.id,
@@ -97,16 +146,16 @@ export class SchedulingService {
       const currentDate = new Date(startDate);
       currentDate.setDate(startDate.getDate() + dayOffset);
       
-      const dayOfWeek = currentDate.getDay() === 0 ? 7 : currentDate.getDay(); // Convert Sunday from 0 to 7
+      const dayOfWeek = currentDate.getDay() === 0 ? 7 : currentDate.getDay();
       const dayShifts = shiftPlan.shifts.filter(shift => shift.dayOfWeek === dayOfWeek);
       
       dayShifts.forEach(shift => {
-        // ✅ Use day-of-week pattern instead of date-based pattern
+        const shiftId = shift.id; // Use the original shift pattern ID
-        const shiftId = `${shift.id}`;
+        const dateStr = currentDate.toISOString().split('T')[0];
         
         shifts.push({
-          id: shiftId, // This matches what frontend expects
+          id: shiftId, // This matches the frontend availability records
-          date: currentDate.toISOString().split('T')[0],
+          date: dateStr,
           timeSlotId: shift.timeSlotId,
           requiredEmployees: shift.requiredEmployees,
           minWorkers: 1,
@@ -114,37 +163,42 @@ export class SchedulingService {
           isPriority: false
         });
 
-        console.log(`✅ Generated shift: ${shiftId} for day ${dayOfWeek}, timeSlot ${shift.timeSlotId}`);
+        console.log(`✅ Generated shift: ${shiftId} for date ${dateStr}, day ${dayOfWeek}, timeSlot ${shift.timeSlotId}`);
       });
     }
 
     console.log("Created shifts for one week. Amount: ", shifts.length);
+    
+    // Debug: Show which shift IDs we're using
+    console.log('🔍 SHIFT IDS IN GENERATED SHIFTS:');
+    shifts.forEach(shift => {
+      console.log(`   - ${shift.id} (Date: ${shift.date}, TimeSlot: ${shift.timeSlotId})`);
+    });
+    
     return shifts;
   }
 
   private prepareAvailabilities(availabilities: Availability[], shiftPlan: ShiftPlan): any[] {
-    return availabilities.map(avail => ({
+    console.log('🔄 Preparing availabilities for worker...');
-      employeeId: avail.employeeId,
+    console.log(`Input availabilities: ${availabilities.length} records`);
-      shiftId: this.findShiftIdForAvailability(avail, shiftPlan),
+    
-      availability: avail.preferenceLevel
+    const workerAvailabilities = availabilities.map(avail => {
-    }));
+      const shiftId = avail.shiftId;
+      
+      console.log(`📋 Availability ${avail.id}: employee=${avail.employeeId}, shift=${shiftId}, preference=${avail.preferenceLevel}`);
+      
+      return {
+        employeeId: avail.employeeId,
+        shiftId: shiftId, // Use the original shift ID from frontend
+        preferenceLevel: avail.preferenceLevel
+      };
+    });
+    console.log(`✅ Mapped ${workerAvailabilities.length} availabilities for worker`);
+    return workerAvailabilities;
   }
 
   private findShiftIdForAvailability(availability: Availability, shiftPlan: ShiftPlan): string {
-    // Find the corresponding scheduled shift ID for this availability
+    return availability.shiftId;
-    if (shiftPlan.scheduledShifts) {
-      const scheduledShift = shiftPlan.scheduledShifts.find(shift => 
-        shift.timeSlotId === availability.timeSlotId && 
-        this.getDayOfWeekFromDate(shift.date) === availability.dayOfWeek
-      );
-      
-      if (scheduledShift) {
-        return scheduledShift.id;
-      }
-    }
-    
-    // Fallback: generate a consistent ID
-    return `shift_${availability.dayOfWeek}_${availability.timeSlotId}`;
   }
 
   private getDayOfWeekFromDate(dateString: string): number {

frontend/src/pages/ShiftPlans/ShiftPlanView.tsx

@@ -95,6 +95,30 @@ const ShiftPlanView: React.FC = () => {
     console.log('🔍 STATE DEBUG - publishing:', publishing);
   }, [showAssignmentPreview, assignmentResult, publishing]);
 
+  const debugAvailabilityShiftIds = () => {
+    if (!availabilities.length) return;
+    
+    console.log('🔍 AVAILABILITY SHIFT ID ANALYSIS:');
+    const uniqueShiftIds = [...new Set(availabilities.map(a => a.shiftId))];
+    
+    console.log(`Unique shift IDs in availabilities: ${uniqueShiftIds.length}`);
+    uniqueShiftIds.forEach(shiftId => {
+      const count = availabilities.filter(a => a.shiftId === shiftId).length;
+      const pref1 = availabilities.filter(a => a.shiftId === shiftId && a.preferenceLevel === 1).length;
+      const pref2 = availabilities.filter(a => a.shiftId === shiftId && a.preferenceLevel === 2).length;
+      const pref3 = availabilities.filter(a => a.shiftId === shiftId && a.preferenceLevel === 3).length;
+      
+      console.log(`   ${shiftId}: ${count} total (✅${pref1} 🔶${pref2} ❌${pref3})`);
+    });
+  };
+
+  // Call this after loading availabilities
+  useEffect(() => {
+    if (availabilities.length > 0) {
+      debugAvailabilityShiftIds();
+    }
+  }, [availabilities]);
+
   // Create a data structure that maps days to their shifts with time slot info - SAME AS AVAILABILITYMANAGER
   const getTimetableData = () => {
     if (!shiftPlan || !shiftPlan.shifts || !shiftPlan.timeSlots) {