zend-mvc icon indicating copy to clipboard operation
zend-mvc copied to clipboard

Fix controller namespace in event manager

Open GeeH opened this issue 8 years ago • 2 comments

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Original Issue: https://api.github.com/repos/zendframework/zendframework/issues/7457 User: @0xPaul Created On: 2015-04-22T09:29:40Z Updated At: 2015-05-04T14:59:12Z Body Controller was only registering the root namespace, e.g. Foo instead of Foo\Bar for a class named Foo\Bar\Baz.


Comment

User: @Martin-P Created On: 2015-04-28T19:52:56Z Updated At: 2015-04-28T19:52:56Z Body Perhaps you can add:

  • a unittest which shows the correct namespace is returned
  • a separate unittest which shows the wrong namespace is not returned

That way regressions can be avoided.


Comment

User: @0xPaul Created On: 2015-04-29T08:48:33Z Updated At: 2015-04-29T08:48:33Z Body If by you, you mean someone, then yes. I'm sure someone can do those things.


Comment

User: @Martin-P Created On: 2015-04-29T09:09:15Z Updated At: 2015-04-29T09:09:15Z Body

If by you, you mean someone, then yes. I'm sure someone can do those things.

I'm sure someone can create unittests, but this is not _someone_s PR, it is your PR. So I meant you, @0xPaul :wink:


GeeH avatar Jun 28 '16 13:06 GeeH

I have a Ns\SubNs\Controller\IndexController and confirm it is being picked up as Ns\SubNs... There is an undocumented feature, "view_manager" => ["controller_map" => ["Ns" => true]] which was required for the view templates to be in ns/subns/index as opposed to ns/index...

internalsystemerror avatar Dec 29 '16 21:12 internalsystemerror

This repository has been closed and moved to laminas/laminas-mvc; a new issue has been opened at https://github.com/laminas/laminas-mvc/issues/29.

weierophinney avatar Dec 31 '19 21:12 weierophinney