Merge pull request #1523 from edx/christina/import-export

Convert import to new URL pattern.

cms/djangoapps/contentstore/tests/test_contentstore.py

@@ -1592,10 +1592,7 @@ class ContentStoreTest(ModuleStoreTestCase):
         # go to various pages
 
         # import page
-        resp = self.client.get(reverse('import_course',
-                                       kwargs={'org': loc.org,
-                                               'course': loc.course,
-                                               'name': loc.name}))
+        resp = self.client.get(new_location.url_reverse('import/', ''), HTTP_ACCEPT='text/html')
         self.assertEqual(resp.status_code, 200)
 
         # export page
@@ -1632,9 +1629,7 @@ class ContentStoreTest(ModuleStoreTestCase):
         self.assertEqual(resp.status_code, 200)
 
         # assets_handler (HTML for full page content)
-        new_location = loc_mapper().translate_location(loc.course_id, loc, False, True)
         url = new_location.url_reverse('assets/', '')
-
         resp = self.client.get(url, HTTP_ACCEPT='text/html')
         self.assertEqual(resp.status_code, 200)
 

cms/djangoapps/contentstore/tests/test_import_export.py

@@ -13,9 +13,9 @@ from uuid import uuid4
 from pymongo import MongoClient
 
 from .utils import CourseTestCase
-from django.core.urlresolvers import reverse
 from django.test.utils import override_settings
 from django.conf import settings
+from xmodule.modulestore.django import loc_mapper
 
 from xmodule.contentstore.django import _CONTENTSTORE
 
@@ -32,11 +32,10 @@ class ImportTestCase(CourseTestCase):
 
     def setUp(self):
         super(ImportTestCase, self).setUp()
-        self.url = reverse("import_course", kwargs={
-            'org': self.course.location.org,
-            'course': self.course.location.course,
-            'name': self.course.location.name,
-        })
+        self.new_location = loc_mapper().translate_location(
+            self.course.location.course_id, self.course.location, False, True
+        )
+        self.url = self.new_location.url_reverse('import/', '')
         self.content_dir = path(tempfile.mkdtemp())
 
         def touch(name):
@@ -89,13 +88,13 @@ class ImportTestCase(CourseTestCase):
         self.assertEquals(resp.status_code, 415)
         # Check that `import_status` returns the appropriate stage (i.e., the
         # stage at which import failed).
-        status_url = reverse("import_status", kwargs={
-            'org': self.course.location.org,
-            'course': self.course.location.course,
-            'name': os.path.split(self.bad_tar)[1],
-        })
-        resp_status = self.client.get(status_url)
-        log.debug(str(self.client.session["import_status"]))
+        resp_status = self.client.get(
+            self.new_location.url_reverse(
+                'import_status',
+                os.path.split(self.bad_tar)[1]
+            )
+        )
+
         self.assertEquals(json.loads(resp_status.content)["ImportStatus"], 2)
 
 
@@ -200,11 +199,11 @@ class ImportTestCase(CourseTestCase):
         # Check that `import_status` returns the appropriate stage (i.e.,
         # either 3, indicating all previous steps are completed, or 0,
         # indicating no upload in progress)
-        status_url = reverse("import_status", kwargs={
-            'org': self.course.location.org,
-            'course': self.course.location.course,
-            'name': os.path.split(self.good_tar)[1],
-        })
-        resp_status = self.client.get(status_url)
+        resp_status = self.client.get(
+            self.new_location.url_reverse(
+                'import_status',
+                os.path.split(self.good_tar)[1]
+            )
+        )
         import_status = json.loads(resp_status.content)["ImportStatus"]
         self.assertIn(import_status, (0, 3))

cms/templates/import.html

 <%inherit file="base.html" />
 <%namespace name='static' file='static_content.html'/>
 <%!
-  from django.core.urlresolvers import reverse
   from django.utils.translation import ugettext as _
 %>
 <%block name="title">${_("Course Import")}</%block>
@@ -28,10 +27,7 @@
           <p>${_("During the initial stages of the import process, please do not navigate away from this page.")}</p>
       </div>
 
-      <form id="fileupload" method="post" enctype="multipart/form-data"
-          class="import-form" url="${reverse('import_course', kwargs=dict(org=context_course.location.org, course=context_course.location.course, name=context_course.location.name))}">
-
-        <input type="hidden" name="csrfmiddlewaretoken" value="${csrf_token}" />
+      <form id="fileupload" method="post" enctype="multipart/form-data" class="import-form">
 
         ## Translators: ".tar.gz" is a file extension, and files with that extension are called "gzipped tar files": these terms should not be translated
         <h2 class="title">${_("Select a File (.tar.gz format) to Replace Your Course Content")}</h2>
@@ -158,7 +154,7 @@ var chooseBtn = $('.choose-file-button');
 
 var allStats = $('#status-infos');
 
-var feedbackUrl = "${reverse('import_status', kwargs=dict(org=context_course.location.org, course=context_course.location.course, name='fillerName'))}"
+var feedbackUrl = "${import_status_url}";
 
 var defaults = [
     '${_("There was an error during the upload process.")}\n',

cms/templates/widgets/header.html

@@ -20,6 +20,7 @@
             checklists_url = location.url_reverse('checklists/', '')
             course_team_url = location.url_reverse('course_team/', '')
             assets_url = location.url_reverse('assets/', '')
+            import_url = location.url_reverse('import/', '')
       %>
       <h2 class="info-course">
         <span class="sr">${_("Current Course:")}</span>
@@ -91,7 +92,7 @@
                     <a href="${checklists_url}">${_("Checklists")}</a>
                   </li>
                   <li class="nav-item nav-course-tools-import">
-                    <a href="${reverse('import_course', kwargs=dict(org=ctx_loc.org, course=ctx_loc.course, name=ctx_loc.name))}">${_("Import")}</a>
+                    <a href="${import_url}">${_("Import")}</a>
                   </li>
                   <li class="nav-item nav-course-tools-export">
                     <a href="${reverse('export_course', kwargs=dict(org=ctx_loc.org, course=ctx_loc.course, name=ctx_loc.name))}">${_("Export")}</a>

cms/urls.py

-import re
 from django.conf import settings
 from django.conf.urls import patterns, include, url
 
@@ -34,11 +33,6 @@ urlpatterns = patterns('',  # nopep8
     url(r'^create_new_course', 'contentstore.views.create_new_course', name='create_new_course'),
     url(r'^reorder_static_tabs', 'contentstore.views.reorder_static_tabs', name='reorder_static_tabs'),
 
-    url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/import/(?P<name>[^/]+)$',
-        'contentstore.views.import_course', name='import_course'),
-    url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/import_status/(?P<name>[^/]+)$',
-        'contentstore.views.import_status', name='import_status'),
-
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/export/(?P<name>[^/]+)$',
         'contentstore.views.export_course', name='export_course'),
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/generate_export/(?P<name>[^/]+)$',
@@ -129,7 +123,9 @@ urlpatterns += patterns(
     url(r'(?ix)^checklists/{}(/)?(?P<checklist_index>\d+)?$'.format(parsers.URL_RE_SOURCE), 'checklists_handler'),
     url(r'(?ix)^course_team/{}(/)?(?P<email>.+)?$'.format(parsers.URL_RE_SOURCE), 'course_team_handler'),
     url(r'(?ix)^orphan/{}$'.format(parsers.URL_RE_SOURCE), 'orphan'),
-    url(r'(?ix)^assets/{}(/)?(?P<asset_id>.+)?$'.format(parsers.URL_RE_SOURCE), 'assets_handler')
+    url(r'(?ix)^assets/{}(/)?(?P<asset_id>.+)?$'.format(parsers.URL_RE_SOURCE), 'assets_handler'),
+    url(r'(?ix)^import/{}$'.format(parsers.URL_RE_SOURCE), 'import_handler'),
+    url(r'(?ix)^import_status/{}/(?P<filename>.+)$'.format(parsers.URL_RE_SOURCE), 'import_status_handler'),
 )
 
 js_info_dict = {