Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[Navigation] Allow non-string permissions #4564

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

davidwindell commented May 30, 2013

This allows, for example, an array of permissions or permission objects to be attached to a page.

Member

ralphschindler commented Jun 6, 2013

Can you attach some unit tests for this change, or direct me to the unit tests that might already cover this use case?

Contributor

davidwindell commented Jun 13, 2013

@ralphschindler There's none in there for string only either at present. I will try and get some in when I have a minute..

Contributor

davidwindell commented Jun 18, 2013

@ralphschindler tests added, ready for merge.

@weierophinney weierophinney added a commit that referenced this pull request Jun 28, 2013

@weierophinney weierophinney Merge pull request #4564 from davidwindell/patch-10
[Navigation] Allow non-string permissions
4dbc37a

@weierophinney weierophinney added a commit that referenced this pull request Jun 28, 2013

@weierophinney weierophinney Merge branch 'hotfix/4564' into develop
Forward port #4564
bfea6b0

@ghost ghost assigned weierophinney Jun 28, 2013

@weierophinney weierophinney added a commit to zendframework/zend-navigation that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge pull request zendframework/zendframework#4564 from davidwindell…
…/patch-10

[Navigation] Allow non-string permissions
cad5c8b

@weierophinney weierophinney added a commit to zendframework/zend-navigation that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/4564' 2e7e46b

@weierophinney weierophinney added a commit to zendframework/zend-navigation that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/4564' into develop d43b026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment