fixed comments

Elgg · Nov 30, 2017 · 5f8d11e · 5f8d11e
1 parent 7fdc81d
commit 5f8d11e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/engine/classes/ElggEntity.php b/engine/classes/ElggEntity.php
@@ -926,7 +926,7 @@ public function getOwnedAccessCollections($options = []) {
 	 * @since 3.0
 	 */
 	public function getOwnedAccessCollection($subtype) {
-		if (!is_string($subtype)) {
+		if (!is_string($subtype) || $subtype === '') {
 			return false;
 		}
 
@@ -1341,11 +1341,11 @@ protected function create() {
 		$container_guid = (int) $container_guid;
 
 		if ($access_id == ACCESS_DEFAULT) {
-			throw new \InvalidParameterException('ACCESS_DEFAULT is not a valid access level. See its documentation in elgglib.php');
+			throw new \InvalidParameterException('ACCESS_DEFAULT is not a valid access level. See its documentation in constants.php');
 		}
 
 		if ($access_id == ACCESS_FRIENDS) {
-			throw new \InvalidParameterException('ACCESS_FRIENDS is not a valid access level. See its documentation in elgglib.php');
+			throw new \InvalidParameterException('ACCESS_FRIENDS is not a valid access level. See its documentation in constants.php');
 		}
 
 		$user_guid = elgg_get_logged_in_user_guid();
@@ -1477,11 +1477,11 @@ protected function update() {
 		$time = $this->getCurrentTime()->getTimestamp();
 
 		if ($access_id == ACCESS_DEFAULT) {
-			throw new \InvalidParameterException('ACCESS_DEFAULT is not a valid access level. See its documentation in elgglib.php');
+			throw new \InvalidParameterException('ACCESS_DEFAULT is not a valid access level. See its documentation in constants.php');
 		}
 
 		if ($access_id == ACCESS_FRIENDS) {
-			throw new \InvalidParameterException('ACCESS_FRIENDS is not a valid access level. See its documentation in elgglib.php');
+			throw new \InvalidParameterException('ACCESS_FRIENDS is not a valid access level. See its documentation in constants.php');
 		}
 
 		// Update primary table

diff --git a/engine/lib/access.php b/engine/lib/access.php
@@ -499,6 +499,10 @@ function access_friends_acl_remove_friend(\Elgg\Event $event) {
  * @param \Elgg\Hook $hook hook
  *
  * @return string|void
+ *
+ * @since 3.0.0
+ *
+ * @internal
  */
 function access_friends_acl_get_name(\Elgg\Hook $hook) {
 	$access_collection = $hook->getParam('access_collection');