fix(manager): limit the count to existing users

lemredd · Dec 6, 2022 · b5914e7 · b5914e7
1 parent 022b40d
commit b5914e7
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/database/managers/department.ts b/database/managers/department.ts
@@ -48,9 +48,12 @@ export default class extends BaseManager<
 					User.sequelize.getQueryInterface().queryGenerator.selectQuery(
 						User.tableName, {
 							"attributes": [ User.sequelize.fn("count", "*") ],
-							"where": new Condition().equal(
-								"departmentID",
-								User.sequelize.col(`${Model.tableName}.id`)
+							"where": new Condition().and(
+								new Condition().equal(
+									"departmentID",
+									User.sequelize.col(`${Model.tableName}.id`)
+								),
+								new Condition().is("deletedAt", null)
 							)
 							.build()
 						}