fix: round up to full days for lowstock estimate

Author: BolverBlitz

config/notifications/core.js

@@ -90,10 +90,13 @@ module.exports = [
                 buildContext: (task) => {
                     const context = JSON.parse(task.custom_message);
                     const estimatedDays = context.trend?.estimatedDaysRemaining;
+                    const estimatedDuration = estimatedDays
+                        ? `ca. ${estimatedDays} ${estimatedDays === 1 ? 'Tag' : 'Tage'}`
+                        : 'Trend unbekannt';
                     return {
                         ...context,
                         itemName: context.item?.name || '',
-                        estimatedDuration: estimatedDays ? `ca. ${estimatedDays} Tage` : 'Trend unbekannt',
+                        estimatedDuration,
                     };
                 },
                 buildText: (context) => [
@@ -104,7 +107,7 @@ module.exports = [
                     context.trend?.averagePerDay > 0
                         ? context.t('emails.LowStock.estimateText', {
                             averagePerDay: context.trend.averagePerDay,
-                            days: context.trend.estimatedDaysRemaining,
+                            estimatedDuration: context.estimatedDuration,
                         })
                         : context.t('emails.LowStock.noTrendText'),
                     '',

config/templates/email/LowStock.ejs

@@ -14,7 +14,7 @@
     <% if (trend && trend.averagePerDay > 0) { %>
         <p><%= t('emails.LowStock.estimate', {
             averagePerDay: trend.averagePerDay,
-            days: trend.estimatedDaysRemaining
+            estimatedDuration
         }) %></p>
     <% } else { %>
         <p><%= t('emails.LowStock.noTrend') %></p>

config/templates/email/locales/de.json

@@ -34,8 +34,8 @@
             "body": "Ein Artikel ist unter den konfigurierten Warenbestand gefallen.",
             "item": "Artikel",
             "stock": "Bestand",
-            "estimate": "Trend: Ø {{averagePerDay}} Stück pro Tag. Geschätzt reicht der Bestand noch {{days}} Tage.",
-            "estimateText": "Trend: Ø {{averagePerDay}} Stück pro Tag. Geschätzt noch {{days}} Tage.",
+            "estimate": "Trend: Ø {{averagePerDay}} Stück pro Tag. Geschätzt reicht der Bestand noch {{estimatedDuration}}.",
+            "estimateText": "Trend: Ø {{averagePerDay}} Stück pro Tag. Geschätzt noch {{estimatedDuration}}.",
             "noTrend": "Noch kein Verkaufs-Trend vorhanden.",
             "noTrendText": "Noch kein Verkaufs-Trend vorhanden."
         },

lib/sqlite/adminNotifications.js

@@ -70,10 +70,12 @@ const getItemSalesTrend = (itemUuid, days = 30) => {
         LEFT JOIN daily_sales ON daily_sales.sale_date = days.sale_date
     `).get(safeDays, safeDays, safeDays, itemUuid, `-${safeDays - 1} days`);
 
+    const weightedAveragePerDay = Number(row?.weighted_average_per_day || 0);
+
     return {
         days: safeDays,
         soldQuantity: Number(row?.sold_quantity || 0),
-        averagePerDay: Number(Number(row?.weighted_average_per_day || 0).toFixed(2)),
+        averagePerDay: weightedAveragePerDay > 0 ? Math.ceil(weightedAveragePerDay) : 0,
     };
 };