view src/pages/merge-requests/view/page_meta.py @ 1050:6c98c0baa038

pages (common/blog/atom.xml): don't use query elements in request_uri: when used in self link, some search engine bots (google bot at least) are trying to retrieve page with "comments_max" query argument, which is then returned in the feed in self link, and it get added in a infinite loop.
author Goffi <goffi@goffi.org>
date Thu, 25 Jan 2018 08:17:29 +0100
parents b92b06f023cb
children 1c09f41e2f52
line wrap: on
line source

#!/usr/bin/env python2.7
# -*- coding: utf-8 -*-

from libervia.server.constants import Const as C
from sat.core.i18n import _
from twisted.internet import defer
from sat.tools.common import template_xmlui
from sat.tools.common import uri
from sat.tools.common import data_objects
from sat.core.log import getLogger
log = getLogger('pages/tickets/view')
"""ticket handling pages"""

name = u"merge-requests_view"
access = C.PAGES_ACCESS_PUBLIC
template = u"merge-request/item.html"


def parse_url(self, request):
    try:
        item_id = self.nextPath(request)
    except IndexError:
        log.warning(_(u"no ticket id specified"))
        self.pageError(request, C.HTTP_BAD_REQUEST)

    data = self.getRData(request)
    data[u'ticket_id'] = item_id

@defer.inlineCallbacks
def prepare_render(self, request):
    data = self.getRData(request)
    template_data = request.template_data
    service, node, ticket_id = data.get(u'service', u''), data.get(u'node', u''), data[u'ticket_id']
    profile = self.getProfile(request)

    if profile is None:
        profile = C.SERVICE_PROFILE

    tickets, metadata, parsed_tickets = yield self.host.bridge.mergeRequestsGet(service.full() if service else u'',
                                                                                node,
                                                                                C.NO_LIMIT,
                                                                                [ticket_id],
                                                                                '',
                                                                                {'parse': C.BOOL_TRUE},
                                                                                profile)
    ticket = template_xmlui.create(self.host, tickets[0], ignore=['publisher', 'request_data', 'type'])
    template_data[u'item'] = ticket
    template_data['patches'] = parsed_tickets[0]
    comments_uri = ticket.widgets['comments_uri'].value
    if comments_uri:
        uri_data = uri.parseXMPPUri(comments_uri)
        template_data['comments_node'] = comments_node = uri_data['node']
        template_data['comments_service'] = comments_service = uri_data['path']
        comments = yield self.host.bridge.mbGet(comments_service,
                                                comments_node,
                                                C.NO_LIMIT,
                                                [],
                                                {},
                                                profile)

        template_data[u'comments'] = data_objects.BlogItems(comments)
        template_data[u'login_url'] = self.getPageRedirectURL(request)

@defer.inlineCallbacks
def on_data_post(self, request):
    type_ = self.getPostedData(request, u'type')
    if type_ == u'comment':
        blog_page = self.getPageByName(u'blog')
        yield blog_page.on_data_post(self, request)
    else:
        log.warning(_(u"Unhandled data type: {}").format(type_))