Rename contentstore/views/requests to helpers
This has caused confusion, since this file gets imported when trying to import the requests library. And besides, how was this file about requests in the first place? Helpers is a much better name.
This commit is contained in:
@@ -3,7 +3,7 @@ import mock
|
||||
|
||||
from django.test import TestCase
|
||||
from django.core.urlresolvers import reverse
|
||||
from contentstore.views.requests import event as cms_user_track
|
||||
from contentstore.views.helpers import event as cms_user_track
|
||||
|
||||
|
||||
class CMSLogTest(TestCase):
|
||||
|
||||
@@ -9,13 +9,13 @@ from .checklist import *
|
||||
from .component import *
|
||||
from .course import *
|
||||
from .error import *
|
||||
from .helpers import *
|
||||
from .item import *
|
||||
from .import_export import *
|
||||
from .preview import *
|
||||
from .public import *
|
||||
from .user import *
|
||||
from .tabs import *
|
||||
from .requests import *
|
||||
try:
|
||||
from .dev import *
|
||||
except ImportError:
|
||||
|
||||
@@ -26,7 +26,7 @@ from contentstore.utils import (get_modulestore, get_lms_link_for_item,
|
||||
|
||||
from models.settings.course_grading import CourseGradingModel
|
||||
|
||||
from .requests import _xmodule_recurse
|
||||
from .helpers import _xmodule_recurse
|
||||
from .access import has_access
|
||||
from xmodule.x_module import XModuleDescriptor
|
||||
from xblock.plugin import PluginMissingError
|
||||
|
||||
@@ -12,7 +12,7 @@ from xmodule.modulestore.inheritance import own_metadata
|
||||
from util.json_request import expect_json, JsonResponse
|
||||
from ..utils import get_modulestore
|
||||
from .access import has_access
|
||||
from .requests import _xmodule_recurse
|
||||
from .helpers import _xmodule_recurse
|
||||
from xmodule.x_module import XModuleDescriptor
|
||||
|
||||
__all__ = ['save_item', 'create_item', 'delete_item']
|
||||
|
||||
@@ -22,7 +22,7 @@ from util.sandboxing import can_execute_unsafe_code
|
||||
|
||||
import static_replace
|
||||
from .session_kv_store import SessionKeyValueStore
|
||||
from .requests import render_from_lms
|
||||
from .helpers import render_from_lms
|
||||
from .access import has_access
|
||||
from ..utils import get_course_for_item
|
||||
|
||||
|
||||
@@ -129,7 +129,7 @@ class TestPublish(unittest.TestCase):
|
||||
"""
|
||||
Applies action depth-first down tree and to item last.
|
||||
|
||||
A copy of cms.djangoapps.contentstore.views.requests._xmodule_recurse to reproduce its use and behavior
|
||||
A copy of cms.djangoapps.contentstore.views.helpers._xmodule_recurse to reproduce its use and behavior
|
||||
outside of django.
|
||||
"""
|
||||
for child in item.get_children():
|
||||
|
||||
Reference in New Issue
Block a user