Be able to delete array item in Vue.delete

[Hanks10100]

Since the Vue.set support to insert an item to array, the Vue.delete should also support to delete an item from array.

Take the following code as an example:

const vm = new Vue({
  template: '<div><span v-for="obj in lists">{{obj.name}}</span></div>',
  data: {
    lists: [
      { name: 'A' },
      { name: 'B' }
    ]
  }
}).$mount()

Vue.delete(vm.lists, 1)
// vm.lists.length--

The Vue.delete(vm.lists, 1) just delete the property 1 from the array, but the vm.lists.length is still be 2. Evaluating obj.name at the second time will caught a TypeError (undefined is not an object).

[posva]

key is either a string or a number, isn't it?

[Hanks10100]

Yes, I just keep the type check be same with Vue.set

[posva]

I see, in that case, we should probably update both

[Hanks10100]

The conflict is the first argument of splice must be number, can't be string.

[posva]

I must be missing something 😅 what is wrong with key: number | string

[Hanks10100]

I have tried, if I set the key: number | string, the type check of obj.splice(key, 1) would be failed. Because the first argument of splice must be a number.

[yyx990803]

blank line here please :)

[Hanks10100]

OK, I will also remove the blank line after .$mount()