Changes between Initial Version and Version 1 of Ticket #15096, comment 10


Ignore:
Timestamp:
Apr 2, 2012, 11:09:39 PM (8 years ago)
Author:
Aleksey Rechinskiy
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #15096, comment 10

    initial v1  
    22> So it's failing on has("ie-event-behavior"), because {{{hasCache["ie-event-behavior"]}}} is the attachEvent() function, rather than a boolean.
    33
    4 I don't understand why you think it's the cause of the failure. But anyway, if that is true, how changing definition of {{{has()}}} function (see [#comment:3 here]) could have helped me to prevent the bug to appear? The only difference from original code is in argument of {{{isFunction()}}}. I changed it from {{{hasCache[name]}}} that refers to {{{name}}} attribute directly to {{{ ( typeof hasCache.name != 'undefined' ? hasCache.name : 0 ) }}} that checks at first whether the attribute exists. The fix removes the failure and it is completely unrelated to what you have said.
     4I don't understand why you think it's the cause of the failure. But anyway, if that is true, how changing definition of {{{has()}}} function (see [#comment:3 here]) could have helped me to prevent the bug to appear? The only difference from original code is in argument of {{{isFunction()}}}. I changed it from {{{hasCache[name]}}} that refers to {{{name}}} attribute directly to {{{ ( typeof hasCache.name != 'undefined' ? hasCache.name : 0 ) }}} that checks first whether the attribute exist. The fix removes the failure and it is completely unrelated to what you have said.