Commit 9643fd46 by Calen Pennington

Rename xblock_field_data to xmodule_field_data

parent 02d8b68a
...@@ -113,7 +113,7 @@ def preview_module_system(request, preview_id, descriptor): ...@@ -113,7 +113,7 @@ def preview_module_system(request, preview_id, descriptor):
debug=True, debug=True,
replace_urls=partial(static_replace.replace_static_urls, data_directory=None, course_id=course_id), replace_urls=partial(static_replace.replace_static_urls, data_directory=None, course_id=course_id),
user=request.user, user=request.user,
xblock_field_data=preview_field_data, xmodule_field_data=preview_field_data,
can_execute_unsafe_code=(lambda: can_execute_unsafe_code(course_id)), can_execute_unsafe_code=(lambda: can_execute_unsafe_code(course_id)),
mixins=settings.XBLOCK_MIXINS, mixins=settings.XBLOCK_MIXINS,
course_id=course_id, course_id=course_id,
......
...@@ -65,7 +65,7 @@ def get_test_system(course_id=''): ...@@ -65,7 +65,7 @@ def get_test_system(course_id=''):
hostname="edx.org", hostname="edx.org",
xqueue={'interface': None, 'callback_url': '/', 'default_queuename': 'testqueue', 'waittime': 10, 'construct_callback' : Mock(side_effect="/")}, xqueue={'interface': None, 'callback_url': '/', 'default_queuename': 'testqueue', 'waittime': 10, 'construct_callback' : Mock(side_effect="/")},
node_path=os.environ.get("NODE_PATH", "/usr/local/lib/node_modules"), node_path=os.environ.get("NODE_PATH", "/usr/local/lib/node_modules"),
xblock_field_data=lambda descriptor: descriptor._field_data, xmodule_field_data=lambda descriptor: descriptor._field_data,
anonymous_student_id='student', anonymous_student_id='student',
open_ended_grading_interface=open_ended_grading_interface, open_ended_grading_interface=open_ended_grading_interface,
course_id=course_id, course_id=course_id,
......
...@@ -74,7 +74,7 @@ class TestXBlockWrapper(object): ...@@ -74,7 +74,7 @@ class TestXBlockWrapper(object):
anonymous_student_id='dummy_anonymous_student_id', anonymous_student_id='dummy_anonymous_student_id',
open_ended_grading_interface={}, open_ended_grading_interface={},
ajax_url='dummy_ajax_url', ajax_url='dummy_ajax_url',
xblock_field_data=lambda d: d._field_data, xmodule_field_data=lambda d: d._field_data,
get_module=Mock(), get_module=Mock(),
replace_urls=Mock(), replace_urls=Mock(),
track_function=Mock(), track_function=Mock(),
......
...@@ -537,7 +537,7 @@ class XModuleDescriptor(XModuleFields, HTMLSnippet, ResourceTemplates, XBlock): ...@@ -537,7 +537,7 @@ class XModuleDescriptor(XModuleFields, HTMLSnippet, ResourceTemplates, XBlock):
module = system.construct_xblock_from_class( module = system.construct_xblock_from_class(
self.module_class, self.module_class,
descriptor=self, descriptor=self,
field_data=system.xblock_field_data(self), field_data=system.xmodule_field_data(self),
scope_ids=self.scope_ids, scope_ids=self.scope_ids,
) )
module.save() module.save()
...@@ -838,7 +838,7 @@ class ModuleSystem(Runtime): ...@@ -838,7 +838,7 @@ class ModuleSystem(Runtime):
''' '''
def __init__( def __init__(
self, ajax_url, track_function, get_module, render_template, self, ajax_url, track_function, get_module, render_template,
replace_urls, xblock_field_data, user=None, filestore=None, replace_urls, xmodule_field_data, user=None, filestore=None,
debug=False, hostname="", xqueue=None, publish=None, node_path="", debug=False, hostname="", xqueue=None, publish=None, node_path="",
anonymous_student_id='', course_id=None, anonymous_student_id='', course_id=None,
open_ended_grading_interface=None, s3_interface=None, open_ended_grading_interface=None, s3_interface=None,
...@@ -883,7 +883,7 @@ class ModuleSystem(Runtime): ...@@ -883,7 +883,7 @@ class ModuleSystem(Runtime):
publish(event) - A function that allows XModules to publish events (such as grade changes) publish(event) - A function that allows XModules to publish events (such as grade changes)
xblock_field_data - A function that constructs a field_data for an xblock from its xmodule_field_data - A function that constructs a field_data for an xblock from its
corresponding descriptor corresponding descriptor
cache - A cache object with two methods: cache - A cache object with two methods:
...@@ -910,7 +910,7 @@ class ModuleSystem(Runtime): ...@@ -910,7 +910,7 @@ class ModuleSystem(Runtime):
self.anonymous_student_id = anonymous_student_id self.anonymous_student_id = anonymous_student_id
self.course_id = course_id self.course_id = course_id
self.user_is_staff = user is not None and user.is_staff self.user_is_staff = user is not None and user.is_staff
self.xblock_field_data = xblock_field_data self.xmodule_field_data = xmodule_field_data
if publish is None: if publish is None:
publish = lambda e: None publish = lambda e: None
......
...@@ -294,7 +294,7 @@ def get_module_for_descriptor_internal(user, descriptor, field_data_cache, cours ...@@ -294,7 +294,7 @@ def get_module_for_descriptor_internal(user, descriptor, field_data_cache, cours
position, wrap_xmodule_display, grade_bucket_type, position, wrap_xmodule_display, grade_bucket_type,
static_asset_path) static_asset_path)
def xblock_field_data(descriptor): def xmodule_field_data(descriptor):
student_data = DbModel(DjangoKeyValueStore(field_data_cache)) student_data = DbModel(DjangoKeyValueStore(field_data_cache))
return lms_field_data(descriptor._field_data, student_data) return lms_field_data(descriptor._field_data, student_data)
...@@ -368,7 +368,7 @@ def get_module_for_descriptor_internal(user, descriptor, field_data_cache, cours ...@@ -368,7 +368,7 @@ def get_module_for_descriptor_internal(user, descriptor, field_data_cache, cours
jump_to_id_base_url=reverse('jump_to_id', kwargs={'course_id': course_id, 'module_id': ''}) jump_to_id_base_url=reverse('jump_to_id', kwargs={'course_id': course_id, 'module_id': ''})
), ),
node_path=settings.NODE_PATH, node_path=settings.NODE_PATH,
xblock_field_data=xblock_field_data, xmodule_field_data=xmodule_field_data,
publish=publish, publish=publish,
anonymous_student_id=unique_id_for_user(user), anonymous_student_id=unique_id_for_user(user),
course_id=course_id, course_id=course_id,
......
...@@ -48,7 +48,7 @@ class BaseTestXmodule(ModuleStoreTestCase): ...@@ -48,7 +48,7 @@ class BaseTestXmodule(ModuleStoreTestCase):
DATA = '' DATA = ''
MODEL_DATA = {'data': '<some_module></some_module>'} MODEL_DATA = {'data': '<some_module></some_module>'}
def xblock_field_data(self, descriptor): def xmodule_field_data(self, descriptor):
field_data = {} field_data = {}
field_data.update(self.MODEL_DATA) field_data.update(self.MODEL_DATA)
student_data = DictFieldData(field_data) student_data = DictFieldData(field_data)
...@@ -91,7 +91,7 @@ class BaseTestXmodule(ModuleStoreTestCase): ...@@ -91,7 +91,7 @@ class BaseTestXmodule(ModuleStoreTestCase):
# different code paths while maintaining the type returned by render_template # different code paths while maintaining the type returned by render_template
self.runtime.render_template = lambda template, context: u'{!r}, {!r}'.format(template, sorted(context.items())) self.runtime.render_template = lambda template, context: u'{!r}, {!r}'.format(template, sorted(context.items()))
self.runtime.xblock_field_data = self.xblock_field_data self.runtime.xmodule_field_data = self.xmodule_field_data
self.item_module = self.item_descriptor.xmodule(self.runtime) self.item_module = self.item_descriptor.xmodule(self.runtime)
......
...@@ -13,6 +13,8 @@ from xmodule.x_module import ModuleSystem ...@@ -13,6 +13,8 @@ from xmodule.x_module import ModuleSystem
from mitxmako.shortcuts import render_to_string from mitxmako.shortcuts import render_to_string
import datetime import datetime
from xblock.field_data import DictFieldData
log = logging.getLogger(__name__) log = logging.getLogger(__name__)
NOTIFICATION_CACHE_TIME = 300 NOTIFICATION_CACHE_TIME = 300
...@@ -68,7 +70,7 @@ def peer_grading_notifications(course, user): ...@@ -68,7 +70,7 @@ def peer_grading_notifications(course, user):
get_module = None, get_module = None,
render_template=render_to_string, render_template=render_to_string,
replace_urls=None, replace_urls=None,
xblock_field_data= {} xmodule_field_data=DictFieldData({}),
) )
peer_gs = peer_grading_service.PeerGradingService(settings.OPEN_ENDED_GRADING_INTERFACE, system) peer_gs = peer_grading_service.PeerGradingService(settings.OPEN_ENDED_GRADING_INTERFACE, system)
pending_grading = False pending_grading = False
...@@ -129,7 +131,7 @@ def combined_notifications(course, user): ...@@ -129,7 +131,7 @@ def combined_notifications(course, user):
get_module = None, get_module = None,
render_template=render_to_string, render_template=render_to_string,
replace_urls=None, replace_urls=None,
xblock_field_data= {} xmodule_field_data=DictFieldData({})
) )
#Initialize controller query service using our mock system #Initialize controller query service using our mock system
controller_qs = ControllerQueryService(settings.OPEN_ENDED_GRADING_INTERFACE, system) controller_qs = ControllerQueryService(settings.OPEN_ENDED_GRADING_INTERFACE, system)
......
...@@ -9,6 +9,8 @@ from xmodule.open_ended_grading_classes.grading_service_module import GradingSer ...@@ -9,6 +9,8 @@ from xmodule.open_ended_grading_classes.grading_service_module import GradingSer
from django.conf import settings from django.conf import settings
from django.http import HttpResponse, Http404 from django.http import HttpResponse, Http404
from xblock.field_data import DictFieldData
from courseware.access import has_access from courseware.access import has_access
from util.json_request import expect_json from util.json_request import expect_json
from xmodule.course_module import CourseDescriptor from xmodule.course_module import CourseDescriptor
...@@ -73,7 +75,7 @@ class StaffGradingService(GradingService): ...@@ -73,7 +75,7 @@ class StaffGradingService(GradingService):
get_module = None, get_module = None,
render_template=render_to_string, render_template=render_to_string,
replace_urls=None, replace_urls=None,
xblock_field_data= {} xmodule_field_data=DictFieldData({})
) )
super(StaffGradingService, self).__init__(config) super(StaffGradingService, self).__init__(config)
self.url = config['url'] + config['staff_grading'] self.url = config['url'] + config['staff_grading']
......
...@@ -167,7 +167,7 @@ class TestPeerGradingService(ModuleStoreTestCase, LoginEnrollmentTestCase): ...@@ -167,7 +167,7 @@ class TestPeerGradingService(ModuleStoreTestCase, LoginEnrollmentTestCase):
get_module=None, get_module=None,
render_template=render_to_string, render_template=render_to_string,
replace_urls=None, replace_urls=None,
xblock_field_data=lambda d: d._field_data, xmodule_field_data=lambda d: d._field_data,
s3_interface=test_util_open_ended.S3_INTERFACE, s3_interface=test_util_open_ended.S3_INTERFACE,
open_ended_grading_interface=test_util_open_ended.OPEN_ENDED_GRADING_INTERFACE, open_ended_grading_interface=test_util_open_ended.OPEN_ENDED_GRADING_INTERFACE,
mixins=settings.XBLOCK_MIXINS, mixins=settings.XBLOCK_MIXINS,
......
...@@ -7,6 +7,8 @@ from django.views.decorators.cache import cache_control ...@@ -7,6 +7,8 @@ from django.views.decorators.cache import cache_control
from mitxmako.shortcuts import render_to_response from mitxmako.shortcuts import render_to_response
from django.core.urlresolvers import reverse from django.core.urlresolvers import reverse
from xblock.field_data import DictFieldData
from student.models import unique_id_for_user from student.models import unique_id_for_user
from courseware.courses import get_course_with_access from courseware.courses import get_course_with_access
...@@ -33,7 +35,7 @@ system = ModuleSystem( ...@@ -33,7 +35,7 @@ system = ModuleSystem(
get_module=None, get_module=None,
render_template=render_to_string, render_template=render_to_string,
replace_urls=None, replace_urls=None,
xblock_field_data={} xmodule_field_data=DictFieldData({}),
) )
controller_qs = ControllerQueryService(settings.OPEN_ENDED_GRADING_INTERFACE, system) controller_qs = ControllerQueryService(settings.OPEN_ENDED_GRADING_INTERFACE, system)
......
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