diff options
Diffstat (limited to 'src/mailman/rest/users.py')
| -rw-r--r-- | src/mailman/rest/users.py | 24 |
1 files changed, 19 insertions, 5 deletions
diff --git a/src/mailman/rest/users.py b/src/mailman/rest/users.py index b1aca9678..49d90338f 100644 --- a/src/mailman/rest/users.py +++ b/src/mailman/rest/users.py @@ -67,8 +67,8 @@ CREATION_FIELDS = dict( email=str, display_name=str, password=str, - is_serverowner=bool, - _optional=('display_name', 'password', 'is_serverowner'), + is_server_owner=bool, + _optional=('display_name', 'password', 'is_server_owner'), ) @@ -109,7 +109,7 @@ class _UserBase(CollectionMixin): user_id=user_id, created_on=user.created_on, self_link=path_to('users/{}'.format(user_id)), - is_serverowner=user.is_serverowner, + is_server_owner=user.is_server_owner, ) # Add the password attribute, only if the user has a password. Same # with the real name. These could be None or the empty string. @@ -295,7 +295,8 @@ class AddressUser(_UserBase): del fields['email'] fields['user_id'] = int fields['auto_create'] = as_boolean - fields['_optional'] = fields['_optional'] + ('user_id', 'auto_create') + fields['_optional'] = fields['_optional'] + ('user_id', 'auto_create', + 'is_server_owner') try: validator = Validator(**fields) arguments = validator(request) @@ -330,7 +331,8 @@ class AddressUser(_UserBase): # Process post data and check for an existing user. fields = CREATION_FIELDS.copy() fields['user_id'] = int - fields['_optional'] = fields['_optional'] + ('user_id', 'email') + fields['_optional'] = fields['_optional'] + ('user_id', 'email', + 'is_server_owner') try: validator = Validator(**fields) arguments = validator(request) @@ -403,6 +405,18 @@ class OwnersForDomain(_UserBase): self._domain.add_owner(owner) return no_content(response) + def on_patch(self, request, response): + # TODO: complete this + pass + + def on_put(self, request, response): + # TODO: complete this + pass + + def on_delete(self, request, response): + # TODO: complete this + pass + @paginate def _get_collection(self, request): """See `CollectionMixin`.""" |