fleetmap-reports 2.0.353 → 2.0.356

package/package.json

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

package/src/fuel-consumption-report.js

@@ -29,7 +29,7 @@ async function createFuelConsumptionReport (from, to, userData, traccar) {
 
   if (isClientSide() && (devices.length > 50 || ((new Date(to).getTime() - new Date(from).getTime()) > (1000 * 60 * 60 * 24 * 15)))) {
     const sliced = automaticReports.sliceArray(devices, 1)
-    allData.devices = await executeServerSide('fuel-consumption-report', sliced, from, to, userData, devices.length, traccar)
+    allData.devices = await executeServerSide('fuel-consumption-report', sliced, from, to, userData, devices.length, traccar, 5)
     allData.totalDevices = allData.devices.length
     reportData.push(allData)
     return reportData
@@ -153,7 +153,7 @@ function exportFuelConsumptionReportToExcel (userData, reportData) {
       data = data.concat(d.days.map((r) => {
         return {
           name: d.device.name,
-          date: r.date.toLocaleDateString(),
+          date: new Date(r.date).toLocaleDateString(),
           kms: formatNumber(r.distance / 1000),
           consumption: r.spentFuel > 0 ? formatNumber(r.spentFuel) : 0,
           avg_consumption: formatNumber(lang === 'pt-BR' ? r.avgConsumption.byLiters : r.avgConsumption.byKms),
@@ -220,7 +220,7 @@ async function exportFuelConsumptionReportToPDF (userData, reportData) {
 
       d.days.forEach(r =>
         data.push([
-          r.date.toLocaleDateString(),
+          new Date(r.date).toLocaleDateString(),
           formatNumber(r.distance / 1000),
           r.spentFuel > 0 ? formatNumber(r.spentFuel) : 0,
           formatNumber(lang === 'pt-BR' ? r.avgConsumption.byLiters : r.avgConsumption.byKms),

package/src/partnerReports/driverranking-report.js

@@ -1,5 +1,5 @@
 const traccarHelper = require('../util/traccar')
-const { getDriverData } = require('../util/driver')
+const { getDriverData, devicesByDriver } = require('../util/driver')
 const { getCanDriverStyleMessages, parseCanDriverStyleMessage } = require('../util/xpert')
 const { calculateRPMSections } = require('./performance-report')
 const { getTranslations, isClientSide, convertToLocaleString, convertMS } = require('../util/utils')
@@ -29,10 +29,12 @@ async function create (from, to, userData, traccar) {
     drivers: []
   }
 
-  const devices = userData.devices
   userData.geofences = []
   userData.roadSpeedLimits = true
 
+  const { devices, drivers } = await devicesByDriver(traccar, from, to, userData.drivers, userData.devices)
+  userData.drivers = drivers
+
   let deviceCount = 0
   const driversData = new Map()
   const overspeedEvents = await getOverspeedEvents(from, to, userData, devices, userData, traccar)

package/src/util/utils.js

@@ -115,6 +115,7 @@ function getDates (startDate, endDate, timezone) {
   let currentDate = new Date(startDate.getFullYear(), startDate.getMonth(), startDate.getDate(), 12, 0, 0, 0)
   let _endDate = new Date(endDate.getFullYear(), endDate.getMonth(), endDate.getDate(), 23, 59, 59, 0)
 
+  timezone = safeTimezone(timezone)
   if (!isClientSide() && timezone) {
     const startDateUserTimezone = convertFromUTC(startDate, timezone)
     currentDate = new Date(startDateUserTimezone.getFullYear(), startDateUserTimezone.getMonth(), startDateUserTimezone.getDate(), 12, 0, 0, 0)
@@ -160,13 +161,25 @@ async function getImgFromUrl (hostname) {
 
 function isClientSide () { return (typeof window !== 'undefined') }
 
+// Guards against corrupt user attributes (e.g. timezone === true) reaching Intl,
+// which throws RangeError: Invalid time zone specified
+function safeTimezone (timezone) {
+  if (typeof timezone !== 'string' || !timezone) return undefined
+  try {
+    Intl.DateTimeFormat(undefined, { timeZone: timezone })
+    return timezone
+  } catch (e) {
+    return undefined
+  }
+}
+
 function convertToLocaleString (value, lang, timezone, user) {
   if (isClientSide()) {
     return new Date(value).toLocaleString()
   }
   if ((lang && timezone) || user) {
     return new Date(value).toLocaleString(lang || getUserPartner(user).lang, {
-      timeZone: timezone || getUserPartner(user).timezone,
+      timeZone: safeTimezone(timezone) || safeTimezone(user && getUserPartner(user).timezone),
       hour12: false
     })
   }
@@ -178,7 +191,7 @@ function convertToLocaleDateString (value, lang, timezone, user) {
     return new Date(value).toLocaleDateString()
   } else {
     return new Date(value).toLocaleDateString(lang || getUserPartner(user).lang, {
-      timeZone: timezone || getUserPartner(user).timezone,
+      timeZone: safeTimezone(timezone) || safeTimezone(user && getUserPartner(user).timezone),
       hour12: false
     })
   }
@@ -189,7 +202,7 @@ function convertToLocaleTimeString (value, lang, timezone, user) {
     return new Date(value).toLocaleTimeString()
   } else {
     return new Date(value).toLocaleTimeString(lang || getUserPartner(user).lang, {
-      timeZone: timezone || getUserPartner(user).timezone,
+      timeZone: safeTimezone(timezone) || safeTimezone(user && getUserPartner(user).timezone),
       hour12: false
     })
   }
@@ -199,6 +212,10 @@ function getTimezoneOffset (timezone, date = new Date()) {
   if (isClientSide()) {
     return date.getTimezoneOffset()
   } else {
+    timezone = safeTimezone(timezone)
+    if (!timezone) {
+      return 0
+    }
     const utcDate = new Date(date.toLocaleString('en-US', { timeZone: 'UTC' }))
     const tzDate = new Date(date.toLocaleString('en-US', { timeZone: timezone }))
     return (utcDate.getTime() - tzDate.getTime()) / 6e4
@@ -259,7 +276,10 @@ async function executeServerSide (type, sliced, from, to, userData, totalItems,
               },
           cookie
         }, { withCredentials: true }).then(d => {
-          reportData.push(...(userData.byDriver ? d.data[0].drivers : d.data[0].devices))
+          const result = d.data[0]
+          if (result) {
+            reportData.push(...((userData.byDriver ? result.drivers : result.devices) || []))
+          }
         })
         itemsCount = itemsCount + items.length
       } catch (e) {