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`."""