From 045878de64b9f609f241a8e710ef6f437f7f25e3 Mon Sep 17 00:00:00 2001
From: Raphael <raphael@joindiaspora.com>
Date: Wed, 18 Aug 2010 10:12:22 -0700
Subject: [PATCH] call ensure_bson once in visible_person_by_id and group_by_id

---
 app/models/user.rb | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/app/models/user.rb b/app/models/user.rb
index 0cebe47481..aa2973acb0 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -282,12 +282,14 @@ class User
   end
   
   def visible_person_by_id( id )
-    return self.person if ensure_bson(id) == self.person.id
-    friends.detect{|x| x.id == ensure_bson( id ) }
+    id = ensure_bson id
+    return self.person if id == self.person.id
+    friends.detect{|x| x.id == id }
   end
 
   def group_by_id( id )
-    groups.detect{|x| x.id == ensure_bson( id ) }
+    id = ensure_bson id
+    groups.detect{|x| x.id == id }
   end
 
   def groups_with_person person
-- 
GitLab