diff --git a/cms/djangoapps/contentstore/views/course.py b/cms/djangoapps/contentstore/views/course.py index 8188e75dbd..ad8b239fd7 100644 --- a/cms/djangoapps/contentstore/views/course.py +++ b/cms/djangoapps/contentstore/views/course.py @@ -470,12 +470,7 @@ def course_info_update_handler(request, tag=None, package_id=None, branch=None, course_location = loc_mapper().translate_locator_to_location( CourseLocator(package_id=package_id), get_course=True ) - # add location to loc_mapper - updates_locator = loc_mapper().translate_location( - course_location.course_id, course_location.replace(category='course_info', name='updates'), - False, True - ) - updates_location = loc_mapper().translate_locator_to_location(updates_locator) + updates_location = course_location.replace(category='course_info', name=block) if provided_id == '': provided_id = None diff --git a/cms/djangoapps/contentstore/views/tests/test_course_updates.py b/cms/djangoapps/contentstore/views/tests/test_course_updates.py index 1997f0de33..447cfab75f 100644 --- a/cms/djangoapps/contentstore/views/tests/test_course_updates.py +++ b/cms/djangoapps/contentstore/views/tests/test_course_updates.py @@ -150,7 +150,7 @@ class CourseUpdateTest(CourseTestCase): def test_post_course_update(self): """ - Test posting a course update through api on a newly created course updates location map with course_updates + Test that a user can successfully post on course updates of a course whose location in not in loc_mapper """ # create a course via the view handler course_location = Location(['i4x', 'Org_1', 'Course_1', 'course', 'Run_1']) @@ -174,7 +174,6 @@ class CourseUpdateTest(CourseTestCase): package_id=course_location.course_id.replace('/', '.'), branch=branch, version_guid=version, block_id=block ) - # test that user can successfully post on course updates whose location in not added in loc_mapper content = u"Sample update" payload = {'content': content, 'date': 'January 8, 2013'} course_update_url = updates_locator.url_reverse('course_info_update') @@ -185,3 +184,9 @@ class CourseUpdateTest(CourseTestCase): payload = json.loads(resp.content) self.assertHTMLEqual(payload['content'], content) + + # now test that calling translate_location returns a locator whose block_id is 'updates' + updates_location = course_location.replace(category='course_info', name=block) + updates_locator = loc_mapper().translate_location(course_location.course_id, updates_location) + self.assertTrue(isinstance(updates_locator, BlockUsageLocator)) + self.assertEqual(updates_locator.block_id, block)