Skip to content

bug: getElemCacheIndex may return null #122

@suconghou

Description

@suconghou

getElemCacheIndex may return null

vue2 called doUnbindEvent more than once

cause destroyed other instance

https://github.com/AlloyTeam/AlloyFinger/blob/master/vue/alloy_finger.vue.js#L93-L98

 if(!isNaN(index)) {
          var delArr = CACHE.splice(index, 1);
          if(delArr.length && delArr[0] && delArr[0].alloyFinger.destroy) {
            delArr[0].alloyFinger.destroy();
          }
        } 

should be

 if(!isNaN(index) && index!==null) {
          var delArr = CACHE.splice(index, 1);
          if(delArr.length && delArr[0] && delArr[0].alloyFinger.destroy) {
            delArr[0].alloyFinger.destroy();
          }
        } 

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions