Module not compatible with CommonJS

[IvanGuardado]

Here the code is accessing to the Slick global variable directly without taking in account that Slick could be defined in the exports variable.

https://github.com/mootools/mootools-core/blob/master/Source/Element/Element.js#L264

The module should be defined like the other, using:

;function(){
  //code...
}).apply(/*<CommonJS>*/(typeof exports != 'undefined') ? exports : /*</CommonJS>*/this);

---

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[stefanklokgieters]

What is the exact fix here? :)

[IvanGuardado]

@stefanklokgieters two years after I don't remember so well but the module should be defined following the CommonJS convention like the others.

[stefanklokgieters]

I fully get that... :)

Can you perhaps try to help me a bit further anyway? What changes exactly needed to be made? I am familiar with the ModuleJS conventions but I am not sure what you exactly mean with the proper definition. All the help is very welcome, thanks in advance!