From f51b6082b49021f41faff079410fdb463a2bc478 Mon Sep 17 00:00:00 2001
From: Aurelio A. Heckert <aurium@gmail.com>
Date: Mon, 4 Oct 2010 15:23:54 -0300
Subject: [PATCH] Order tags alphabetically

---
 app/helpers/tags_helper.rb           |  3 +--
 public/designs/themes/base/style.css |  1 +
 test/unit/tags_block_test.rb         |  4 ++++
 test/unit/tags_helper_test.rb        | 21 +++++++++++++++++++++
 4 files changed, 27 insertions(+), 2 deletions(-)
 create mode 100644 test/unit/tags_helper_test.rb

diff --git a/app/helpers/tags_helper.rb b/app/helpers/tags_helper.rb
index fef502a..f55ac0d 100644
--- a/app/helpers/tags_helper.rb
+++ b/app/helpers/tags_helper.rb
@@ -28,7 +28,6 @@ module TagsHelper
   # courtesy of Aurelio: http://www.colivre.coop.br/Aurium/Nuvem 
   # (pt_BR only).
   def tag_cloud(tags, tagname_option, url, options = {})
-  
 
     return content_tag('em', _('No tags yet.')) +
            ' <a href="' + _('http://en.wikipedia.org/wiki/Tag_%28metadata%29') +
@@ -42,7 +41,7 @@ module TagsHelper
     max = tags.values.max.to_f
     min = tags.values.min.to_f
 
-    tags.map do |tag,count|
+    tags.sort_by{ |k,v| k.downcase }.map do |tag,count|
       if ( max == min )
         v = 0.5
       else
diff --git a/public/designs/themes/base/style.css b/public/designs/themes/base/style.css
index 90ec5b1..0e45aa9 100644
--- a/public/designs/themes/base/style.css
+++ b/public/designs/themes/base/style.css
@@ -666,6 +666,7 @@ div#notice {
 
 .tag_cloud a {
   text-decoration: none;
+  white-space: nowrap;
 }
 
 .tag_cloud a small {
diff --git a/test/unit/tags_block_test.rb b/test/unit/tags_block_test.rb
index 5b5863d..1bd1ae2 100644
--- a/test/unit/tags_block_test.rb
+++ b/test/unit/tags_block_test.rb
@@ -37,4 +37,8 @@ class TagsBlockTest < Test::Unit::TestCase
     assert_match /profile\/testinguser\/tags\/first-tag/, block.content
   end
 
+  should 'order tags alphabetically' do
+    assert /\/first-tag".*\/second-tag".*\/third-tag"/m =~  block.content
+  end
+
 end
diff --git a/test/unit/tags_helper_test.rb b/test/unit/tags_helper_test.rb
new file mode 100644
index 0000000..da5f0ae
--- /dev/null
+++ b/test/unit/tags_helper_test.rb
@@ -0,0 +1,21 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class TagsHelperTest < Test::Unit::TestCase
+
+  include ApplicationHelper
+  include TagsHelper
+  include ActionController::UrlWriter
+
+  def h(s); s; end
+  def link_to(text, *args); text; end
+
+  should 'order tags alphabetically' do
+    result = tag_cloud(
+      { 'tag1'=>9, 'Tag3'=>2, 'Tag2'=>2, 'aTag'=>2, 'beTag'=>2 },
+      :id,
+      { :host=>'noosfero.org', :controller=>'test', :action=>'tag' }
+    )
+    assert_equal %w(aTag beTag tag1 Tag2 Tag3).join("\n"), result
+  end
+
+end
--
libgit2 0.21.2