Revert "Add group roles to @groups serializer"

This reverts commit a430318.
plone · Mar 4, 2019 · d59fb19 · d59fb19
1 parent a430318
commit d59fb19
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 34 deletions.
diff --git a/news/682.feature b/news/682.feature
diff --git a/src/plone/restapi/serializer/group.py b/src/plone/restapi/serializer/group.py
@@ -28,7 +28,6 @@ def __call__(self):
             'email': group.getProperty('email'),
             'title': group.getProperty('title'),
             'description': group.getProperty('description'),
-            'roles': group.getRoles(),
         }
 
 

diff --git a/src/plone/restapi/tests/http-examples/groups.resp b/src/plone/restapi/tests/http-examples/groups.resp
@@ -8,10 +8,6 @@ Content-Type: application/json
     "email": "", 
     "groupname": "Administrators", 
     "id": "Administrators", 
-    "roles": [
-      "Manager", 
-      "Authenticated"
-    ], 
     "title": "Administrators"
   }, 
   {
@@ -20,10 +16,6 @@ Content-Type: application/json
     "email": "", 
     "groupname": "Reviewers", 
     "id": "Reviewers", 
-    "roles": [
-      "Reviewer", 
-      "Authenticated"
-    ], 
     "title": "Reviewers"
   }, 
   {
@@ -32,10 +24,6 @@ Content-Type: application/json
     "email": "", 
     "groupname": "Site Administrators", 
     "id": "Site Administrators", 
-    "roles": [
-      "Site Administrator", 
-      "Authenticated"
-    ], 
     "title": "Site Administrators"
   }, 
   {
@@ -44,9 +32,6 @@ Content-Type: application/json
     "email": "ploneteam@plone.org", 
     "groupname": "ploneteam", 
     "id": "ploneteam", 
-    "roles": [
-      "Authenticated"
-    ], 
     "title": "Plone Team"
   }, 
   {
@@ -55,7 +40,6 @@ Content-Type: application/json
     "email": "", 
     "groupname": "AuthenticatedUsers", 
     "id": "AuthenticatedUsers", 
-    "roles": [], 
     "title": "Authenticated Users (Virtual Group)"
   }
 ]
diff --git a/src/plone/restapi/tests/http-examples/groups_created.resp b/src/plone/restapi/tests/http-examples/groups_created.resp
@@ -8,10 +8,6 @@ Location: http://localhost:55001/plone/@groups/fwt
   "email": "fwt@plone.org", 
   "groupname": "fwt", 
   "id": "fwt", 
-  "roles": [
-    "Manager", 
-    "Authenticated"
-  ], 
   "title": "Framework Team", 
   "users": {
     "@id": "http://localhost:55001/plone/@groups", 

diff --git a/src/plone/restapi/tests/http-examples/groups_filtered_by_groupname.resp b/src/plone/restapi/tests/http-examples/groups_filtered_by_groupname.resp
@@ -8,9 +8,6 @@ Content-Type: application/json
     "email": "ploneteam@plone.org", 
     "groupname": "ploneteam", 
     "id": "ploneteam", 
-    "roles": [
-      "Authenticated"
-    ], 
     "title": "Plone Team"
   }
 ]
diff --git a/src/plone/restapi/tests/http-examples/groups_get.resp b/src/plone/restapi/tests/http-examples/groups_get.resp
@@ -7,9 +7,6 @@ Content-Type: application/json
   "email": "ploneteam@plone.org", 
   "groupname": "ploneteam", 
   "id": "ploneteam", 
-  "roles": [
-    "Authenticated"
-  ], 
   "title": "Plone Team", 
   "users": {
     "@id": "http://localhost:55001/plone/@groups/ploneteam", 

diff --git a/src/plone/restapi/tests/http-examples/principals.resp b/src/plone/restapi/tests/http-examples/principals.resp
@@ -9,9 +9,6 @@ Content-Type: application/json
       "email": "ploneteam@plone.org", 
       "groupname": "ploneteam", 
       "id": "ploneteam", 
-      "roles": [
-        "Authenticated"
-      ], 
       "title": "Plone Team"
     }
   ], 

diff --git a/src/plone/restapi/tests/test_services_groups.py b/src/plone/restapi/tests/test_services_groups.py
@@ -62,8 +62,6 @@ def test_list_groups(self):
         self.assertEqual('Plone Team', ptgroup.get('title'))
         self.assertEqual('We are Plone', ptgroup.get('description'))
 
-        self.assertEqual(ptgroup.get('roles'), ['Authenticated'])
-
         # We don't want the group members listed in the overview as there
         # might be loads.
         self.assertTrue(