Skip to content

Event refactor #730

@ahmed3mar

Description

@ahmed3mar

Before feedback (在反馈之前)

  • There are no features that I want to submit in Issues (当前 Issues 中没有我将要提交的新功能)

Describe Feature (描述功能)

As the current implementation only have Register, Job and get events and deals only with struct

and some cases not work for example

type Trigger {
    Name string
}

facades.Event.Job(&Trigger{ Name: "x" })

pointer will not be the same

also need to pass event into the listner as need to pass same listners for multiple events


need to refactor to match laravel function naming and structure same in
https://laravel.com/docs/12.x/events

reference source
https://github.com/laravel/framework/tree/12.x/src/Illuminate/Events

as we can see here even trigger can be string or pointer as we will convert it into string to solve the top issue

Listen clouse, string & listener, pointer & listner same in laravel
Dispatch can be triggered without and params and can be triggered using same event

also need this implementation to implment events in cache same in
https://github.com/laravel/framework/tree/12.x/src/Illuminate/Cache/Events

will break it down today

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

Status

🏗 In progress

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions