Skip to content

Commit

Permalink
Fix InvalidWriteError when viewing a combined open ended problem in t…
Browse files Browse the repository at this point in the history
…he LMS as an instructor.
  • Loading branch information
Chris Rossi committed Oct 11, 2013
1 parent 8f57c59 commit eb76cf1
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion lms/djangoapps/courseware/model_data.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,13 @@ def set_many(self, kv_dict):
for field in kv_dict:
# Check field for validity
if field.field_name in self._descriptor_model_data:
raise InvalidWriteError("Not allowed to overwrite descriptor model data", field.field_name)
# xblock model data will set any mutable field as dirty whether
# it's been mutated or not. In light of that it's better to
# silently skip these fields rather than raise an error.
#raise InvalidWriteError(
# "Not allowed to overwrite descriptor model data",
# field.field_name)
continue

if field.scope not in self._allowed_scopes:
raise InvalidScopeError(field.scope)
Expand Down

0 comments on commit eb76cf1

Please sign in to comment.