# HG changeset patch
# User Chris Cannam
# Date 1363102018 0
# Node ID d280360758e5d2dbb0d5962c5168b745053d04d2
# Parent  edf7563e26536d6753a087b8767772f5a8761809
Fix dodgy merge, syntax, defaults

diff -r edf7563e2653 -r d280360758e5 app/controllers/my_controller.rb
--- a/app/controllers/my_controller.rb	Tue Mar 12 15:14:07 2013 +0000
+++ b/app/controllers/my_controller.rb	Tue Mar 12 15:26:58 2013 +0000
@@ -38,7 +38,7 @@
            }.merge(Redmine::Views::MyPage::Block.additional_blocks).freeze
 
   DEFAULT_LAYOUT = {  'left' => ['myprojects', 'activitymyprojects'], 
-                      'right' => ['tipoftheday', 'colleagues', 'issueswatched'] 
+                      'right' => ['tipoftheday', 'issueswatched'] 
                    }.freeze
 
   verify :xhr => true,
diff -r edf7563e2653 -r d280360758e5 app/helpers/activities_helper.rb
--- a/app/helpers/activities_helper.rb	Tue Mar 12 15:14:07 2013 +0000
+++ b/app/helpers/activities_helper.rb	Tue Mar 12 15:26:58 2013 +0000
@@ -67,29 +67,37 @@
 
     start = Time.now
 
+    my_inst = ""
+    if ! User.current.ssamr_user_detail.nil?
+      my_inst = User.current.ssamr_user_detail.institution_name
+    end
+
+    s << "<dl>"
     for c in colleagues
       u = User.find_by_id(c)
       active_projects = projects_by_activity(u, 3)
       if !active_projects.empty?
-        s << "<dl>"
         s << "<dt>"
         s << avatar(u, :size => '24')
         s << "<span class='user'>"
         s << h(u.name)
         s << "</span>"
         if !u.ssamr_user_detail.nil?
-          s << " - <span class='institution'>"
-          s << h(u.ssamr_user_detail.institution_name)
-          s << "</span>"
+          inst = u.ssamr_user_detail.institution_name
+          if inst != "" and inst != my_inst
+            s << " - <span class='institution'>"
+            s << h(u.ssamr_user_detail.institution_name)
+            s << "</span>"
+          end
         end
         s << "</dt>"
         s << "<dd>"
         s << "<span class='active'>"
         s << (active_projects.map { |p| link_to_project(p) }.join ", ")
         s << "</span>"
-        s << "</dl>"
       end
     end
+    s << "</dl>"
 
     finish = Time.now
     logger.info "render_active_colleagues: took #{finish-start}"
diff -r edf7563e2653 -r d280360758e5 app/helpers/my_helper.rb
--- a/app/helpers/my_helper.rb	Tue Mar 12 15:14:07 2013 +0000
+++ b/app/helpers/my_helper.rb	Tue Mar 12 15:26:58 2013 +0000
@@ -25,50 +25,4 @@
     user.projects.select { |p| p.visible? }.map { |p| p.members.map { |m| m.user_id } }.flatten.sort.uniq.reject { |i| user.id == i }
   end
 
-  def render_active_colleagues(colleagues)
-
-    s = ""
-
-    start = Time.now
-
-    my_inst = ""
-    if ! User.current.ssamr_user_detail.nil?
-      my_inst = User.current.ssamr_user_detail.institution_name
-    end
-
-    for c in colleagues
-      u = User.find_by_id(c)
-      active_projects = projects_by_activity(u, 3)
-      if !active_projects.empty?
-        s << "<div class='active-person'>"
-        s << avatar(u, :size => '24')
-        s << "<span class='user'>"
-        s << h(u.name)
-        s << "</span>"
-        if !u.ssamr_user_detail.nil?
-          inst = u.ssamr_user_detail.institution_name
-          if inst != "" and inst != my_inst
-            s << " - <span class='institution'>"
-            s << h(inst)
-            s << "</span>"
-          end
-        end
-        s << "<br>"
-        s << "<span class='active'>"
-        s << (active_projects.map { |p| link_to_project(p) }.join ", ")
-        s << "</span>"
-        s << "</div>"
-      end
-    end
-
-    finish = Time.now
-    logger.info "render_active_colleagues: took #{finish-start}"
-    
-    if s != ""
-      s
-    else
-      l(:label_no_active_colleagues)
-    end
-  end
-
 end
diff -r edf7563e2653 -r d280360758e5 app/views/my/blocks/_colleagues.html.erb
--- a/app/views/my/blocks/_colleagues.html.erb	Tue Mar 12 15:14:07 2013 +0000
+++ b/app/views/my/blocks/_colleagues.html.erb	Tue Mar 12 15:26:58 2013 +0000
@@ -1,5 +1,5 @@
 
-<% colleagues = all_colleagues_of(@user).sample(10) %>
+<% colleagues = all_colleagues_of(@user) %>
 
 <% if !colleagues.empty? %>
 <div id="activity">