Increase event name readability/writability

[gianarb]

I know that this proposal broke compatibility but I think to change

My proposal is:
To change classname.method (indexcontroller.index) with class.method.. In this way we can replace

$eventManager->attach("indexcontroller.index", ...)

with

$eventManager->attach(IndexController::class.".index", ...)

Maybe this approach is more readable/writable.. What do you think?

[EmanueleMinotto]

I think a :: should be used as separator for systems like that, this separator is also used "natively" by PHP to invoke static method: http://php.net/manual/en/language.types.callable.php (Example 1, Type 4)

[gianarb]

I use this syntax to print the namespace.. IndexController::class
IndexController::class.".index" == "App\Controller\IndexController.index"}

[EmanueleMinotto]

@gianarb reuse it? :D

[gianarb]

Guys.. Feedback to change this code?!
@pennyphp/community-review @wdalmut @EmanueleMinotto

IMO indexcontroller.index is very very very very ugly..

App\Controller\IndexController.index
App\Controller\IndexController.index_error
maybe it sounds better

[samsonasik]

+1 for ::class usage

[samsonasik]

also, classname.method make collision when there is same classname but different namespace