plone.restapi breaks Plone 5.0.4 File bulk uploads about plone.restapi

[tisto]

@mjmare thanks for the detailed bug report. we will look into it!

[mjmare]

You're welcome!

[lukasgraf]

This issue most likely originates in plone.rest.

The failing request is made to the fileUpload browser view with Accept: application/json.

If a request with an Accept header other than text/html is handled, mark_as_api_request() flags the request with IAPIRequest as soon as at least one service registered for that content-type and HTTP method exists in the service registry.

This will then cause a KeyError to be raised in plone.rest.traverse because the conditions don't match. => 404

Failing test (for plone.rest):

#class TestTraversal(unittest.TestCase):

    def test_json_request_to_regular_view_still_possible(self):
        obj = self.traverse('/plone/some-view')
        self.assertEquals(obj(), '{"msg": "Hello from SomeView"}')

from Products.Five.browser import BrowserView
import json

class SomeView(BrowserView):
    def __call__(self):
        return json.dumps({'msg': 'Hello from SomeView'})

  <browser:page
    name="some-view"
    for="*"
    class=".tests.view.SomeView"
    permission="zope2.View"
    />

/cc @buchi

[tisto]

This is actually a plone.rest issue. Moved it there: plone/plone.rest#50

[lukasgraf]

@mjmare fixed in current master of plone.rest (see plone/plone.rest#53).