Hi Milo,
I checked it. It is a side effect of the wall gallery being first public and then private.
Here is what is happening:-

1. When your default privacy is public and you upload from activity a gallery is created with current default privacy( ‘public’) and set as wall gallery. We stored the id of this gallery in user meta to make it easier for future access.

Now, when you uncheck public, any public gallery will not be visible/available in the front end. Still when you upload, the script is just using that gallery id and moving the media to that public gallery.

Since the gallery is not visible publicly, the single link does not work from activity.

It is certainly a bug. Now, we have 2 ways to handle it:-

1.When uploading from activity, check if a wall gallery exists? If the status of wall gallery is not same as current default status, change the wall gallery status to current default status and move media to this gallery with the current status. Any old media inside the gallery will sustain their old status.

2. Or we check for the current status, if the wall gallery status is not same as the current status, we create a new gallery with current status and set it as wall gallery and move media here.

Which strategy is better? I am inclined towards the first one. Please let me know and I will have it in today’s update.

Thank you.
I will push it tonight Indian time with 2 other changes.

Hi Aleksandar,
It is definitely the “JPEG Rotation and EXIF Orientation” issue. We haven’t put any mechanism to check for that and relay on default WordPress implementation( which suffers from same if I remember correctly).

I will put a fix for the rotation by in tomorrow’s update.

Thank you
Brajesh

Hi Jay,
Thank you.
Please find the code below.

you can put the code in your bp-custom.php or your theme’s functions.php (I will prefer bp-custom.php )

/**
 * Make a site Private, works with/Withouthg BuddyPress
 * 
 * @author sbrajesh
 * @global string $pagenow
 * 
 */
function buddydev_private_site() {
	
	//first exclude the wp-login.php
	//register
	//activate
	global $pagenow;
	
	//do not restrict logged in users
	if( is_user_logged_in() ) {
		return ;
	}
	//if we are here, the user is not logged in, so let us check for exclusion
	//we selectively exclude pages from the list
	
	//are we on login page?
	if( $pagenow == 'wp-login.php' ) {
		return ;
	}
	
	
	$exclude_pages = array( 'register', 'activate', 'alphabeta' );//add the slugs here
	
	//is it one of the excluded pages, if yes, we just return and don't care
	if( is_page( $exclude_pages ) ) {
		return ;
	}
	
	$redirect_url = wp_login_url( site_url('/') );//get login url,
	
	wp_safe_redirect( $redirect_url );
	exit( 0 );
}

add_action( 'template_redirect', 'buddydev_private_site', 0 );

You can exclude any page by using the slug, It will work irrespective of the fact that BuddyPress may or may not be active.

Please do check and let me know if that worked for you or not?

Hi Milo,
Now I get the idea. You are referring to the top level comments made on the media from single media page right? They are infact individual items and not related(Only for the top level comments), any child comment on that will be threaded.

The best option I see here is when listing top level comment, we should include that media in the content of the activity. Will that approach be good?

By Privacy, you mean activity privacy and not the gallery privacy right?