Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Can't set a controller map template replacement to an empty string #12

Open
carnage opened this issue Jul 11, 2015 · 2 comments
Open

Can't set a controller map template replacement to an empty string #12

carnage opened this issue Jul 11, 2015 · 2 comments

Comments

@carnage
Copy link
Contributor

carnage commented Jul 11, 2015

The Inject template listener allows specifying a controller_map to map namespaces or specific controllers to directories. It is not currently possible to specify a mapping for a namespace to '' eg:

'controller_map' => [
    'my/long/namespace' => '',
]

This is due to a == comparison with false in the listener on line 142:

which should probably be ===

Fixing this may break BC...

@Ocramius
Copy link
Member

Suggestions:

  • proposed fix is a bc break, no-go, even for 3.x (it is subtle)
  • use a separate listener for this functionality, and make it act with a lower priority (should be BC compliant then)

@michalbundyra
Copy link
Member

This repository has been closed and moved to laminas/laminas-mvc; a new issue has been opened at laminas/laminas-mvc#38.

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

No branches or pull requests

3 participants