refactor: xmodule/error_module.py -> xmodule/error_block.py
This commit is contained in:
@@ -78,7 +78,7 @@ from openedx.features.content_type_gating.partitions import CONTENT_TYPE_GATING_
|
|||||||
from openedx.features.course_experience.waffle import ENABLE_COURSE_ABOUT_SIDEBAR_HTML
|
from openedx.features.course_experience.waffle import ENABLE_COURSE_ABOUT_SIDEBAR_HTML
|
||||||
from xmodule.contentstore.content import StaticContent # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.contentstore.content import StaticContent # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.course_block import CourseBlock, DEFAULT_START_DATE, CourseFields # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.course_block import CourseBlock, DEFAULT_START_DATE, CourseFields # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore import EdxJSONEncoder # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore import EdxJSONEncoder # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.exceptions import DuplicateCourseError, ItemNotFoundError # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.exceptions import DuplicateCourseError, ItemNotFoundError # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|||||||
@@ -17,7 +17,7 @@ from common.djangoapps.student.roles import GlobalStaff
|
|||||||
from common.djangoapps.student.tests.factories import UserFactory
|
from common.djangoapps.student.tests.factories import UserFactory
|
||||||
from common.djangoapps.student.views import get_course_enrollments
|
from common.djangoapps.student.views import get_course_enrollments
|
||||||
from common.djangoapps.util.milestones_helpers import get_pre_requisite_courses_not_completed, set_prerequisite_courses
|
from common.djangoapps.util.milestones_helpers import get_pre_requisite_courses_not_completed, set_prerequisite_courses
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore import ModuleStoreEnum # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore import ModuleStoreEnum # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|||||||
@@ -14,7 +14,7 @@ from lazy import lazy
|
|||||||
from opaque_keys.edx.django.models import CourseKeyField, UsageKeyField
|
from opaque_keys.edx.django.models import CourseKeyField, UsageKeyField
|
||||||
from pytz import utc
|
from pytz import utc
|
||||||
|
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.modulestore.django import modulestore
|
from xmodule.modulestore.django import modulestore
|
||||||
|
|
||||||
log = logging.getLogger("edx.ccx")
|
log = logging.getLogger("edx.ccx")
|
||||||
|
|||||||
@@ -62,7 +62,7 @@ from common.djangoapps.util.milestones_helpers import (
|
|||||||
is_prerequisite_courses_enabled
|
is_prerequisite_courses_enabled
|
||||||
)
|
)
|
||||||
from xmodule.course_block import CATALOG_VISIBILITY_ABOUT, CATALOG_VISIBILITY_CATALOG_AND_ABOUT, CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.course_block import CATALOG_VISIBILITY_ABOUT, CATALOG_VISIBILITY_CATALOG_AND_ABOUT, CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.partitions.partitions import NoSuchUserPartitionError, NoSuchUserPartitionGroupError # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.partitions.partitions import NoSuchUserPartitionError, NoSuchUserPartitionGroupError # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|
||||||
log = logging.getLogger(__name__)
|
log = logging.getLogger(__name__)
|
||||||
|
|||||||
@@ -20,7 +20,7 @@ from openedx.core.djangoapps.enrollments.api import is_enrollment_valid_for_proc
|
|||||||
from common.djangoapps.student.models import CourseAccessRole
|
from common.djangoapps.student.models import CourseAccessRole
|
||||||
from common.djangoapps.student.roles import CourseRole, OrgRole
|
from common.djangoapps.student.roles import CourseRole, OrgRole
|
||||||
from xmodule.course_block import CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.course_block import CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|
||||||
|
|
||||||
from .access import has_access
|
from .access import has_access
|
||||||
|
|||||||
@@ -12,7 +12,7 @@ from opaque_keys.edx.keys import CourseKey
|
|||||||
|
|
||||||
from lms.djangoapps.courseware.tests.helpers import LoginEnrollmentTestCase
|
from lms.djangoapps.courseware.tests.helpers import LoginEnrollmentTestCase
|
||||||
from lms.djangoapps.lms_xblock.field_data import LmsFieldData
|
from lms.djangoapps.lms_xblock.field_data import LmsFieldData
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.tests.django_utils import TEST_DATA_MIXED_MODULESTORE, ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.tests.django_utils import TEST_DATA_MIXED_MODULESTORE, ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.tests.factories import ToyCourseFactory # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.tests.factories import ToyCourseFactory # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|||||||
@@ -29,7 +29,7 @@ from openedx.core.lib.cache_utils import request_cached, RequestCache
|
|||||||
from common.djangoapps.static_replace.models import AssetBaseUrlConfig
|
from common.djangoapps.static_replace.models import AssetBaseUrlConfig
|
||||||
from xmodule import block_metadata_utils, course_metadata_utils # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule import block_metadata_utils, course_metadata_utils # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.course_block import DEFAULT_START_DATE, CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.course_block import DEFAULT_START_DATE, CourseBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.tabs import CourseTab # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.tabs import CourseTab # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|
||||||
|
|||||||
@@ -33,7 +33,7 @@ from xmodule.course_block import ( # lint-amnesty, pylint: disable=wrong-import
|
|||||||
CATALOG_VISIBILITY_CATALOG_AND_ABOUT,
|
CATALOG_VISIBILITY_CATALOG_AND_ABOUT,
|
||||||
CATALOG_VISIBILITY_NONE
|
CATALOG_VISIBILITY_NONE
|
||||||
)
|
)
|
||||||
from xmodule.error_module import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.error_block import ErrorBlock # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore import ModuleStoreEnum # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore import ModuleStoreEnum # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.django import modulestore # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase # lint-amnesty, pylint: disable=wrong-import-order
|
||||||
|
|||||||
2
setup.py
2
setup.py
@@ -16,7 +16,7 @@ XBLOCKS = [
|
|||||||
"custom_tag_template = xmodule.template_module:CustomTagTemplateBlock",
|
"custom_tag_template = xmodule.template_module:CustomTagTemplateBlock",
|
||||||
"discuss = xmodule.template_module:TranslateCustomTagBlock",
|
"discuss = xmodule.template_module:TranslateCustomTagBlock",
|
||||||
"discussion = xmodule.discussion_block:DiscussionXBlock",
|
"discussion = xmodule.discussion_block:DiscussionXBlock",
|
||||||
"error = xmodule.error_module:ErrorBlock",
|
"error = xmodule.error_block:ErrorBlock",
|
||||||
"hidden = xmodule.hidden_module:HiddenDescriptor",
|
"hidden = xmodule.hidden_module:HiddenDescriptor",
|
||||||
"html = xmodule.html_module:HtmlBlock",
|
"html = xmodule.html_module:HtmlBlock",
|
||||||
"image = xmodule.template_module:TranslateCustomTagBlock",
|
"image = xmodule.template_module:TranslateCustomTagBlock",
|
||||||
|
|||||||
@@ -113,7 +113,7 @@ class ErrorBlock(
|
|||||||
})
|
})
|
||||||
return system.construct_xblock_from_class(
|
return system.construct_xblock_from_class(
|
||||||
cls,
|
cls,
|
||||||
# The error module doesn't use scoped data, and thus doesn't need
|
# The error block doesn't use scoped data, and thus doesn't need
|
||||||
# real scope keys
|
# real scope keys
|
||||||
ScopeIds(None, 'error', location, location),
|
ScopeIds(None, 'error', location, location),
|
||||||
field_data,
|
field_data,
|
||||||
@@ -303,7 +303,7 @@ def inherit_metadata(descriptor, inherited_data):
|
|||||||
"""
|
"""
|
||||||
try:
|
try:
|
||||||
descriptor.xblock_kvs.inherited_settings = inherited_data
|
descriptor.xblock_kvs.inherited_settings = inherited_data
|
||||||
except AttributeError: # the kvs doesn't have inherited_settings probably b/c it's an error module
|
except AttributeError: # the kvs doesn't have inherited_settings probably b/c it's an error block
|
||||||
pass
|
pass
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -36,7 +36,7 @@ from xblock.runtime import KvsFieldData
|
|||||||
|
|
||||||
from xmodule.assetstore import AssetMetadata, CourseAssetsFromStorage
|
from xmodule.assetstore import AssetMetadata, CourseAssetsFromStorage
|
||||||
from xmodule.course_block import CourseSummary
|
from xmodule.course_block import CourseSummary
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.errortracker import exc_info_to_str, null_error_tracker
|
from xmodule.errortracker import exc_info_to_str, null_error_tracker
|
||||||
from xmodule.exceptions import HeartbeatFailure
|
from xmodule.exceptions import HeartbeatFailure
|
||||||
from xmodule.mako_module import MakoDescriptorSystem
|
from xmodule.mako_module import MakoDescriptorSystem
|
||||||
|
|||||||
@@ -9,7 +9,7 @@ from opaque_keys.edx.locator import BlockUsageLocator, DefinitionLocator, LocalI
|
|||||||
from xblock.fields import ScopeIds
|
from xblock.fields import ScopeIds
|
||||||
from xblock.runtime import KeyValueStore, KvsFieldData
|
from xblock.runtime import KeyValueStore, KvsFieldData
|
||||||
|
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.errortracker import exc_info_to_str
|
from xmodule.errortracker import exc_info_to_str
|
||||||
from xmodule.library_tools import LibraryToolsService
|
from xmodule.library_tools import LibraryToolsService
|
||||||
from xmodule.mako_module import MakoDescriptorSystem
|
from xmodule.mako_module import MakoDescriptorSystem
|
||||||
|
|||||||
@@ -80,7 +80,7 @@ from xblock.fields import Reference, ReferenceList, ReferenceValueDict, Scope
|
|||||||
|
|
||||||
from xmodule.assetstore import AssetMetadata
|
from xmodule.assetstore import AssetMetadata
|
||||||
from xmodule.course_block import CourseSummary
|
from xmodule.course_block import CourseSummary
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.errortracker import null_error_tracker
|
from xmodule.errortracker import null_error_tracker
|
||||||
from xmodule.library_content_module import LibrarySummary
|
from xmodule.library_content_module import LibrarySummary
|
||||||
from xmodule.modulestore import (
|
from xmodule.modulestore import (
|
||||||
|
|||||||
@@ -48,7 +48,7 @@ class TestXMLModuleStore(TestCase):
|
|||||||
with pytest.raises(UnicodeDecodeError):
|
with pytest.raises(UnicodeDecodeError):
|
||||||
xml.decode('ascii')
|
xml.decode('ascii')
|
||||||
|
|
||||||
# Load the course, but don't make error modules. This will succeed,
|
# Load the course, but don't make error blocks. This will succeed,
|
||||||
# but will record the errors.
|
# but will record the errors.
|
||||||
modulestore = XMLModuleStore(
|
modulestore = XMLModuleStore(
|
||||||
DATA_DIR,
|
DATA_DIR,
|
||||||
|
|||||||
@@ -23,7 +23,7 @@ from xblock.fields import ScopeIds
|
|||||||
from xblock.runtime import DictKeyValueStore
|
from xblock.runtime import DictKeyValueStore
|
||||||
|
|
||||||
from common.djangoapps.util.monitoring import monitor_import_failure
|
from common.djangoapps.util.monitoring import monitor_import_failure
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.errortracker import exc_info_to_str, make_error_tracker
|
from xmodule.errortracker import exc_info_to_str, make_error_tracker
|
||||||
from xmodule.mako_module import MakoDescriptorSystem
|
from xmodule.mako_module import MakoDescriptorSystem
|
||||||
from xmodule.modulestore import COURSE_ROOT, LIBRARY_ROOT, ModuleStoreEnum, ModuleStoreReadBase
|
from xmodule.modulestore import COURSE_ROOT, LIBRARY_ROOT, ModuleStoreEnum, ModuleStoreReadBase
|
||||||
|
|||||||
@@ -14,7 +14,7 @@ from xblock.field_data import DictFieldData
|
|||||||
from xblock.fields import ScopeIds
|
from xblock.fields import ScopeIds
|
||||||
|
|
||||||
from xmodule.conditional_block import ConditionalBlock
|
from xmodule.conditional_block import ConditionalBlock
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.modulestore.xml import CourseLocationManager, ImportSystem, XMLModuleStore
|
from xmodule.modulestore.xml import CourseLocationManager, ImportSystem, XMLModuleStore
|
||||||
from xmodule.tests import DATA_DIR, get_test_descriptor_system, get_test_system
|
from xmodule.tests import DATA_DIR, get_test_descriptor_system, get_test_system
|
||||||
from xmodule.tests.xml import XModuleXmlImportTest
|
from xmodule.tests.xml import XModuleXmlImportTest
|
||||||
|
|||||||
@@ -7,7 +7,7 @@ import unittest
|
|||||||
|
|
||||||
from opaque_keys.edx.locator import CourseLocator
|
from opaque_keys.edx.locator import CourseLocator
|
||||||
|
|
||||||
from xmodule.error_module import ErrorBlock
|
from xmodule.error_block import ErrorBlock
|
||||||
from xmodule.modulestore.xml import CourseLocationManager
|
from xmodule.modulestore.xml import CourseLocationManager
|
||||||
from xmodule.tests import get_test_system
|
from xmodule.tests import get_test_system
|
||||||
from xmodule.x_module import STUDENT_VIEW
|
from xmodule.x_module import STUDENT_VIEW
|
||||||
Reference in New Issue
Block a user