fleetmap-reports 1.0.262 → 1.0.266

package/.idea/workspace.xml

@@ -1,14 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project version="4">
   <component name="ChangeListManager">
-    <list default="true" id="eb665c86-c893-4333-bd2a-721dc27980b9" name="Changes" comment="calculate avg speed">
+    <list default="true" id="eb665c86-c893-4333-bd2a-721dc27980b9" name="Changes" comment="Refactor trip and speeding reports">
       <change beforePath="$PROJECT_DIR$/package-lock.json" beforeDir="false" afterPath="$PROJECT_DIR$/package-lock.json" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/package.json" beforeDir="false" afterPath="$PROJECT_DIR$/package.json" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/activity-report.js" beforeDir="false" afterPath="$PROJECT_DIR$/src/activity-report.js" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/src/index.test.js" beforeDir="false" afterPath="$PROJECT_DIR$/src/index.test.js" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/kms-report.js" beforeDir="false" afterPath="$PROJECT_DIR$/src/kms-report.js" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/trip-report.js" beforeDir="false" afterPath="$PROJECT_DIR$/src/trip-report.js" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/util/traccar.js" beforeDir="false" afterPath="$PROJECT_DIR$/src/util/traccar.js" afterDir="false" />
     </list>
     <option name="SHOW_DIALOG" value="false" />
     <option name="HIGHLIGHT_CONFLICTS" value="true" />
@@ -222,7 +217,11 @@
       <workItem from="1640866364417" duration="67000" />
       <workItem from="1640866782794" duration="123000" />
       <workItem from="1640866964393" duration="1022000" />
-      <workItem from="1640870147277" duration="2391000" />
+      <workItem from="1640870147277" duration="8236000" />
+      <workItem from="1641321068227" duration="14805000" />
+      <workItem from="1641387675790" duration="32947000" />
+      <workItem from="1641509412324" duration="41000" />
+      <workItem from="1641510744190" duration="1404000" />
     </task>
     <task id="LOCAL-00001" summary="Fix getStyle">
       <created>1636983717385</created>
@@ -357,7 +356,35 @@
       <option name="project" value="LOCAL" />
       <updated>1640867522438</updated>
     </task>
-    <option name="localTasksCounter" value="20" />
+    <task id="LOCAL-00020" summary="fix activity report by driver">
+      <created>1640872780966</created>
+      <option name="number" value="00020" />
+      <option name="presentableId" value="LOCAL-00020" />
+      <option name="project" value="LOCAL" />
+      <updated>1640872780968</updated>
+    </task>
+    <task id="LOCAL-00021" summary="activity report by driver grouped by day">
+      <created>1640878664617</created>
+      <option name="number" value="00021" />
+      <option name="presentableId" value="LOCAL-00021" />
+      <option name="project" value="LOCAL" />
+      <updated>1640878664617</updated>
+    </task>
+    <task id="LOCAL-00022" summary="fix kms-report by driver and fix activity report with time period">
+      <created>1641348028566</created>
+      <option name="number" value="00022" />
+      <option name="presentableId" value="LOCAL-00022" />
+      <option name="project" value="LOCAL" />
+      <updated>1641348028566</updated>
+    </task>
+    <task id="LOCAL-00023" summary="Refactor trip and speeding reports">
+      <created>1641511586537</created>
+      <option name="number" value="00023" />
+      <option name="presentableId" value="LOCAL-00023" />
+      <option name="project" value="LOCAL" />
+      <updated>1641511586537</updated>
+    </task>
+    <option name="localTasksCounter" value="24" />
     <servers />
   </component>
   <component name="TypeScriptGeneratedFilesManager">
@@ -394,6 +421,10 @@
     <MESSAGE value="location report - check if it's running on client side" />
     <MESSAGE value="timezone - check if it's running on client side" />
     <MESSAGE value="calculate avg speed" />
-    <option name="LAST_COMMIT_MESSAGE" value="calculate avg speed" />
+    <MESSAGE value="fix activity report by driver" />
+    <MESSAGE value="activity report by driver grouped by day" />
+    <MESSAGE value="fix kms-report by driver and fix activity report with time period" />
+    <MESSAGE value="Refactor trip and speeding reports" />
+    <option name="LAST_COMMIT_MESSAGE" value="Refactor trip and speeding reports" />
   </component>
 </project>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fleetmap-reports",
-  "version": "1.0.262",
+  "version": "1.0.266",
   "description": "",
   "main": "src/index.js",
   "scripts": {

package/src/activity-report.js

@@ -87,11 +87,14 @@ async function createActivityReportByDevice(from, to){
     //Get report data from traccar
     devicesToProcess.sort((a, b) => (a.name > b.name) ? 1 : -1)
 
+    // use the km report to calculate kms
+    const kmsReport = await createKmsReport(from, to, userData, traccarInstance)
+
     let data = []
     if(userData.groupByDay) {
         const dates = getDates(from, to)
         for(const date of dates){
-            if(userData.allWeek){
+            if(userData.allWeek || !userData.weekDays){
                 const fromByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 0, 0, 0)
                 const toByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 23, 59, 59)
 
@@ -110,9 +113,33 @@ async function createActivityReportByDevice(from, to){
                     const fromByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.startTime)
                     const toByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.endTime)
 
-                    const summary = await traccar.getSummary(traccarInstance, fromByDay, toByDay, devicesToProcess)
-                    summary.forEach(s => s.date = date)
-                    data = data.concat(summary)
+                    if(fromByDay.getTime() < toByDay.getTime()) {
+                        const summary = await traccar.getSummary(traccarInstance, new Date(fromByDay.toUTCString()), new Date(toByDay.toUTCString()), devicesToProcess)
+                        summary.forEach(s => s.date = date)
+                        data = data.concat(summary)
+                    } else {
+                        const dayStart = new Date(new Date(date).toISOString().split('T')[0] + ' 00:00:00')
+                        const dayEnd = new Date(new Date(date).toISOString().split('T')[0] + ' 23:59:59')
+
+                        const summaryStart = await traccar.getSummary(traccarInstance, new Date(dayStart.toUTCString()), new Date(toByDay.toUTCString()), devicesToProcess)
+                        const summaryEnd = await traccar.getSummary(traccarInstance, new Date(fromByDay.toUTCString()), new Date(dayEnd.toUTCString()), devicesToProcess)
+
+                        const summary = []
+                        summaryStart.forEach(s => {
+                            const sEnd = summaryEnd.find(a => s.deviceId === a.deviceId)
+                            summary.push({
+                                date: date,
+                                deviceId: s.deviceId,
+                                averageSpeed: (s.averageSpeed + sEnd.averageSpeed) / 2,
+                                distance: s.distance + sEnd.distance,
+                                engineHours: s.engineHours + sEnd.engineHours,
+                                maxSpeed: s.maxSpeed > sEnd.maxSpeed ? s.maxSpeed : sEnd.maxSpeed,
+                                endOdometer: 0,
+                                startOdometer: 0
+                            })
+                        })
+                        data = data.concat(summary)
+                    }
                 }else {
                     data = data.concat({date: date})
                 }
@@ -122,9 +149,6 @@ async function createActivityReportByDevice(from, to){
         data = await traccar.getSummary(traccarInstance, from, to, devicesToProcess)
     }
 
-    // use the km report to calculate kms
-    const kmsReport = await createKmsReport(from, to, userData, traccarInstance)
-
     console.log('Summary:' + data.length)
 
     //Process report data
@@ -136,11 +160,11 @@ async function createActivityReportByDevice(from, to){
 }
 
 async function createActivityReportByDriver(from, to){
-    const deviceIds = await drivers.devicesByDriver(traccarInstance, from, to, userData.drivers, userData.devices)
-    console.log(deviceIds)
+    const devices = await drivers.devicesByDriver(traccarInstance, from, to, userData.drivers, userData.devices)
+
     let tripsData = []
-    if(deviceIds.length > 0) {
-        tripsData = await traccar.getTrips(traccarInstance, from, to, deviceIds)
+    if(devices.length > 0) {
+        tripsData = await traccar.getTrips(traccarInstance, from, to, devices)
     }
 
     const allData = {
@@ -149,7 +173,7 @@ async function createActivityReportByDriver(from, to){
         to: to
     }
 
-    allData.drivers = processDrivers(userData.drivers, tripsData)
+    allData.drivers = processDrivers(from, to, userData.drivers, tripsData)
 
     return allData
 }
@@ -163,9 +187,6 @@ function processDevices(devices, data, kmsReport) {
         if (summary) {
             summary.forEach(s => {
                 const deviceKms = kmsReport[0].devices.filter(r => r.device.id === d.id)
-
-                console.log(deviceKms)
-
                 s.convertedSpentFuel = automaticReports.calculateSpentFuel(s.spentFuel, d)
                 s.distance = deviceKms.length ? (userData.groupByDay ? deviceKms[0].days.find(d => d.date.getTime() === s.date.getTime()).kms : deviceKms[0].summary.distance) : 0
             })
@@ -180,7 +201,7 @@ function processDevices(devices, data, kmsReport) {
     return devicesResult
 }
 
-function processDrivers(drivers, data) {
+function processDrivers(from, to, drivers, data) {
     console.log(data)
     const driversResult = []
     drivers.forEach(d => {
@@ -188,12 +209,63 @@ function processDrivers(drivers, data) {
         if (trips.length > 0) {
             const driverData = {
                 driver: d,
-                summary: [{
+                summary: [],
+            }
+            if(userData.groupByDay) {
+                const dates = getDates(from, to)
+                for(const date of dates){
+                    if(userData.allWeek || !userData.weekDays){
+                        const fromByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 0, 0, 0)
+                        const toByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 23, 59, 59)
+
+                        const tripsByDay = trips.filter(t => new Date(t.startTime) > fromByDay && new Date(t.endTime) < toByDay)
+
+                        driverData.summary.push({
+                            date: date,
+                            distance: tripsByDay.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0),
+                            engineHours: tripsByDay.reduce((a, b) => a + b.duration, 0),
+                            maxSpeed: tripsByDay.reduce((a, b) => Math.max(a, b.maxSpeed), 0),
+                            averageSpeed: tripsByDay.length > 0 ? Math.round(tripsByDay.reduce((a, b) => a + b.averageSpeed, 0) / tripsByDay.length) : 0,
+                        })
+                    } else {
+                        if((date.getDay() === 0 && userData.weekDays.sunday) ||
+                            (date.getDay() === 1 && userData.weekDays.monday) ||
+                            (date.getDay() === 2 && userData.weekDays.tuesday) ||
+                            (date.getDay() === 3 && userData.weekDays.wednesday)  ||
+                            (date.getDay() === 4 && userData.weekDays.thursday) ||
+                            (date.getDay() === 5 && userData.weekDays.friday) ||
+                            (date.getDay() === 6 && userData.weekDays.saturday)) {
+
+                            const fromByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.startTime)
+                            const toByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.endTime)
+
+                            const tripsByDay = trips.filter(t => new Date(t.startTime) > fromByDay && new Date(t.endTime) < toByDay)
+
+                            driverData.summary.push({
+                                date: date,
+                                distance: tripsByDay.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0),
+                                engineHours: tripsByDay.reduce((a, b) => a + b.duration, 0),
+                                maxSpeed: tripsByDay.reduce((a, b) => Math.max(a, b.maxSpeed), 0),
+                                averageSpeed: tripsByDay.length > 0 ? Math.round(tripsByDay.reduce((a, b) => a + b.averageSpeed, 0) / tripsByDay.length) : 0,
+                            })
+                        }else {
+                            driverData.summary.push({
+                                date: date,
+                                distance: 0,
+                                engineHours: 0,
+                                maxSpeed: 0,
+                                averageSpeed: 0,
+                            })
+                        }
+                    }
+                }
+            } else {
+                driverData.summary.push({
                     distance: trips.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0),
                     engineHours: trips.reduce((a, b) => a + b.duration, 0),
                     maxSpeed: trips.reduce((a, b) => Math.max(a, b.maxSpeed), 0),
                     averageSpeed: Math.round(trips.reduce((a, b) => a + b.averageSpeed, 0) / trips.length),
-                }],
+                })
             }
             driversResult.push(driverData)
         }
@@ -211,9 +283,11 @@ async function exportActivityReportToPDF(userData, reportData) {
         const doc = new jsPDF.jsPDF('l')
         await headerFromUser(doc, translations.report.titleActivityReport, userData.user)
 
-        if(userData.groupByDay){
+        if(userData.groupByDay && userData.byDriver) {
+            exportActivityReportToPDF_Driver_GroupByDay(doc, translations, reportData)
+        } else if(userData.groupByDay) {
             exportActivityReportToPDF_Vehicle_GroupByDay(doc, translations, reportData)
-        } else if(userData.byDriver){
+        } else if(userData.byDriver) {
             exportActivityReportToPDF_Driver(doc, translations, reportData)
         } else {
             exportActivityReportToPDF_Vehicle(doc, translations, reportData)
@@ -335,7 +409,7 @@ function exportActivityReportToPDF_Vehicle_GroupByDay(doc, translations, reportD
         doc.text(group ? translations.report.group + ': ' + group.name : '', 150, space + 20)
         doc.text(new Date(reportData.from).toLocaleString() + ' - ' + new Date(reportData.to).toLocaleString(), 20, space + 25)
 
-        if (!userData.allWeek) {
+        if (!userData.allWeek && userData.weekDays) {
             doc.text((userData.weekDays.monday ? ' ' + translations.report.monday + ',' : '')
                 + (userData.weekDays.tuesday ? ' ' + translations.report.tuesday + ',' : '')
                 + (userData.weekDays.wednesday ? ' ' + translations.report.wednesday + ',' : '')
@@ -430,6 +504,122 @@ function exportActivityReportToExcel_Vehicle_GroupByDay(settings, translations,
     }
 }
 
+function exportActivityReportToPDF_Driver_GroupByDay(doc, translations, reportData) {
+    const headers = [translations.report.date,
+        translations.report.distance,
+        translations.report.avgSpeed + ' (Km/h)',
+        translations.report.maxSpeed + ' (Km/h)',
+        translations.report.engineHours]
+
+    const weekDays = [
+        translations.report.sunday,
+        translations.report.monday,
+        translations.report.tuesday,
+        translations.report.wednesday,
+        translations.report.thursday,
+        translations.report.friday,
+        translations.report.saturday
+    ]
+
+    reportData.drivers.forEach(function (d, index) {
+        const name = d.driver.name
+        const group = userData.groups.find(g => g.drivers.includes(d.id))
+
+        let space = index === 0 ? 8 : 0
+        if(index){
+            doc.addPage()
+        }
+
+        doc.setFontSize(13)
+        doc.text(name, 20, space + 20)
+        doc.setFontSize(11)
+        doc.text(group ? translations.report.group + ': ' + group.name : '', 150, space + 20)
+        doc.text(new Date(reportData.from).toLocaleString() + ' - ' + new Date(reportData.to).toLocaleString(), 20, space + 25)
+
+        if (!userData.allWeek && userData.weekDays) {
+            doc.text((userData.weekDays.monday ? ' ' + translations.report.monday + ',' : '')
+                + (userData.weekDays.tuesday ? ' ' + translations.report.tuesday + ',' : '')
+                + (userData.weekDays.wednesday ? ' ' + translations.report.wednesday + ',' : '')
+                + (userData.weekDays.thursday ? ' ' + translations.report.thursday + ',' : '')
+                + (userData.weekDays.friday ? ' ' + translations.report.friday + ',' : '')
+                + (userData.weekDays.saturday ? ' ' + translations.report.saturday + ',' : '')
+                + (userData.weekDays.sunday ? ' ' + translations.report.sunday + ',' : '')
+                + ' das '
+                + userData.dayHours.startTime + ' - ' + userData.dayHours.endTime, 20, space + 30)
+        }
+
+        const data = []
+        d.summary.forEach(s => {
+            const temp = [
+                new Date(s.date).toLocaleDateString() + ' - ' + weekDays[s.date.getDay()],
+                Number((s.distance / 1000).toFixed(2)),
+                Math.round(s.averageSpeed * 1.85200),
+                Math.round(s.maxSpeed * 1.85200),
+                convertMS(s.engineHours)
+            ]
+
+            data.push(temp)
+        })
+
+        const footValues = ['Total:' + d.summary.length,
+            (d.summary.reduce((a, b) => a + b.distance, 0) / 1000).toFixed(0),
+            getSumAvgSpeed(d.summary),
+            getSumMaxSpeed(d.summary),
+            convertMS(d.summary.reduce((a, b) => a + b.engineHours, 0))]
+
+        const style = getStyle(getUserPartner(userData.user))
+        addTable(doc, headers, data, footValues, style, 40)
+    })
+}
+
+function exportActivityReportToExcel_Driver_GroupByDay(settings, translations, reportData) {
+    const headers = [{label: translations.report.driver, value: 'name'},
+        {label: translations.report.group, value: 'group'},
+        {label: translations.report.date, value: 'date'},
+        {label: translations.report.weekDay, value: 'weekday'},
+        {label: translations.report.distance, value: 'distance'},
+        {label: translations.report.speed + ' ' + translations.report.avgSpeed + ' (Km/h)', value: 'avgSpeed'},
+        {label: translations.report.speed + ' ' + translations.report.maxSpeed + ' (Km/h)', value: 'maxSpeed'},
+        {label: translations.report.engineHours, value: 'engineHours'}]
+
+    const weekDays = [
+        translations.report.sunday,
+        translations.report.monday,
+        translations.report.tuesday,
+        translations.report.wednesday,
+        translations.report.thursday,
+        translations.report.friday,
+        translations.report.saturday
+    ]
+
+    let data = []
+    if (reportData.drivers) {
+        reportData.drivers.forEach(d => {
+            const group = userData.groups.find(g => g.drivers.includes(d.id))
+
+            data = data.concat([{}])
+            data = data.concat(d.summary.map(s => {
+                return {
+                    name: d.driver.name,
+                    group: group ? group.name : '',
+                    date: new Date(s.date).toLocaleDateString(),
+                    weekday: weekDays[s.date.getDay()],
+                    distance: Number((s.distance / 1000).toFixed(0)),
+                    avgSpeed: Math.round(s.averageSpeed * 1.85200),
+                    maxSpeed: Math.round(s.maxSpeed * 1.85200),
+                    engineHours: convertMS(s.engineHours)
+                }
+            }))
+        })
+        console.log(data)
+        return {
+            headers,
+            data,
+            settings
+        }
+    }
+}
+
 function exportActivityReportToExcel_Driver(settings, translations, reportData) {
     const headers = [{label: translations.report.driver, value: 'driver'},
         {label: translations.report.distance, value: 'distance'},
@@ -505,7 +695,9 @@ function exportActivityReportToExcel(reportUserData, reportData) {
         fileName: fileName // The name of the spreadsheet
     }
 
-    if(userData.groupByDay){
+    if(userData.groupByDay && userData.byDriver){
+        return exportActivityReportToExcel_Driver_GroupByDay(settings, translations, reportData)
+    } else if(userData.groupByDay){
         return exportActivityReportToExcel_Vehicle_GroupByDay(settings, translations, reportData)
     } else if(userData.byDriver){
         return exportActivityReportToExcel_Driver(settings, translations, reportData)

package/src/kms-report.js

@@ -7,6 +7,7 @@ const {getUserPartner} = require("fleetmap-partners");
 const traccar = require("./util/traccar");
 const {getDates} = require("./util/utils");
 const trips = require("./util/trips");
+const drivers = require("./util/driver");
 
 
 let traccarInstance
@@ -26,7 +27,7 @@ async function createKmsReportByDevice(from, to) {
 
     devicesToProcess.sort((a, b) => (a.name > b.name) ? 1 : -1)
     const route = userData.allWeek ? [] : await traccar.getRoute(traccarInstance, from, to, devicesToProcess)
-    const tripsData = await traccar.getTrips(traccarInstance, from, to, devicesToProcess.map(d => d.id))
+    const tripsData = await traccar.getTrips(traccarInstance, from, to, devicesToProcess)
 
     console.log('trips:' + tripsData.length)
 
@@ -39,13 +40,84 @@ async function createKmsReportByDevice(from, to) {
 }
 
 async function createKmsReportByDriver(from, to) {
+    const devices = await drivers.devicesByDriver(traccarInstance, from, to, userData.drivers, userData.devices)
+    console.log(devices.length)
 
+    if(!devices.length) {
+        //empty report
+        return { drivers: [] }
+    }
+
+    const tripsData = await traccar.getTrips(traccarInstance, from, to, devices)
+    return { drivers: processDrivers(from, to, userData.drivers, tripsData) }
 }
 
 async function createKmsReportByGroup(from, to) {
 
 }
 
+function processDrivers(from, to, drivers, data) {
+    console.log(data)
+    const driversResult = []
+    drivers.forEach(d => {
+        const trips = data.filter(t => t.driverUniqueId === d.uniqueId)
+        if (trips.length > 0) {
+            const driverData = {
+                driver: d
+            }
+            if(userData.groupByDay) {
+                driverData.days = []
+                const dates = getDates(from, to)
+                for(const date of dates){
+                    if(userData.allWeek || !userData.weekDays){
+                        const fromByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 0, 0, 0)
+                        const toByDay = new Date(date.getFullYear(), date.getMonth(), date.getDate(), 23, 59, 59)
+
+                        const tripsByDay = trips.filter(t => new Date(t.startTime) > fromByDay && new Date(t.endTime) < toByDay)
+
+                        driverData.days.push({
+                            date: date,
+                            kms: tripsByDay.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0)
+                        })
+                    } else {
+                        if((date.getDay() === 0 && userData.weekDays.sunday) ||
+                            (date.getDay() === 1 && userData.weekDays.monday) ||
+                            (date.getDay() === 2 && userData.weekDays.tuesday) ||
+                            (date.getDay() === 3 && userData.weekDays.wednesday)  ||
+                            (date.getDay() === 4 && userData.weekDays.thursday) ||
+                            (date.getDay() === 5 && userData.weekDays.friday) ||
+                            (date.getDay() === 6 && userData.weekDays.saturday)) {
+
+                            const fromByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.startTime)
+                            const toByDay = new Date(new Date(date).toISOString().split('T')[0] + ' ' + userData.dayHours.endTime)
+
+                            const tripsByDay = trips.filter(t => new Date(t.startTime) > fromByDay && new Date(t.endTime) < toByDay)
+
+                            driverData.days.push({
+                                date: date,
+                                kms: tripsByDay.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0)
+                            })
+                        }else {
+                            driverData.days.push({
+                                date: date,
+                                kms: 0
+                            })
+                        }
+                    }
+                }
+            } else {
+                driverData.summary = {
+                    distance: trips.filter(t => t.distance > 0).reduce((a, b) => a + b.distance, 0)
+                }
+            }
+            driversResult.push(driverData)
+        }
+    })
+
+    return driversResult
+}
+
+
 function processDevices(from, to, devices, route, trips) {
     const devicesResult = []
 
@@ -81,7 +153,7 @@ function processDevices(from, to, devices, route, trips) {
                         kms: 0
                     }
 
-                    if(userData.allWeek){
+                    if(userData.allWeek || !userData.weekDays){
                         day.kms = dayTrips.reduce((a, b) => a + b.distance, 0) || 0
                     } else {
                         if((currentDate.getDay() === 0 && userData.weekDays.sunday) ||
@@ -230,7 +302,7 @@ async function exportKmsReportToPDF(userData, reportData) {
                 doc.text(group ? translations.report.group + ': ' + group.name : '', 150, space + 20)
                 doc.text(new Date(reportData.from).toLocaleString() + ' - ' + new Date(reportData.to).toLocaleString(), 20, space + 25)
 
-                if(!userData.allWeek) {
+                if(!userData.allWeek && userData.weekDays) {
                     doc.text((userData.weekDays.monday ? ' ' + translations.report.monday + ',' : '')
                         + (userData.weekDays.tuesday ? ' ' + translations.report.tuesday + ',' : '')
                         + (userData.weekDays.wednesday ? ' ' + translations.report.wednesday + ',' : '')