Skip to content
Permalink
Browse files
fix(v-model): fix trim modifier on events with non-string args ( (#5770)
fix #5765
  • Loading branch information
shadowings-zy committed Oct 26, 2022
1 parent bb06819 commit 018b8503994c2dae3c42dd03ea50d062956be970
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 1 deletion.
@@ -385,6 +385,28 @@ describe('component: emit', () => {
expect(fn2).toHaveBeenCalledTimes(1)
expect(fn2).toHaveBeenCalledWith(1)
})

test('only trim string parameter when work with v-model on component', () => {
const Foo = defineComponent({
render() {},
created() {
this.$emit('update:modelValue', ' foo ', { bar: ' bar ' })
}
})

const fn = jest.fn()
const Comp = () =>
h(Foo, {
modelValue: null,
modelModifiers: { trim: true },
'onUpdate:modelValue': fn
})

render(h(Comp), nodeOps.createElement('div'))

expect(fn).toHaveBeenCalledTimes(1)
expect(fn).toHaveBeenCalledWith('foo', { bar: ' bar ' })
})

test('isEmitListener', () => {
const options = {
@@ -8,6 +8,7 @@ import {
isArray,
isFunction,
isObject,
isString,
isOn,
toNumber,
UnionToIntersection
@@ -122,7 +123,7 @@ export function emit(
}Modifiers`
const { number, trim } = props[modifiersKey] || EMPTY_OBJ
if (trim) {
args = rawArgs.map(a => a.trim())
args = rawArgs.map(a => isString(a) ? a.trim() : a)
}
if (number) {
args = rawArgs.map(toNumber)

0 comments on commit 018b850

Please sign in to comment.