Browse Source

fix some bug

storage-in-realtime
zhaohe 12 months ago
parent
commit
d3a2e52858
  1. 10
      appsrc/baseservice/db/db_service.cpp
  2. 2
      appsrc/service/user_mgr_service.cpp

10
appsrc/baseservice/db/db_service.cpp

@ -117,15 +117,7 @@ shared_ptr<db::User> DBService::delUser(int id) {
auto usertable = make_storage(USER_DB_STRUCT); auto usertable = make_storage(USER_DB_STRUCT);
usertable.sync_schema(); usertable.sync_schema();
/**
* @brief find admin user
*/
auto admin = usertable.get_all<User>(where(c(&User::uid) == "admin"));
ZCHECK(admin.size() == 1, "admin user not found");
if (admin[0].id == id) {
logger->error("can not delete admin user");
return nullptr;
}
auto remove_user = usertable.get_all<User>(where(c(&User::id) == id)); auto remove_user = usertable.get_all<User>(where(c(&User::id) == id));
if (remove_user.size() == 0) { if (remove_user.size() == 0) {
logger->error("remove user fail, user not found"); logger->error("remove user fail, user not found");

2
appsrc/service/user_mgr_service.cpp

@ -60,7 +60,7 @@ void UserMgrService::delUser(shared_ptr<MsgProcessContext> cxt, int id) {
void UserMgrService::updateUserUid(shared_ptr<MsgProcessContext> cxt, int id, string uid) { void UserMgrService::updateUserUid(shared_ptr<MsgProcessContext> cxt, int id, string uid) {
string olduid; string olduid;
auto user = m_db->updateUserUid(id, uid, olduid); auto user = m_db->updateUserUid(id, uid, olduid);
APPCHECK(!user, err::kappe_user_not_exist, fmt::format("user {} not exist", id));
APPCHECK(user, err::kappe_user_not_exist, fmt::format("user {} not exist", id));
} }
void UserMgrService::getAllUser(shared_ptr<MsgProcessContext> cxt) { void UserMgrService::getAllUser(shared_ptr<MsgProcessContext> cxt) {
auto users = m_db->getAllUserJson(); auto users = m_db->getAllUserJson();

Loading…
Cancel
Save