Add missing @return annotations #3808

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

Contributor
micheh commented Feb 17, 2013

No description provided.

@samsonasik samsonasik commented on the diff Feb 17, 2013
library/Zend/Permissions/Rbac/Rbac.php
@@ -123,8 +123,9 @@ public function getRole($objectOrName)
/**
* Determines if access is granted by checking the role and child roles for permission.
*
- * @param string $permission
- * @param \Zend\Permissions\Rbac\AssertionInterface|Callable|null $assert
+ * @param string $permission
+ * @param \Zend\Permissions\Rbac\AssertionInterface|Callable|null $assert
samsonasik
samsonasik Feb 17, 2013 Contributor

in same folder, it should be only AssertionInterface|Callable|null $assert without \Zend\Permissions\Rbac\ prefix.

micheh
micheh Feb 19, 2013 Contributor

The phpDoc of this method misses the $role parameter as well. But both changes should not be part of this PR, as that's not the topic of this PR.

@weierophinney weierophinney added a commit that referenced this pull request Feb 19, 2013
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop
Forward port #3808
438686c
@weierophinney weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' 1ff4f12
@weierophinney weierophinney added a commit to zendframework/zend-test that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop c84afac
@weierophinney weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' f7d6cbb
@weierophinney weierophinney added a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop 6b7a960
@weierophinney weierophinney added a commit to zendframework/zend-crypt that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' e547383
@weierophinney weierophinney added a commit to zendframework/zend-crypt that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop 0ca6653
@weierophinney weierophinney added a commit to zendframework/zend-filter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' aed1a4d
@weierophinney weierophinney added a commit to zendframework/zend-filter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop a813d6e
@weierophinney weierophinney added a commit to zendframework/zend-permissions-rbac that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' 9515119
@weierophinney weierophinney added a commit to zendframework/zend-permissions-rbac that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3808' into develop 2a58924
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment