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

Placeholder View Helper "containerExists" makes no sense #16

Open
bernhard-efler opened this issue Oct 2, 2015 · 1 comment
Open

Placeholder View Helper "containerExists" makes no sense #16

bernhard-efler opened this issue Oct 2, 2015 · 1 comment

Comments

@bernhard-efler
Copy link

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.

@michalbundyra
Copy link
Member

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

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

No branches or pull requests

2 participants