Merge pull request #3 from DeanMauro/fix/groupBy_for_multi_lvl_relations

Update FindQueryBuilder.js
DeanMauro · Jul 29, 2022 · e25369d · e25369d
2 parents cabfaf5 + f2afc83
commit e25369d
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/FindQueryBuilder.js b/lib/FindQueryBuilder.js
@@ -437,9 +437,13 @@ class FindQueryBuilder {
    */
   _buildGroupBy(params, builder) {
     const groupByParam = _.find(params, { key: 'groupBy' });
+    const joinParam = _.find(builder._operations, { name: 'innerJoin' });
+    const idColumn = this._modelClass.idColumn;
+
     if (groupByParam) {
-      builder.select(groupByParam.value.split(','));
       builder.groupBy(groupByParam.value.split(','));
+    } else if (joinParam) {
+      builder.groupBy(Array.isArray(idColumn) ? idColumn : idColumn.split(','));
     }
   }