MDL-34737 Course: Modchooser should use coursebase to determine the section class
authorAndrew Nicols <andrew@nicols.co.uk>
Mon, 17 Feb 2014 14:30:59 +0000 (22:30 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Thu, 22 May 2014 04:30:32 +0000 (12:30 +0800)
course/format/upgrade.txt
course/yui/build/moodle-course-modchooser/moodle-course-modchooser-debug.js
course/yui/build/moodle-course-modchooser/moodle-course-modchooser-min.js
course/yui/build/moodle-course-modchooser/moodle-course-modchooser.js
course/yui/src/modchooser/js/modchooser.js

index fffbb93..9d6bc3e 100644 (file)
@@ -2,6 +2,10 @@ This files describes API changes for course formats
 
 Overview of this plugin type at http://docs.moodle.org/dev/Course_formats
 
 
 Overview of this plugin type at http://docs.moodle.org/dev/Course_formats
 
+=== 2.8 ===
+* The activity chooser now uses M.course.format.get_sectionwrapperclass()
+  to determine the section selector, rather than a hard-coded `li.section`.
+
 === 2.7 ===
 * The ->testedbrowsers array no longer needs to be defined in supports_ajax().
 * format_section_renderer_base::section_hidden has an new second optional argument $courseorid.
 === 2.7 ===
 * The ->testedbrowsers array no longer needs to be defined in supports_ajax().
 * format_section_renderer_base::section_hidden has an new second optional argument $courseorid.
index 54312b4..1689ac9 100644 (file)
Binary files a/course/yui/build/moodle-course-modchooser/moodle-course-modchooser-debug.js and b/course/yui/build/moodle-course-modchooser/moodle-course-modchooser-debug.js differ
index 0425628..1f08287 100644 (file)
Binary files a/course/yui/build/moodle-course-modchooser/moodle-course-modchooser-min.js and b/course/yui/build/moodle-course-modchooser/moodle-course-modchooser-min.js differ
index 54312b4..1689ac9 100644 (file)
Binary files a/course/yui/build/moodle-course-modchooser/moodle-course-modchooser.js and b/course/yui/build/moodle-course-modchooser/moodle-course-modchooser.js differ
index da70ad8..ef6ac4d 100644 (file)
@@ -6,7 +6,7 @@
 
 var CSS = {
     PAGECONTENT : 'body',
 
 var CSS = {
     PAGECONTENT : 'body',
-    SECTION : 'li.section',
+    SECTION: null,
     SECTIONMODCHOOSER : 'span.section-modchooser-link',
     SITEMENU : 'div.block_site_main_menu',
     SITETOPIC : 'div.sitetopic'
     SECTIONMODCHOOSER : 'span.section-modchooser-link',
     SITEMENU : 'div.block_site_main_menu',
     SITETOPIC : 'div.sitetopic'
@@ -42,6 +42,10 @@ Y.extend(MODCHOOSER, M.core.chooserdialogue, {
      * @method initializer
      */
     initializer : function() {
      * @method initializer
      */
     initializer : function() {
+        var sectionclass = M.course.format.get_sectionwrapperclass();
+        if (sectionclass) {
+            CSS.SECTION = '.' + sectionclass;
+        }
         var dialogue = Y.one('.chooserdialoguebody');
         var header = Y.one('.choosertitle');
         var params = {};
         var dialogue = Y.one('.chooserdialoguebody');
         var header = Y.one('.choosertitle');
         var params = {};
@@ -73,9 +77,11 @@ Y.extend(MODCHOOSER, M.core.chooserdialogue, {
         }, this);
 
         // Setup for standard course topics
         }, this);
 
         // Setup for standard course topics
-        Y.one(baseselector).all(CSS.SECTION).each(function(section) {
-            this._setup_for_section(section);
-        }, this);
+        if (CSS.SECTION) {
+            Y.one(baseselector).all(CSS.SECTION).each(function(section) {
+                this._setup_for_section(section);
+            }, this);
+        }
 
         // Setup for the block site menu
         Y.one(baseselector).all(CSS.SITEMENU).each(function(section) {
 
         // Setup for the block site menu
         Y.one(baseselector).all(CSS.SITEMENU).each(function(section) {