From 27e74873ef19bcb94c172062531606d9bf4eddae Mon Sep 17 00:00:00 2001 From: Kenneth Trecy Tobias <19201.tobias.kennethtrecy.c@gmail.com> Date: Tue, 6 Dec 2022 14:45:55 +0800 Subject: [PATCH] fix(manager): limit the count to existing users --- database/managers/role.ts | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/database/managers/role.ts b/database/managers/role.ts index 5d693defe..70ec39796 100644 --- 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() }