fix(createEventHook): trigger should not ignore falsy values (#3561)

Co-authored-by: 丶远方 <yangpanteng@gmail.com>

packages/shared/createEventHook/index.test.ts

@@ -39,6 +39,25 @@ describe('createEventHook', () => {
     expect(timesFired).toBe(2)
   })
 
+  it('should trigger event and pass falsy values', () => {
+    let timesFired = 0
+
+    type Falsy = false | 0 | '' | null | undefined
+    const { on: onResult, trigger } = createEventHook<Falsy>()
+
+    const values: Falsy[] = [false, 0, '', null, undefined]
+    const results: Falsy[] = []
+    onResult((value: Falsy) => {
+      timesFired++
+      results.push(value)
+    })
+    for (const value of values)
+      trigger(value)
+
+    expect(timesFired).toBe(values.length)
+    expect(results).toMatchObject(values)
+  })
+
   it('should add and remove event listener', () => {
     const listener = vi.fn()
     const { on, off, trigger } = createEventHook<string>()

packages/shared/createEventHook/index.ts

@@ -48,8 +48,8 @@ export function createEventHook<T = any>(): EventHook<T> {
     }
   }
 
-  const trigger: EventHookTrigger<T> = (param?: T) => {
-    return Promise.all(Array.from(fns).map(fn => param ? fn(param) : (fn as Callback<void>)()))
+  const trigger: EventHookTrigger<T> = (...args) => {
+    return Promise.all(Array.from(fns).map(fn => fn(...(args as [T]))))
   }
 
   return {