diff src/pages/merge-requests/page_meta.py @ 1081:ed67265205c4

pages (merge-requests): added a discovery page similar to tickets one: new "mr_handlers_json" option allows to add well known merge-requests handlers in configuration.
author Goffi <goffi@goffi.org>
date Tue, 27 Mar 2018 08:40:12 +0200
parents d8666fdb015c
children 3996acd7c796
line wrap: on
line diff
--- a/src/pages/merge-requests/page_meta.py	Tue Mar 27 08:36:37 2018 +0200
+++ b/src/pages/merge-requests/page_meta.py	Tue Mar 27 08:40:12 2018 +0200
@@ -2,51 +2,29 @@
 # -*- coding: utf-8 -*-
 
 from libervia.server.constants import Const as C
-from sat.core.i18n import _
 from twisted.internet import defer
-from twisted.words.protocols.jabber import jid
 from sat.tools.common import template_xmlui
 from sat.tools.common import data_objects
 from sat.core.log import getLogger
 log = getLogger('pages/ticket')
 """ticket handling pages"""
 
-name = u'merge-requests_list'
+name = u'merge-requests'
 access = C.PAGES_ACCESS_PUBLIC
 template = u"ticket/overview.html"
 
 
 def parse_url(self, request):
-    # check the service and node to use
-    try:
-        service = self.nextPath(request)
-        node = self.nextPath(request)
-    except IndexError:
-        log.warning(_(u"missing service and node"))
-        self.pageError(request, C.HTTP_BAD_REQUEST)
-
-    if not service or service == u'@':
-        service = u''
-
-    if service:
-        try:
-            service = jid.JID(service)
-        except Exception:
-            log.warning(_(u"bad service entered: {}").format(service))
-            self.pageError(request, C.HTTP_BAD_REQUEST)
-
-    if not node or node == u'@':
-        node = u''
-
-
+    self.getPathArgs(request, ['service', 'node'], service='jid')
+    data = self.getRData(request)
+    service, node = data[u'service'], data[u'node']
+    if node is None:
+        self.pageRedirect(u"merge-requests_disco", request)
+    if node == u'@':
+        node = data[u'node'] = u''
     self.checkCache(request, C.CACHE_PUBSUB, service=service, node=node, short='merge-requests')
-
-    data = self.getRData(request)
-    data['service'] = service
-    data['node'] = node
-
     template_data = request.template_data
-    template_data[u'url_tickets_list'] = self.getPageByName('merge-requests_list').getURL(service.full(), node or u'@')
+    template_data[u'url_tickets_list'] = self.getPageByName('merge-requests').getURL(service.full(), node)
     template_data[u'url_tickets_new'] = self.getSubPageURL(request, 'merge-requests_new')
 
 @defer.inlineCallbacks