]> source.dussan.org Git - jquery.git/commitdiff
Landing pull request 331. Add support for registering jQuery as an AMD module. Fixes...
authorjrburke <jrburke@gmail.com>
Mon, 19 Sep 2011 20:37:22 +0000 (16:37 -0400)
committertimmywil <timmywillisn@gmail.com>
Mon, 19 Sep 2011 20:37:22 +0000 (16:37 -0400)
More Details:
 - https://github.com/jquery/jquery/pull/331
 - http://bugs.jquery.com/ticket/7102

1  2 
src/core.js
test/data/testinit.js
test/unit/core.js

diff --cc src/core.js
index cee1cee45fcc704a27a782830c6d66885ab4cd80,9fae9ae28424372996552bfa631046fd251e6a96..7f30edea2b2ab03faa8da7b62b76de937fa65cae
@@@ -935,6 -930,21 +935,20 @@@ function doScrollCheck() 
        jQuery.ready();
  }
  
 -// Expose jQuery to the global object
+ // Expose jQuery as an AMD module, but only for AMD loaders that
+ // understand the issues with loading multiple versions of jQuery
+ // in a page that all might call define(). The loader will indicate
+ // they have special allowances for multiple jQuery versions by
+ // specifying define.amd.jQuery = true. Register as a named module,
+ // since jQuery can be concatenated with other files that may use define,
+ // but not use a proper concatenation script that understands anonymous
+ // AMD modules. A named AMD is safest and most robust way to register.
+ // Lowercase jquery is used because AMD module names are derived from
+ // file names, and jQuery is normally delivered in a lowercase file name.
+ if ( typeof define === "function" && define.amd && define.amd.jQuery ) {
+       define( "jquery", [], function () { return jQuery; } );
+ }
  return jQuery;
  
  })();
Simple merge
Simple merge