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

Placeholder View Helper "containerExists" makes no sense #32

Closed
michalbundyra opened this issue Jan 15, 2020 · 0 comments · Fixed by #153
Closed

Placeholder View Helper "containerExists" makes no sense #32

michalbundyra opened this issue Jan 15, 2020 · 0 comments · Fixed by #153
Labels
Bug Something isn't working
Milestone

Comments

@michalbundyra
Copy link
Member

At the moment the public function containerExists does not have any change to work:

because __invoke checks if a name is set it is impossible to call placeholder()
So to check if a placeholder name exists you have to all call
placeholder('foobar')->containerExists('foobar')

but then the container is created in __invoke and containerExists returns in every case true.

And because placeholder('dummy') returns a container containerExists is never reachable and usable.


Originally posted by @bernhard-efler at zendframework/zend-view#16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
2 participants