From ec5047b7496834ad1a12afe7846b49b7cd94d0c2 Mon Sep 17 00:00:00 2001
From: Boone B Gorges <boonebgorges@gmail.com>
Date: Mon, 16 Dec 2024 10:52:04 -0600
Subject: [PATCH] Change definiton of 'is_open' filter on group directory.

Changed previously on group profiles in #3391.

See #3470.
---
 .../openlab/buddypress/groups/groups-loop.php | 14 +++---
 wp-content/themes/openlab/lib/group-funcs.php | 44 +++++++++++++++++++
 2 files changed, 51 insertions(+), 7 deletions(-)

diff --git a/wp-content/themes/openlab/buddypress/groups/groups-loop.php b/wp-content/themes/openlab/buddypress/groups/groups-loop.php
index d0c5a8af41..e0806079fb 100644
--- a/wp-content/themes/openlab/buddypress/groups/groups-loop.php
+++ b/wp-content/themes/openlab/buddypress/groups/groups-loop.php
@@ -99,13 +99,8 @@
 
 $is_open = openlab_get_current_filter( 'is_open' );
 if ( $is_open ) {
-	$meta_query['blog_public'] = array(
-		'key'      => 'blog_public',
-		'value'    => [ '1', '0' ],
-		'operator' => 'IN',
-	);
-
-	$group_args['status'] = 'public';
+	add_filter( 'bp_groups_get_paged_groups_sql', 'openlab_is_open_group_query_callback', 10, 3 );
+	add_filter( 'bp_groups_get_total_groups_sql', 'openlab_is_open_group_query_callback', 10, 3 );
 }
 
 $active_status = openlab_get_current_filter( 'active-status' );
@@ -327,6 +322,11 @@
 	remove_filter( 'bp_groups_get_paged_groups_sql', 'openlab_filter_groups_query_for_active_status', 10 );
 	remove_filter( 'bp_groups_get_total_groups_sql', 'openlab_filter_groups_query_for_active_status', 10 );
 }
+
+if ( $is_open ) {
+	remove_filter( 'bp_groups_get_paged_groups_sql', 'openlab_is_open_group_query_callback', 10, 3 );
+	remove_filter( 'bp_groups_get_total_groups_sql', 'openlab_is_open_group_query_callback', 10, 3 );
+}
 ?>
 
 <?php
diff --git a/wp-content/themes/openlab/lib/group-funcs.php b/wp-content/themes/openlab/lib/group-funcs.php
index ab5c572f0a..3153286484 100644
--- a/wp-content/themes/openlab/lib/group-funcs.php
+++ b/wp-content/themes/openlab/lib/group-funcs.php
@@ -418,6 +418,50 @@ function openlab_filter_groups_query_for_active_status( $sql, $sql_clauses, $r )
 	return $sql;
 }
 
+/**
+ * Implementation of 'is_open' parameter for groups_get_groups().
+ *
+ * @param string $groups_sql  The SQL query.
+ * @param array  $sql_clauses The SQL clauses.
+ * @param array  $r           The query parameters.
+ */
+function openlab_is_open_group_query_callback( $groups_sql, $sql_clauses, $r ) {
+	$is_open = openlab_get_current_filter( 'is_open' );
+	if ( ! $is_open ) {
+		return $groups_sql;
+	}
+
+	$original_where_clause = $sql_clauses['where'];
+
+	$where_clauses = explode( 'AND', $sql_clauses['where'] );
+
+	// Remove the existing g.status clause.
+	$where_clauses = array_filter(
+		$where_clauses,
+		function( $clause ) {
+			return 0 !== strpos( $clause, 'g.status' );
+		}
+	);
+
+	$where_clauses[] = "(g.status = 'public' OR gm_blog_public.meta_value = 1)";
+
+	$groups_sql = str_replace(
+		$original_where_clause,
+		implode( ' AND ', $where_clauses ),
+		$groups_sql
+	);
+
+	$bp = buddypress();
+
+	$groups_sql = str_replace(
+		$sql_clauses['from'],
+		$sql_clauses['from'] . " LEFT JOIN {$bp->groups->table_name_groupmeta} gm_blog_public ON (g.id = gm_blog_public.group_id AND gm_blog_public.meta_key = 'blog_public')",
+		$groups_sql
+	);
+
+	return $groups_sql;
+}
+
 /**
  * Unhooks group join button if it's disabled for the group.
  */