@things-factory/dataset 6.0.90 → 6.0.92

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/dataset",
-  "version": "6.0.90",
+  "version": "6.0.92",
   "main": "dist-server/index.js",
   "browser": "dist-client/index.js",
   "things-factory": true,
@@ -36,19 +36,19 @@
     "@operato/shell": "^1.0.1",
     "@operato/styles": "^1.0.0",
     "@operato/utils": "^1.0.1",
-    "@things-factory/auth-base": "^6.0.90",
-    "@things-factory/aws-base": "^6.0.90",
-    "@things-factory/board-service": "^6.0.90",
-    "@things-factory/env": "^6.0.90",
-    "@things-factory/organization": "^6.0.90",
-    "@things-factory/scheduler-client": "^6.0.90",
-    "@things-factory/shell": "^6.0.90",
-    "@things-factory/work-shift": "^6.0.90",
-    "@things-factory/worklist": "^6.0.90",
+    "@things-factory/auth-base": "^6.0.91",
+    "@things-factory/aws-base": "^6.0.91",
+    "@things-factory/board-service": "^6.0.91",
+    "@things-factory/env": "^6.0.91",
+    "@things-factory/organization": "^6.0.91",
+    "@things-factory/scheduler-client": "^6.0.91",
+    "@things-factory/shell": "^6.0.91",
+    "@things-factory/work-shift": "^6.0.91",
+    "@things-factory/worklist": "^6.0.91",
     "cron-parser": "^4.3.0",
     "moment-timezone": "^0.5.40",
     "simple-statistics": "^7.8.3",
     "statistics": "^3.3.0"
   },
-  "gitHead": "f414cc54d34c91def66b229a9c417b663399c031"
+  "gitHead": "6b20d32f29809cd6c99fd22296f809816a6c954d"
 }

package/server/controllers/query-data-summary-by-period.ts

@@ -5,7 +5,6 @@ import { getQueryBuilderFromListParams, getRepository, ListParam } from '@things
 import { DataSet, DataSetSummaryPeriodType } from '../service/data-set/data-set'
 
 import { DataSummary } from '../service/data-summary/data-summary'
-import { DataSummaryList } from '../service/data-summary/data-summary-type'
 
 const STAT_FUNCTION_MAP = {
   sum: 'sum',
@@ -57,6 +56,7 @@ async function getTimesForPeriod(
 export async function queryDataSummaryByPeriod(
   period: 'today' | 'this month' | '30 days' | 'this year' | '12 months',
   dataSetName: string,
+  dataKeys: string[] | null,
   params: ListParam,
   context: ResolverContext
 ): Promise<DataSummary[]> {
@@ -80,7 +80,19 @@ export async function queryDataSummaryByPeriod(
     throw new Error(`given dataSet(${dataSetName}) is not allowed for getting summary for ${period}`)
   }
 
-  const dataKeyItems = dataSet.dataKeySet?.dataKeyItems || []
+  // dataKeys 가 설정되지 않았다면, dataSet의 dataKeySet을 그대로 적용한다는 의미임.
+  // dataKeys == [] 라면, dataKeySet을 적용하지 않는다는 의미임.
+  // dataKeys에는 dataKeySet의 dataKey 값을 따라야 한다.
+  const dataKeyItems = !dataSet.dataKeySet?.dataKeyItems
+    ? []
+    : dataKeys
+    ? dataKeys
+        .map(dataKey => {
+          return dataSet.dataKeySet.dataKeyItems.find(item => item.dataKey == dataKey)
+        })
+        .filter(Boolean)
+    : dataSet.dataKeySet.dataKeyItems
+
   const searchables = dataKeyItems.map((item, index) => `key0${index + 1}`)
   const dataItems = dataSet.dataItems.filter(item => item.stat)
 
@@ -138,7 +150,7 @@ export async function queryDataSummaryByPeriod(
     period == 'today'
       ? 'summary.date, summary.period'
       : ['this year', '12 months'].includes(period)
-      ? 'month, summary.date'
+      ? 'month'
       : 'summary.date'
   const groupByKeys = dataKeyItems
     .map((item, index) => {
@@ -162,7 +174,7 @@ export async function queryDataSummaryByPeriod(
       ? [{ sort: 'month', order: 'DESC' }]
       : [{ sort: 'summary.date', order: 'DESC' }]
 
-  const queryBuilder = getQueryBuilderFromListParams({
+  var queryBuilder = getQueryBuilderFromListParams({
     repository: getRepository(DataSummary),
     params,
     domain,
@@ -172,13 +184,12 @@ export async function queryDataSummaryByPeriod(
     .select('summary.dataSet')
     .addSelect('ds.name', 'name')
     .addSelect('ds.description', 'description')
-    .addSelect(selectKeys)
     .addSelect(selectData)
     .addSelect('SUM(summary.count)', 'count')
     .addSelect('SUM(summary.countOoc)', 'countOoc')
     .addSelect('SUM(summary.countOos)', 'countOos')
 
-  selectPeriod(queryBuilder)
+  queryBuilder = selectPeriod(queryBuilder)
     .innerJoin('summary.dataSet', 'ds', 'ds.id = :dataSetName', {
       dataSetName: dataSet.id
     })
@@ -189,7 +200,10 @@ export async function queryDataSummaryByPeriod(
     .addGroupBy('ds.name')
     .addGroupBy('ds.description')
     .addGroupBy(groupByPeriod)
-    .addGroupBy(groupByKeys)
+
+  if (dataKeyItems.length > 0) {
+    queryBuilder.addSelect(selectKeys).addGroupBy(groupByKeys)
+  }
 
   orderByPeriod.map(orderBy => {
     const { sort, order } = orderBy

package/server/service/data-summary/data-summary-query.ts

@@ -39,10 +39,11 @@ export class DataSummaryQuery {
   async dataSummaryByPeriod(
     @Arg('period') period: 'today' | 'this month' | '30 days' | 'this year' | '12 months',
     @Arg('dataSetName') dataSetName: string,
+    @Arg('dataKeys', type => [String], { nullable: true }) dataKeys: string[] | null,
     @Args() params: ListParam,
     @Ctx() context: ResolverContext
   ): Promise<DynamicDataSummary[]> {
-    return await queryDataSummaryByPeriod(period, dataSetName, params, context)
+    return await queryDataSummaryByPeriod(period, dataSetName, dataKeys, params, context)
   }
 
   @Directive('@privilege(category: "data-summary", privilege: "query", domainOwnerGranted: true)')