From c009a6f7d490a145630d5e3dcd65528bacc869de Mon Sep 17 00:00:00 2001
From: mrfry <mr.fry@tutanota.com>
Date: Wed, 12 Apr 2023 15:11:27 +0200
Subject: [PATCH] ranklist using files.ts, minor loggin improvements

---
 src/modules/api/submodules/ranklist.ts | 8 +++-----
 src/utils/files.ts                     | 6 ++++--
 submodules/qmining-page                | 2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/modules/api/submodules/ranklist.ts b/src/modules/api/submodules/ranklist.ts
index 54ab997..f785a5f 100644
--- a/src/modules/api/submodules/ranklist.ts
+++ b/src/modules/api/submodules/ranklist.ts
@@ -21,6 +21,7 @@
 import logger from '../../../utils/logger'
 import utils from '../../../utils/utils'
 import { Request, SubmoduleData, User } from '../../../types/basicTypes'
+import { paths } from '../../../utils/files'
 
 interface Subjects {
     [key: string]: number
@@ -41,9 +42,6 @@ interface IdStatWithUID extends IdStat {
     userId: number
 }
 
-const idStatFile = 'stats/idstats'
-const idvStatFile = 'stats/idvstats'
-
 function mergeObjSum(a: Subjects, b: Subjects) {
     const res = { ...b }
     Object.keys(a).forEach((key) => {
@@ -67,14 +65,14 @@ function setup(data: SubmoduleData): void {
         const user: User = req.session.user
 
         if (!querySince) {
-            result = utils.ReadJSON(idStatFile)
+            result = utils.ReadJSON(paths.idstatsFile)
         } else {
             try {
                 const since = new Date(querySince)
                 if (!(since instanceof Date) || isNaN(since.getTime())) {
                     throw new Error('Not a date')
                 }
-                const data = utils.ReadJSON(idvStatFile)
+                const data = utils.ReadJSON(paths.idvstatsFile)
                 result = {}
 
                 Object.keys(data).forEach((key) => {
diff --git a/src/utils/files.ts b/src/utils/files.ts
index 35405d7..ca6375e 100644
--- a/src/utils/files.ts
+++ b/src/utils/files.ts
@@ -44,7 +44,9 @@ export const validateAndSetupFiles = (): boolean => {
         }
 
         if (file.warningIfMissing && !fileExists) {
-            const warningMsg = [`File "${file.path}" does not exist! (${key})`]
+            const warningMsg = [
+                `Optional file "${file.path}" does not exist! (${key})`,
+            ]
             if (file.description) {
                 warningMsg.push(`Should be: ${file.description}`)
             }
@@ -81,7 +83,7 @@ const filesFiles = {
     httpsFiles: {
         path: 'data/httpsfiles.json',
         description:
-            'file paths for https functionality (privkey, chain files). Optional if https server is not used',
+            'file paths for https functionality (privkey, chain files)',
         schema: HttpsFilesSchema,
         warningIfMissing: true,
     },
diff --git a/submodules/qmining-page b/submodules/qmining-page
index 8267534..338db8e 160000
--- a/submodules/qmining-page
+++ b/submodules/qmining-page
@@ -1 +1 @@
-Subproject commit 8267534a3f4279169c9ed11d0bef019b12cac2a2
+Subproject commit 338db8e1417ba4aa647a74e0a5bad70fb04bc79e