node-js-libs/chain.js

IE7 issue with thenRun

revolunet opened this issue · 3 comments

Hi Chris, thanks for your handy lib.

I have an issue with IE7, see https://gist.github.com/806143

On line 80, this doesnt work with IE

 if (!--len ) {
        handlers['then' +method[0].toUpperCase() + method.substr(1)] = handler;
  }
  createChain(exports);

method[0] is undefined

any idea ?

Thank you

Julien

I try to use thenRun from load.js this may be related.

Hi Julien, I don't currently have access to a windows machine to be able to test this out (I switch between Mac and Ubuntu) - but next time I'm on a windows machine I'll try and fix it. There's no reason why this would fail though - I've tested it in all recent browsers. Take a look at line 82 of chain.js.. method is defined as the first argument of the function addMethod(). You could try replacing method[0] with method.substr(0,1) and seeing if that works..

0

this seems to work much better thank you !