Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change httpflow with event manager #52

Closed
gianarb opened this issue Sep 10, 2015 · 5 comments
Closed

change httpflow with event manager #52

gianarb opened this issue Sep 10, 2015 · 5 comments
Milestone

Comments

@gianarb
Copy link
Contributor

gianarb commented Sep 10, 2015

Only Http? I'm not sure that this is the best name for this concept..

I call it with a classic name EventManager

@gianarb gianarb added this to the 0.3.0 milestone Sep 10, 2015
@samsonasik
Copy link
Member

how about "app.flow" for naming?

Warm regards,

Abdul Malik Ikhsan

Pada 10 Sep 2015, pukul 18.55, Gianluca Arbezzano notifications@github.com menulis:

Only Http? I'm not sure that this is the best name for this concept..

I call it with a classic name EventManager


Reply to this email directly or view it on GitHub.

gianarb pushed a commit that referenced this issue Sep 10, 2015
@gianarb
Copy link
Contributor Author

gianarb commented Sep 10, 2015

Yeh.. :) it sounds good

@gianarb
Copy link
Contributor Author

gianarb commented Sep 10, 2015

@samsonasik at the moment I prefer eventmanager it represents itself :D

gianarb pushed a commit that referenced this issue Sep 12, 2015
Fixed #52 Replace http.flow with eventManager
@fntlnz
Copy link
Contributor

fntlnz commented Sep 12, 2015

I preferred the one suggested by @samsonasik .

app.flow was more significant about the fact that is handling a flow of events around in the application and was even better than http.flow because it made it decoupled from the concept of http in itself.

@gianarb
Copy link
Contributor Author

gianarb commented Sep 12, 2015

to be honest you can use it to trigger all events into the application :)
We can use php-di alias to use it, if this stuff helps to understand :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants