MDL-57197 js: fix AMD module warnings
authorDan Poltawski <dan@moodle.com>
Wed, 30 Nov 2016 13:49:38 +0000 (13:49 +0000)
committerDan Poltawski <dan@moodle.com>
Wed, 30 Nov 2016 13:50:38 +0000 (13:50 +0000)
In 3.2 we are free of AMD lint warnings, unfortunately we can't expose
them through grunt yet due to
"File ignored because of a matching ignore pattern. Use "--no-ignore" to
override"

ps. this isn't related to MDL-57197 at all, i'm just a Rebel

admin/tool/templatelibrary/amd/src/search.js
theme/boost/amd/build/drawer.min.js
theme/boost/amd/src/drawer.js

index 04d27b0..18623df 100644 (file)
@@ -41,6 +41,7 @@ define(['jquery', 'core/ajax', 'core/log', 'core/notification', 'core/templates'
      * Get the current values for the form inputs and refresh the list of matching templates.
      *
      * @method refreshSearch
+     * @param {String} themename The naeme of the theme.
      */
     var refreshSearch = function(themename) {
         var componentStr = $('[data-field="component"]').val();
index e77a4e3..854b6b9 100644 (file)
Binary files a/theme/boost/amd/build/drawer.min.js and b/theme/boost/amd/build/drawer.min.js differ
index 733b64f..a0aae1b 100644 (file)
@@ -83,7 +83,7 @@ define(['jquery', 'core/custom_interaction_events', 'core/log'],
             drawer.attr('aria-hidden', 'true');
             drawer.addClass('closed');
             M.util.set_user_preference(preference, 'false');
-        }.bind(this));
+        });
     };
 
     /**
@@ -128,6 +128,7 @@ define(['jquery', 'core/custom_interaction_events', 'core/log'],
      * Prevent the page from scrolling when the drawer is at max scroll.
      *
      * @method preventPageScroll
+     * @param  {Event} e
      */
     Drawer.prototype.preventPageScroll = function(e) {
         var delta = e.wheelDelta || (e.originalEvent && e.originalEvent.wheelDelta) || -e.originalEvent.detail,