Skip to content

Commit

Permalink
Merge pull request #91 from NETHINKS/bugfix/NET-641_no-admin-initialized
Browse files Browse the repository at this point in the history
[NET-641] Fix user management insert convertion
  • Loading branch information
johndgrey committed Oct 22, 2020
2 parents 8be2070 + 38ba156 commit 07c167c
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 8 deletions.
7 changes: 5 additions & 2 deletions cmdb/__setup__.py
Expand Up @@ -16,6 +16,8 @@

import logging
from enum import Enum

from cmdb.user_management import UserGroupModel
from cmdb.utils.system_config import SystemConfigReader
from cmdb.data_storage.database_manager import DatabaseManagerMongo

Expand Down Expand Up @@ -158,9 +160,10 @@ def __create_user_management(self):
admin_user = UserModel(
public_id=1,
user_name=admin_name,
password=scm.generate_hmac(admin_pass),
active=True,
group_id=__FIXED_GROUPS__[0].get_public_id(),
registration_time=datetime.datetime.utcnow()
registration_time=datetime.datetime.utcnow(),
password=scm.generate_hmac(admin_pass),
)
user_manager.insert(admin_user)
return True
Expand Down
7 changes: 3 additions & 4 deletions cmdb/user_management/models/group.py
Expand Up @@ -78,16 +78,15 @@ def from_data(cls, data: dict, rights: List[BaseRight] = None) -> "UserGroupMode

@classmethod
def to_data(cls, instance: "UserGroupModel"):
return dumps({
return {
'public_id': instance.public_id,
'name': instance.name,
'label': instance.label,
'rights': [BaseRight.to_dict(right) for right in instance.rights]
}, default=default)
'rights': [right.name for right in instance.rights]
}

@classmethod
def to_dict(cls, instance: "UserGroupModel") -> dict:

return {
'public_id': instance.public_id,
'name': instance.name,
Expand Down
4 changes: 2 additions & 2 deletions cmdb/user_management/models/user.py
Expand Up @@ -138,8 +138,8 @@ def from_data(cls, data: dict) -> "UserModel":
)

@classmethod
def to_data(cls, instance: "UserModel") -> str:
return dumps(cls.to_dict(instance), default=default)
def to_data(cls, instance: "UserModel") -> dict:
return cls.to_dict(instance)

@classmethod
def to_dict(cls, instance: "UserModel") -> dict:
Expand Down

0 comments on commit 07c167c

Please sign in to comment.