bp-activity-classes.php: add a sql statements filter

[aesqe]

in file "activity/bp-activity-classes.php", line 123, could someone please add a filter for the SQL WHERE statements?

something like this:

$where_sql = 'WHERE ' . join( ' AND ', apply_filters("bp_activity_get_where_conditions", $where_conditions));

thanks :)

[paulhastings0]

...eh... too many people have been testing the current RC. I think we should punt this to 1.3.

[DJPaul]

Punting to 1.3 pending review

[aesqe]

might be better if the whole string could be filtered, with $where_conditions included as second argument:

$where_sql = apply_filters("bp_activity_get_where_conditions", 'WHERE ' . join(' AND ', $where_conditions), $where_conditions );

[DJPaul]

Fixed in [3617]

[aesqe]

Thanks DJPaul, but a filter on $where_sql, before all the DB queries are performed, would be of much more use to me :/ Any chance you could add another filter just for $where_sql? It's used both in fetching $activities and $total_activities_sql and it would save me from using preg_replace on the whole SQL statement.

I hope I'm not asking too much :)

[DJPaul]

These queries are typically built from associative arrays, which are passed to these filters. You can either preg_replace the main query, or re-construct the query from the entire array, customising as necessary. It's not appropriate for a specialised filter just for the 'where' part of the statement, because someone may turn around and ask for the same for the 'limit' part, and so on.

[aesqe]

Replying to DJPaul:

These queries are typically built from associative arrays, which are passed to these filters.

Maybe I'm looking at the wrong thing, but in the ​trunk version of bp-activity-classes.php, $where_sql is not passed to the 'bp_activity_get_user_join_filter':

$wpdb->get_results(
	apply_filters(
		'bp_activity_get_user_join_filter',
		$wpdb->prepare(
			"{$select_sql} {$from_sql} {$where_sql} ORDER BY a.date_recorded {$sort} {$pag_sql}", 
			$select_sql, $from_sql, $where_sql, $sort, $pag_sql
		)
	)
);

if it were passed to the filter, that's all I would need :)

anyhow, the code above looks kind of wrong, are you sure it's not supposed to be:

$wpdb->get_results(
	apply_filters(
		'bp_activity_get_user_join_filter',
		$wpdb->prepare(
			"%s %s %s ORDER BY a.date_recorded %s %s", 
			$select_sql, $from_sql, $where_sql, $sort, $pag_sql
		),
		$select_sql, $from_sql, $where_sql, $sort, $pag_sql
	)
);

or

$wpdb->get_results(
	apply_filters(
		'bp_activity_get_user_join_filter',
		"{$select_sql} {$from_sql} {$where_sql} ORDER BY a.date_recorded {$sort} {$pag_sql}", 
		$select_sql, $from_sql, $where_sql, $sort, $pag_sql
	)
);

(and I apologize in advance if I'm misreading something :|)