Skip to main content
Code Review

Return to Answer

Commonmark migration
Source Link
Community Bot
Community Bot
  • 1

Why not use an existing ajax file upload library, like Fine Uploader?

As for your code. #1) I

1)

I think you meant to return _this.methods.init.apply(this, arguments);. #2) You

2)

You should rewrite Uploader to not use arguments.callee for two reasons; it makes your code harder to read and for the browser to optimize.

Why not use an existing ajax file upload library, like Fine Uploader?

As for your code. #1) I think you meant to return _this.methods.init.apply(this, arguments);. #2) You should rewrite Uploader to not use arguments.callee for two reasons; it makes your code harder to read and for the browser to optimize.

Why not use an existing ajax file upload library, like Fine Uploader?

As for your code.

1)

I think you meant to return _this.methods.init.apply(this, arguments);.

2)

You should rewrite Uploader to not use arguments.callee for two reasons; it makes your code harder to read and for the browser to optimize.

Source Link
Larry Battle
Larry Battle
  • 2.2k
  • 11
  • 19

Why not use an existing ajax file upload library, like Fine Uploader?

As for your code. #1) I think you meant to return _this.methods.init.apply(this, arguments);. #2) You should rewrite Uploader to not use arguments.callee for two reasons; it makes your code harder to read and for the browser to optimize.