Merge pull request #5 from dom1nga/find_between_users_refactoring

Removed duplicated calling to discussion.users.

1 files changed, 1 insertions, 1 deletions

diff --git a/app/models/discussion.rb b/app/models/discussion.rb
index d6b7731..a4ad82b 100644
--- a/app/models/discussion.rb
+++ b/app/models/discussion.rb
@@ -67,7 +67,7 @@ class Discussion < ActiveRecord::Base
     discussions = self.joins(:speakers).includes(:users).where("speakers.user_id = ?", user.id)
     Rails.logger.info "Searching for ids: #{user.id}, #{user2.id}"
     discussions.each do |discussion|
-      dialog = discussion if discussion.private? && ((discussion.users.first == user && discussion.users.last == user2) || (discussion.users.first == user2 && discussion.users.last == user))
+      dialog = discussion if discussion.private? && ([discussion.users.first, discussion.users.last] - [user, user2]).empty?
     end
     dialog
   end