Unscoped bp_nouveau_ajax_querystring on single activity page is affecting other loops

[sbrajesh]

Hi,
The function bp_nouveau_ajax_querystring which filters on 'bp_ajax_querystring' has a part of code like this

	// Single activity.
	if ( bp_is_single_activity() ) {
		$qs = array(
			'display_comments=threaded',
			'show_hidden=true',
			'include=' . bp_current_action(),
		);
	}

Since it does not check the $object for activity type, It is affecting all other loops on the single activity page. I have a theme with sidebar for single activity page and the members loop is getting affected(include=activityid)

If we update the code and add test for object type, that will fix the issue. Please find the patch attached.

[imath]

Patch looks good! Thanks for your contribution 👌. We’ll have it fixed asap.

[sbrajesh]

Thank you.

[imath]

In 12947:

Nouveau: improve the single Activity screen Ajax querystring override

To avoid interfering with other objects loop on this screen, we need to make sure the Ajax querystring is about the activity object before overriding it.

Props sbrajesh

Fixes #8473