Quantcast

Change to Zend\Mvc\Application's event identifiers

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate
star

Change to Zend\Mvc\Application's event identifiers

weierophinney
Administrator
Hey, all --

Just a heads-up.

Zend\Mvc\Application was setting the following shared event identifiers
during setEventManager():

    Zend\Mvc\Application
    get_class($this) (would only differ when you extend the class)
    'application'

This latter one has led to some confusion. It's inconsistent with the
rest of the framework, where we do not use short identifiers, only fully
qualified class names, namespaces, and fully qualified interface names.
Additionally, because controllers add an identifier for the top-level
namespace (to allow triggering module-specific listeners), the skeleton
application was additionally defining the identifier "Application".
Since identifiers are case-sensitive, there'd be no collision - but the
situation was confusing.

Internally in the framework, we never referred to the "application"
identifier, and none of the documentation refers to it, either.

I submitted a pull request this morning, and Even merged it, removing
that identifier. While it's technically a BC break, leaving it in would
have led to more issues.

So, if you have any listeners using the "application" identifier, please
update those to use "Zend\Mvc\Application" instead.

--
Matthew Weier O'Phinney
Project Lead            | [hidden email]
Zend Framework          | http://framework.zend.com/
PGP key: http://framework.zend.com/zf-matthew-pgp-key.asc
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate
star

Re: Change to Zend\Mvc\Application's event identifiers

weierophinney
Administrator
-- Matthew Weier O'Phinney <[hidden email]> wrote
(on Friday, 24 August 2012, 12:33 PM -0500):
> I submitted a pull request this morning, and Even merged it, removing

That should have been "Evan". Ooops!

--
Matthew Weier O'Phinney
Project Lead            | [hidden email]
Zend Framework          | http://framework.zend.com/
PGP key: http://framework.zend.com/zf-matthew-pgp-key.asc
Loading...