Commit b5cb1264 by Nimisha Asthagiri

fixup! course_api

parent 505efe02
...@@ -92,7 +92,7 @@ class BlockListGetForm(Form): ...@@ -92,7 +92,7 @@ class BlockListGetForm(Form):
try: try:
requested_user = User.objects.get(username=requested_username) requested_user = User.objects.get(username=requested_username)
except (User.DoesNotExist): except (User.DoesNotExist):
raise Http404("'{username}' does not exist.".format(username=requested_username)) raise Http404("Requested user '{username}' does not exist.".format(username=requested_username))
# verify whether the requested user's blocks can be accessed # verify whether the requested user's blocks can be accessed
if not can_access_users_blocks(requested_user, course_key): if not can_access_users_blocks(requested_user, course_key):
......
...@@ -53,7 +53,7 @@ class BlockSerializer(serializers.Serializer): ...@@ -53,7 +53,7 @@ class BlockSerializer(serializers.Serializer):
if 'children' in self.context['requested_fields']: if 'children' in self.context['requested_fields']:
children = self.context['block_structure'].get_children(block_key) children = self.context['block_structure'].get_children(block_key)
if children: if children:
data['children'] = children data['children'] = [unicode(child) for child in children]
return data return data
......
...@@ -131,3 +131,9 @@ class TestCourseBlocksView(ModuleStoreTestCase): ...@@ -131,3 +131,9 @@ class TestCourseBlocksView(ModuleStoreTestCase):
self.assert_in_iff('format', block_data, xblock.format is not None) self.assert_in_iff('format', block_data, xblock.format is not None)
self.assert_true_iff(block_data['student_view_multi_device'], block_data['type'] == 'html') self.assert_true_iff(block_data['student_view_multi_device'], block_data['type'] == 'html')
self.assertNotIn('not_a_field', block_data) self.assertNotIn('not_a_field', block_data)
if xblock.has_children:
self.assertSetEqual(
set(unicode(child.location) for child in xblock.get_children()),
set(block_data['children']),
)
...@@ -97,6 +97,11 @@ CACHES = { ...@@ -97,6 +97,11 @@ CACHES = {
'BACKEND': 'django.core.cache.backends.locmem.LocMemCache', 'BACKEND': 'django.core.cache.backends.locmem.LocMemCache',
'LOCATION': 'edx_course_structure_mem_cache', 'LOCATION': 'edx_course_structure_mem_cache',
}, },
'lms.course_blocks': {
'BACKEND': 'django.core.cache.backends.locmem.LocMemCache',
'KEY_FUNCTION': 'util.memcache.safe_key',
'LOCATION': 'lms_course_blocks_cache',
},
} }
......
...@@ -234,9 +234,3 @@ SECRET_KEY = '85920908f28904ed733fe576320db18cabd7b6cd' ...@@ -234,9 +234,3 @@ SECRET_KEY = '85920908f28904ed733fe576320db18cabd7b6cd'
##################################################################### #####################################################################
# Mobile # Mobile
FEATURES['ENABLE_RENDER_XBLOCK_API'] = True FEATURES['ENABLE_RENDER_XBLOCK_API'] = True
CACHES['lms.course_blocks'] = {
'BACKEND': 'django.core.cache.backends.locmem.LocMemCache',
'KEY_FUNCTION': 'util.memcache.safe_key',
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment