diff --git a/common/lib/xmodule/xmodule/course_module.py b/common/lib/xmodule/xmodule/course_module.py index 0f9ccdce1f..e51b6e7d3e 100644 --- a/common/lib/xmodule/xmodule/course_module.py +++ b/common/lib/xmodule/xmodule/course_module.py @@ -917,6 +917,19 @@ class CourseDescriptor(CourseFields, SequenceDescriptor): return set(config.get("cohorted_discussions", [])) + @property + def always_cohort_inline_discussions(self): + """ + This allow to change the default behavior of inline discussions cohorting. By + setting this to False, all inline discussions are non-cohorted unless their + ids are specified in cohorted_discussions. + """ + config = self.cohort_config + if config is None: + return True + + return bool(config.get("always_cohort_inline_discussions", True)) + @property def is_newish(self): """ diff --git a/common/lib/xmodule/xmodule/discussion_module.py b/common/lib/xmodule/xmodule/discussion_module.py index 1bcdf20fb9..4e952ec983 100644 --- a/common/lib/xmodule/xmodule/discussion_module.py +++ b/common/lib/xmodule/xmodule/discussion_module.py @@ -11,7 +11,11 @@ _ = lambda text: text class DiscussionFields(object): - discussion_id = String(scope=Scope.settings, default="$$GUID$$") + discussion_id = String( + display_name=_("Discussion Id"), + help=_("The id is a unique identifier for the discussion. It is non editable."), + scope=Scope.settings, + default="$$GUID$$") display_name = String( display_name=_("Display Name"), help=_("Display name for this module"), diff --git a/common/static/coffee/spec/discussion/discussion_spec_helper.coffee b/common/static/coffee/spec/discussion/discussion_spec_helper.coffee index 5d693794c9..446f8a9448 100644 --- a/common/static/coffee/spec/discussion/discussion_spec_helper.coffee +++ b/common/static/coffee/spec/discussion/discussion_spec_helper.coffee @@ -321,7 +321,7 @@ browser and pasting the output. When that file changes, this one should be rege