Merge pull request #1951 from normanjaeckel/Autoupdate

Fixed problem with some REST requests during autoupdate. Fixed #1935.
This commit is contained in:
Emanuel Schütze 2016-02-07 14:17:33 +01:00
commit c40dfb120b
2 changed files with 14 additions and 9 deletions

View File

@ -188,7 +188,8 @@ angular.module('OpenSlidesApp.core', [
} }
DS.eject(data.collection, data.id); DS.eject(data.collection, data.id);
} }
// TODO: handle other statuscodes // If you want to handle more status codes, change server
// restrictions in utils/autoupdate.py.
}); });
} }
]) ])

View File

@ -79,6 +79,10 @@ class OpenSlidesSockJSConnection(SockJSConnection):
This method is called after succesful response of AsyncHTTPClient(). This method is called after succesful response of AsyncHTTPClient().
See send_object(). See send_object().
""" """
if response.code in (200, 404):
# Only send something to the client in case of one of these status
# codes. You have to change the client code (autoupdate.onMessage)
# if you want to handle some more codes.
collection, obj_id = get_collection_and_id_from_url(response.request.url) collection, obj_id = get_collection_and_id_from_url(response.request.url)
data = { data = {
'url': response.request.url, 'url': response.request.url,