fix(manager): limit the count to existing users

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