Display badges separated by the owner in dashboard

Victor Costa
1 parent 60b56e2f
Showing 3 changed files with 56 additions and 17 deletions Show diff stats
lib/gamification_plugin/dashboard_helper.rb
test/unit/dashboard_helper_test.rb
views/gamification/dashboard.html.erb
@@ -35,4 +35,16 @@ module GamificationPlugin::DashboardHelper
     (context_ranking.blank? ? '' : render(:partial => 'gamification/ranking', :locals => {:ranking => context_ranking, :target_ranking => target_ranking, :ranking_class => 'context'}))
   end
+  def badges_title(owner)
+    return _('Badges for %s' % owner.name) if owner.kind_of?(Organization)
+    _('Badges')
+  end
+
+  def grouped_badges
+    environment.gamification_plugin_badges.all.group_by(&:owner).sort do |a, b|
+      return -1 if a.first.kind_of?(Environment)
+      a.first.name <=> b.first.name
+    end
+  end
+
 end
@@ -0,0 +1,25 @@
+require_relative "../test_helper"
+
+class DashboardHelperTest < ActiveSupport::TestCase
+
+  include GamificationPlugin::DashboardHelper
+
+  should 'return title for global badges' do
+    owner = Environment.new
+    assert_equal 'Badges', badges_title(owner)
+  end
+
+  should 'return title for organization badges' do
+    owner = Organization.new(:name => 'organization')
+    assert_equal 'Badges for organization', badges_title(owner)
+  end
+
+  should 'return badges grouped by owner' do
+    environment = Environment.default
+    expects(:environment).at_least_once.returns(environment)
+    badge1 = GamificationPlugin::Badge.create!(:owner => fast_create(Organization))
+    badge2 = GamificationPlugin::Badge.create!(:owner => environment)
+    assert_equal [[badge2.owner, [badge2]], [badge1.owner, [badge1]]], grouped_badges
+  end
+
+end
@@ -34,24 +34,26 @@
   <% unless environment.gamification_plugin_badges.empty? %>
   <div class="badges">
-    <h3><%= _('Badges') %></h3>
-    <ul class="badge-list">
-      <% environment.gamification_plugin_badges.group(:name).count.each do |badge_name, amount| %>
-        <% person_badge = @target.badges.where(:name => badge_name).last %>
-        <% badge =  environment.gamification_plugin_badges.where(:name => badge_name).last %>
-        <li class="badge <%= badge.name %>">
-          <div class="badge" title="<%= badge.description %>">
-            <div class="image <%= badge.name %>"></div>
-            <ul class="level rating">
-              <% 1.upto(badge.level).map do |n|%>
-                <span class="star <%= (person_badge && person_badge.level >= n) ? 'earned' : 'not-earned' %>" >★</span>
-              <% end %>
-            </ul>
-            <div class="title"><%= badge.title %></div>
-          </div>
-        </li>
+      <% grouped_badges.each do |owner, badges| %>
+        <h3><%= badges_title owner %></h3>
+        <ul class="badge-list">
+          <% badges.group_by(&:name).each do |badge_name, badges_group| %>
+            <% badge = badges_group.sort_by(&:level).last %>
+            <% person_badge = @target.badges.where(:name => badge.name).last %>
+            <li class="badge <%= badge.name %>">
+              <div class="badge" title="<%= badge.description %>">
+                <div class="image <%= badge.name %>"></div>
+                <ul class="level rating">
+                  <% 1.upto(badge.level).map do |n|%>
+                    <span class="star <%= (person_badge && person_badge.level >= n) ? 'earned' : 'not-earned' %>" >★</span>
+                  <% end %>
+                </ul>
+                <div class="title"><%= badge.title %></div>
+              </div>
+            </li>
+          <% end %>
+        </ul>
       <% end %>
-    </ul>
   </div>
   <% end %>
 </div>
	@@ -35,4 +35,16 @@ module GamificationPlugin::DashboardHelper		@@ -35,4 +35,16 @@ module GamificationPlugin::DashboardHelper
35	(context_ranking.blank? ? '' : render(:partial => 'gamification/ranking', :locals => {:ranking => context_ranking, :target_ranking => target_ranking, :ranking_class => 'context'}))	35	(context_ranking.blank? ? '' : render(:partial => 'gamification/ranking', :locals => {:ranking => context_ranking, :target_ranking => target_ranking, :ranking_class => 'context'}))
36	end	36	end
37		37
		38	+ def badges_title(owner)
		39	+ return _('Badges for %s' % owner.name) if owner.kind_of?(Organization)
		40	+ _('Badges')
		41	+ end
		42	+
		43	+ def grouped_badges
		44	+ environment.gamification_plugin_badges.all.group_by(&:owner).sort do \|a, b\|
		45	+ return -1 if a.first.kind_of?(Environment)
		46	+ a.first.name <=> b.first.name
		47	+ end
		48	+ end
		49	+
38	end	50	end
@@ -0,0 +1,25 @@		@@ -0,0 +1,25 @@
	1	+require_relative "../test_helper"
	2	+
	3	+class DashboardHelperTest < ActiveSupport::TestCase
	4	+
	5	+ include GamificationPlugin::DashboardHelper
	6	+
	7	+ should 'return title for global badges' do
	8	+ owner = Environment.new
	9	+ assert_equal 'Badges', badges_title(owner)
	10	+ end
	11	+
	12	+ should 'return title for organization badges' do
	13	+ owner = Organization.new(:name => 'organization')
	14	+ assert_equal 'Badges for organization', badges_title(owner)
	15	+ end
	16	+
	17	+ should 'return badges grouped by owner' do
	18	+ environment = Environment.default
	19	+ expects(:environment).at_least_once.returns(environment)
	20	+ badge1 = GamificationPlugin::Badge.create!(:owner => fast_create(Organization))
	21	+ badge2 = GamificationPlugin::Badge.create!(:owner => environment)
	22	+ assert_equal [[badge2.owner, [badge2]], [badge1.owner, [badge1]]], grouped_badges
	23	+ end
	24	+
	25	+end