Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

requirejs patch for anonymous define()

  • Loading branch information...
commit 6f9361b31420deee51ed9d4963fee38f43bcb25d 1 parent 08f795d
@vuongdothanhhuy vuongdothanhhuy authored
Showing with 2 additions and 2 deletions.
  1. +1 −1  dist/jspdf.debug.js
  2. +1 −1  jspdf.js
View
2  dist/jspdf.debug.js
@@ -1700,7 +1700,7 @@ var jsPDF = (function(global) {
jsPDF.version = "1.0.178-debug 2014-06-27T15:34:diegocr";
if (typeof define === 'function' && define.amd) {
- define(function() {
+ define('jsPDF', function() {
return jsPDF;
});
} else {
View
2  jspdf.js
@@ -1700,7 +1700,7 @@ var jsPDF = (function(global) {
jsPDF.version = "1.0.0-trunk";
if (typeof define === 'function' && define.amd) {
- define(function() {
+ define('jsPDF', function() {
return jsPDF;
});
} else {

2 comments on commit 6f9361b

@metaman

This patch seems to have broken the ability to requirejs in the file. When I require in the latest version I get undefined however if I revert this change everything loads ok. Have you got any ideas? I know you are just naming the module so strange the impact it has had.

@vuongdothanhhuy

Hi @metaman,
Thank you for your feedback. I'm also quite surprise how could one very simple and legal change can result in undefined in the latest requirejs. I will look into it as soon as I have time. Meanwhile, it would be very welcome if anyone can come and help us! (especially those who master AMD/requirejs).

Please sign in to comment.
Something went wrong with that request. Please try again.