plone.restapi/expansion: Refactor @navigation view to return a single object with the

Repository: plone.restapi
Branch: refs/heads/expansion
Date: 2017-07-16T12:06:34+02:00
Author: Timo Stollenwerk (tisto) <[email protected]>
Commit: https://github.com/plone/plone.restapi/commit/a0dbb5f9ceeafbf96aa8788de1899d6d1ef81520

Refactor @navigation view to return a single object with the @id attr.

Files changed:
M src/plone/restapi/services/navigation/get.py
M src/plone/restapi/tests/test_services_navigation.py

diff --git a/src/plone/restapi/services/navigation/get.py b/src/plone/restapi/services/navigation/get.py
index a2bf0504..ab48b2e1 100644
--- a/src/plone/restapi/services/navigation/get.py
+++ b/src/plone/restapi/services/navigation/get.py
@@ -8,9 +8,14 @@ class NavigationGet(Service):
     def reply(self):
         tabs = getMultiAdapter((self.context, self.request),
                                name="portal_tabs_view")
-        result = []
+        result = {
+            '@id': '{}/@navigation'.format(
+                self.context.absolute_url()
+            ),
+            'items': []
+        }
         for tab in tabs.topLevelTabs():
-            result.append({
+            result['items'].append({
                 'title': tab.get('title', tab.get('name')),
                 'url': tab['url'] + ''
             })
diff --git a/src/plone/restapi/tests/test_services_navigation.py b/src/plone/restapi/tests/test_services_navigation.py
index ad55a612..a0c47213 100644
--- a/src/plone/restapi/tests/test_services_navigation.py
+++ b/src/plone/restapi/tests/test_services_navigation.py
@@ -41,14 +41,17 @@ def test_navigation(self):
         self.assertEqual(response.status_code, 200)
         self.assertEqual(
             response.json(),
-            [
-                {
-                    u'title': u'Home',
-                    u'url': u'http://localhost:55001/plone'
-                },
-                {
-                    u'title': u'Some Folder',
-                    u'url': u'http://localhost:55001/plone/folder'
-                }
-            ]
+            {
+                '@id': 'http://localhost:55001/plone/folder/@navigation',
+                'items': [
+                    {
+                        u'title': u'Home',
+                        u'url': u'http://localhost:55001/plone'
+                    },
+                    {
+                        u'title': u'Some Folder',
+                        u'url': u'http://localhost:55001/plone/folder'
+                    }
+                ]
+            }
         )



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot