Context Navigation

Changes between Initial Version and Version 1 of Ticket #7048, comment 13

initial	v1
1		Looking at the new function, I'm not sure the name "permalink access" is correct -- it's not specifically dealing with the permalink. Perhaps something like `bp_activity_user_can_read()`? ~~That way~~ Maybe @boonebgorges has an idea.
	1	Looking at the new function, I'm not sure the name "permalink access" is correct -- it's not specifically dealing with the permalink. Perhaps something like `bp_activity_user_can_read()`? Maybe @boonebgorges has an idea.
2	2
3	3	What do you think about moving the "If activity is from a group, do an extra cap check" underneath "If activity author match user, allow access as well", and add a check - e.g. `if ( ! $retval && bp_is_active( 'groups' ) && $activity->component === $bp->groups->id ) {` -- I'm not sure if ` $retval = $group->user_has_access; ` could ever be false and thus prevent a `true` we explicitly set earlier, but this would avoid that.