Commit 27b57bb4 by Calen Pennington

Don't update settings based on data from content for discussion modules during init

parent 993c24b7
...@@ -17,21 +17,12 @@ class DiscussionModule(XModule): ...@@ -17,21 +17,12 @@ class DiscussionModule(XModule):
discussion_target = String(scope=Scope.settings) discussion_target = String(scope=Scope.settings)
sort_key = String(scope=Scope.settings) sort_key = String(scope=Scope.settings)
data = String(help="XML definition of inline discussion", scope=Scope.content)
def get_html(self): def get_html(self):
context = { context = {
'discussion_id': self.discussion_id, 'discussion_id': self.discussion_id,
} }
return self.system.render_template('discussion/_discussion_module.html', context) return self.system.render_template('discussion/_discussion_module.html', context)
def __init__(self, *args, **kwargs):
XModule.__init__(self, *args, **kwargs)
xml_data = etree.fromstring(self.data)
self.discussion_id = xml_data.attrib['id']
self.title = xml_data.attrib['for']
self.discussion_category = xml_data.attrib['discussion_category']
class DiscussionDescriptor(RawDescriptor): class DiscussionDescriptor(RawDescriptor):
module_class = DiscussionModule module_class = DiscussionModule
......
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