@happychef/algorithm 1.2.6 → 1.2.7

package/getAvailableTimeblocks.js

@@ -98,7 +98,8 @@ function getAvailableTimeblocks(data, dateStr, reservations, guests, blockedSlot
   }
 
   // Apply last booking time filter (lunchStop, dinerStop, ontbijtStop) - skip for admin
-  if (!isAdmin) {
+  // IMPORTANT: This filter should only apply to TODAY, just like uurOpVoorhand
+  if (!isAdmin && isToday) {
     const settings = data?.['general-settings'] || {};
     const breakfastStop = settings.ontbijtStop || null;
     const lunchStop = settings.lunchStop || null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happychef/algorithm",
-  "version": "1.2.6",
+  "version": "1.2.7",
   "description": "Restaurant and reservation algorithm utilities",
   "main": "index.js",
   "author": "happy chef",

package/test-detailed-filter.js

@@ -0,0 +1,100 @@
+// Detailed filter test - simulating getAvailableTimeblocks logic
+const { timeblocksAvailable } = require('./processing/timeblocksAvailable');
+const { getMealTypeByTime, parseTime } = require('./tableHelpers');
+
+const restaurantData = {
+  "_id": "demo",
+  "openinghours-lunch": {
+    "schemeSettings": {
+      "Monday": {
+        "enabled": true,
+        "startTime": "14:15",
+        "endTime": "15:00",
+        "maxCapacityEnabled": true,
+        "maxCapacity": "10"
+      }
+    }
+  },
+  "general-settings": {
+    "zitplaatsen": 13,
+    "uurOpVoorhand": 0,
+    "dagenInToekomst": 365,
+    "intervalReservatie": 30,
+    "duurReservatie": 60,
+    "ontbijtStop": "",
+    "lunchStop": "",
+    "dinerStop": ""
+  },
+  "max-arrivals-lunch": {
+    "14:15": 10,
+    "14:30": 10,
+    "14:45": 10
+  }
+};
+
+const testDate = "2025-12-08";
+const guests = 2;
+const reservations = [];
+const uurOpVoorhand = 0;
+const isAdmin = false;
+
+console.log('=== DETAILED FILTER TEST ===\n');
+
+// Get timeblocks
+const availableTimeblocks = timeblocksAvailable(restaurantData, testDate, reservations, guests, [], null, isAdmin);
+console.log('Initial timeblocks:', Object.keys(availableTimeblocks));
+
+// Simulate the time filtering logic
+const now = new Date();
+const [year, month, day] = testDate.split('-').map(Number);
+const targetDateInTimeZone = new Date(year, month - 1, day);
+const isToday = now.toDateString() === targetDateInTimeZone.toDateString();
+
+console.log('\nTime filter check:');
+console.log('Is today:', isToday);
+console.log('uurOpVoorhand:', uurOpVoorhand);
+console.log('Current time:', now.toString());
+
+if (isToday && uurOpVoorhand >= 0) {
+  const cutoffTime = new Date(now.getTime());
+  cutoffTime.setHours(cutoffTime.getHours() + uurOpVoorhand);
+  console.log('Cutoff time:', cutoffTime.toString());
+
+  for (const key of Object.keys(availableTimeblocks)) {
+    const timeStr = key;
+    const [hours, minutes] = timeStr.split(':').map(Number);
+    const timeBlockDateTime = new Date(year, month - 1, day, hours, minutes);
+
+    console.log(`\n  Checking ${timeStr}:`);
+    console.log(`    timeBlockDateTime: ${timeBlockDateTime.toString()}`);
+    console.log(`    cutoffTime: ${cutoffTime.toString()}`);
+    console.log(`    timeBlockDateTime < cutoffTime: ${timeBlockDateTime < cutoffTime}`);
+    console.log(`    Would be deleted: ${timeBlockDateTime < cutoffTime ? 'YES' : 'NO'}`);
+  }
+}
+
+// Check lunchStop filter
+console.log('\n\nLunch Stop filter check:');
+const settings = restaurantData['general-settings'];
+const lunchStop = settings.lunchStop || null;
+console.log('lunchStop value:', JSON.stringify(lunchStop));
+console.log('lunchStop truthy:', !!lunchStop);
+
+if (lunchStop) {
+  console.log('LunchStop is set, would apply filter');
+  for (const time in availableTimeblocks) {
+    const mealType = getMealTypeByTime(time);
+    console.log(`  ${time}: mealType=${mealType}`);
+
+    if (mealType === 'lunch') {
+      const timeMinutes = parseTime(time);
+      const stopMinutes = parseTime(lunchStop);
+      console.log(`    timeMinutes=${timeMinutes}, stopMinutes=${stopMinutes}`);
+      console.log(`    Would remove: ${timeMinutes >= stopMinutes ? 'YES' : 'NO'}`);
+    }
+  }
+} else {
+  console.log('LunchStop is NOT set, no filter applied');
+}
+
+console.log('\n=== END TEST ===');

package/test-lunch-debug.js

@@ -0,0 +1,110 @@
+// Test script to debug why Monday lunch isn't showing
+const { getAvailableTimeblocks } = require('./getAvailableTimeblocks');
+const { timeblocksAvailable } = require('./processing/timeblocksAvailable');
+const { getDailyGuestCounts } = require('./processing/dailyGuestCounts');
+
+// Your restaurant data
+const restaurantData = {
+  "_id": "demo",
+  "openinghours-lunch": {
+    "schemeSettings": {
+      "Monday": {
+        "enabled": true,
+        "startTime": "14:15",
+        "endTime": "15:00",
+        "maxCapacityEnabled": true,
+        "maxCapacity": "10",
+        "shiftsEnabled": false,
+        "shifts": [],
+        "giftcardsEnabled": false,
+        "giftcards": [],
+        "giftcardTakeawayEnabled": false,
+        "giftcardTakeawayStart": "",
+        "giftcardTakeawayEnd": ""
+      },
+      "Wednesday": {
+        "enabled": true,
+        "startTime": "13:45",
+        "endTime": "14:45",
+        "maxCapacityEnabled": false,
+        "maxCapacity": "",
+        "shiftsEnabled": false,
+        "shifts": [],
+        "giftcardsEnabled": false,
+        "giftcards": [],
+        "giftcardTakeawayEnabled": true,
+        "giftcardTakeawayStart": "11:00",
+        "giftcardTakeawayEnd": "16:00"
+      }
+    }
+  },
+  "general-settings": {
+    "zitplaatsen": 13,
+    "uurOpVoorhand": 0,
+    "dagenInToekomst": 365,
+    "minGasten": 1,
+    "maxGasten": 10,
+    "intervalReservatie": 30,
+    "duurReservatie": 60,
+    "ontbijtStop": "",
+    "lunchStop": "",
+    "dinerStop": ""
+  },
+  "max-arrivals-lunch": {
+    "12:00": 6,
+    "12:15": 1,
+    "12:30": 6,
+    "12:45": 4,
+    "13:00": 10,
+    "14:15": 10,
+    "14:30": 10,
+    "14:45": 10,
+    "storedNumber": 30
+  },
+  "exceptions": [],
+  "blocked-slots": []
+};
+
+// Test for Monday December 8, 2025
+const testDate = "2025-12-08"; // Monday
+const guests = 2;
+const reservations = [];
+
+console.log('=== LUNCH AVAILABILITY DEBUG ===\n');
+console.log('Date:', testDate, '(Monday)');
+console.log('Guests:', guests);
+console.log('Current time:', new Date().toISOString());
+console.log('\n--- Restaurant Settings ---');
+console.log('Lunch Opening Hours (Monday):', restaurantData['openinghours-lunch'].schemeSettings.Monday);
+console.log('General Settings:', restaurantData['general-settings']);
+console.log('Max Arrivals Lunch:', restaurantData['max-arrivals-lunch']);
+
+console.log('\n--- Step 1: getDailyGuestCounts ---');
+const { guestCounts, shiftsInfo } = getDailyGuestCounts(restaurantData, testDate, reservations);
+console.log('Guest Counts:', guestCounts);
+console.log('Shifts Info:', shiftsInfo);
+
+console.log('\n--- Step 2: timeblocksAvailable ---');
+const timeblocks = timeblocksAvailable(restaurantData, testDate, reservations, guests, [], null, false);
+console.log('Available Timeblocks:', timeblocks);
+
+console.log('\n--- Step 3: getAvailableTimeblocks (with filters) ---');
+const finalTimeblocks = getAvailableTimeblocks(restaurantData, testDate, reservations, guests, [], null, false);
+console.log('Final Available Timeblocks:', finalTimeblocks);
+
+console.log('\n--- Step 4: Test with Admin Flag ---');
+const adminTimeblocks = getAvailableTimeblocks(restaurantData, testDate, reservations, guests, [], null, true);
+console.log('Admin Timeblocks (no time restrictions):', adminTimeblocks);
+
+// Check if lunch times exist
+const lunchTimes = ['14:15', '14:30', '14:45'];
+console.log('\n--- Lunch Time Analysis ---');
+lunchTimes.forEach(time => {
+  console.log(`${time}:`);
+  console.log(`  - In guestCounts: ${guestCounts.hasOwnProperty(time) ? 'YES (capacity: ' + guestCounts[time] + ')' : 'NO'}`);
+  console.log(`  - In timeblocks: ${timeblocks.hasOwnProperty(time) ? 'YES' : 'NO'}`);
+  console.log(`  - In final: ${finalTimeblocks.hasOwnProperty(time) ? 'YES' : 'NO'}`);
+  console.log(`  - In admin: ${adminTimeblocks.hasOwnProperty(time) ? 'YES' : 'NO'}`);
+});
+
+console.log('\n=== END DEBUG ===');

package/test-max-arrivals-filter.js

@@ -0,0 +1,79 @@
+// Test the max arrivals filter specifically
+const { filterTimeblocksByMaxArrivals } = require('./filters/maxArrivalsFilter');
+const { getAvailableTimeblocks } = require('./getAvailableTimeblocks');
+
+const restaurantData = {
+  "_id": "demo",
+  "openinghours-lunch": {
+    "schemeSettings": {
+      "Monday": {
+        "enabled": true,
+        "startTime": "14:15",
+        "endTime": "15:00",
+        "maxCapacityEnabled": true,
+        "maxCapacity": "10"
+      }
+    }
+  },
+  "general-settings": {
+    "zitplaatsen": 13,
+    "uurOpVoorhand": 0,
+    "dagenInToekomst": 365,
+    "intervalReservatie": 30,
+    "duurReservatie": 60,
+    "ontbijtStop": "",
+    "lunchStop": "",
+    "dinerStop": ""
+  },
+  "max-arrivals-lunch": {
+    "12:00": 6,
+    "12:15": 1,
+    "12:30": 6,
+    "12:45": 4,
+    "13:00": 10,
+    "14:15": 10,
+    "14:30": 10,
+    "14:45": 10,
+    "storedNumber": 30
+  }
+};
+
+const testDate = "2025-12-08";
+const guests = 2;
+const reservations = [];
+
+console.log('=== MAX ARRIVALS FILTER TEST ===\n');
+
+// Get initial timeblocks from algorithm
+const timeblocks = getAvailableTimeblocks(restaurantData, testDate, reservations, guests, [], null, true);
+console.log('Timeblocks from algorithm (admin mode):', Object.keys(timeblocks));
+
+// Apply max arrivals filter
+const filtered = filterTimeblocksByMaxArrivals(restaurantData, testDate, timeblocks, reservations, guests);
+console.log('\nAfter max arrivals filter:', Object.keys(filtered));
+
+// Debug each time slot
+console.log('\n--- Detailed Analysis ---');
+for (const time of Object.keys(timeblocks)) {
+  const maxArrivalsConfig = restaurantData['max-arrivals-lunch'];
+  const maxArrivals = maxArrivalsConfig ? maxArrivalsConfig[time] : null;
+
+  const currentArrivals = reservations
+    .filter(r => r.date === testDate && r.time === time)
+    .reduce((sum, r) => sum + (parseInt(r.guests, 10) || 0), 0);
+
+  const isInFiltered = filtered.hasOwnProperty(time);
+
+  console.log(`\n${time}:`);
+  console.log(`  Max arrivals setting: ${maxArrivals || 'NOT SET'}`);
+  console.log(`  Current arrivals: ${currentArrivals}`);
+  console.log(`  New reservation: ${guests} guests`);
+  console.log(`  Total after adding: ${currentArrivals + guests}`);
+  console.log(`  Passes filter: ${isInFiltered ? 'YES' : 'NO'}`);
+
+  if (maxArrivals !== null && maxArrivals !== undefined) {
+    console.log(`  Logic: ${currentArrivals} + ${guests} = ${currentArrivals + guests} ${currentArrivals + guests <= maxArrivals ? '<=' : '>'} ${maxArrivals}`);
+  }
+}
+
+console.log('\n=== END TEST ===');

package/test-timezone-debug.js

@@ -0,0 +1,47 @@
+// Test timezone handling
+const moment = require('moment-timezone');
+
+const testDate = "2025-12-08";
+const testTime = "14:15";
+
+console.log('=== TIMEZONE DEBUG ===\n');
+
+// What the algorithm does
+const now = new Date();
+console.log('System now:', now);
+console.log('System now ISO:', now.toISOString());
+console.log('System now local string:', now.toString());
+
+// Parse the time as the algorithm does (line 68-69 in getAvailableTimeblocks.js)
+const [year, month, day] = testDate.split('-').map(Number);
+const targetDateInTimeZone = new Date(year, month - 1, day);
+console.log('\nTarget date:', targetDateInTimeZone);
+console.log('Target date string:', targetDateInTimeZone.toString());
+
+const isToday = now.toDateString() === targetDateInTimeZone.toDateString();
+console.log('\nIs today?:', isToday);
+
+// With uurOpVoorhand = 0
+const uurOpVoorhand = 0;
+const cutoffTime = new Date(now.getTime());
+cutoffTime.setHours(cutoffTime.getHours() + uurOpVoorhand);
+console.log('\nCutoff time (now + 0 hours):', cutoffTime);
+console.log('Cutoff time string:', cutoffTime.toString());
+
+// Parse the test time
+const [hours, minutes] = testTime.split(':').map(Number);
+const timeBlockDateTime = new Date(year, month - 1, day, hours, minutes);
+console.log('\nTime block (14:15):', timeBlockDateTime);
+console.log('Time block string:', timeBlockDateTime.toString());
+
+console.log('\nComparison:');
+console.log('timeBlockDateTime < cutoffTime:', timeBlockDateTime < cutoffTime);
+console.log('Should be filtered out?:', timeBlockDateTime < cutoffTime ? 'YES' : 'NO');
+
+// Check with Europe/Brussels timezone
+console.log('\n=== Using moment-timezone ===');
+const nowBrussels = moment.tz('Europe/Brussels');
+console.log('Now in Brussels:', nowBrussels.format('YYYY-MM-DD HH:mm:ss Z'));
+const targetTimeBrussels = moment.tz(`${testDate} ${testTime}`, 'YYYY-MM-DD HH:mm', 'Europe/Brussels');
+console.log('Target time in Brussels:', targetTimeBrussels.format('YYYY-MM-DD HH:mm:ss Z'));
+console.log('Is target time in past?:', targetTimeBrussels.isBefore(nowBrussels));