updated shared access search to include email as well as name (#5681)

bigbluebutton · Jan 26, 2024 · a60752d · a60752d
1 parent b65912b
commit a60752d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/controllers/api/v1/shared_accesses_controller.rb b/app/controllers/api/v1/shared_accesses_controller.rb
@@ -76,7 +76,7 @@ def shareable_users
                               .with_provider(current_provider)
                               .where.not(id: [@room.shared_users.pluck(:id) << @room.user_id])
                               .where(role_id: [role_ids])
-                              .name_search(params[:search])
+                              .shared_access_search(params[:search])
         render_data data: shareable_users, serializer: SharedAccessSerializer, status: :ok
       end
 

diff --git a/app/models/user.rb b/app/models/user.rb
@@ -81,8 +81,8 @@ def self.search(input)
     all
   end
 
-  def self.name_search(input)
-    return where('users.name ILIKE :input', input: "%#{input}%") if input
+  def self.shared_access_search(input)
+    return where('users.name ILIKE :input OR users.email ILIKE :input', input: "%#{input}%") if input
 
     all
   end