ActionItem410: putting back people directory

git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@1895 3f533792-8f58-4932-b0fe-aaf55b0a4547

ActionItem410: putting back people directory
git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@1895 3f533792-8f58-4932-b0fe-aaf55b0a4547
AntonioTerceiro
1 parent 141aa503
Showing 6 changed files with 47 additions and 19 deletions Show diff stats
app/controllers/public/search_controller.rb
app/helpers/assets_helper.rb
app/views/search/_directory.rhtml
app/views/search/_display_results.rhtml
app/views/search/people.rhtml
test/functional/search_controller_test.rb
@@ -86,11 +86,18 @@ class SearchController &lt; ApplicationController
     if @results.keys.size == 1
       specific_action = @results.keys.first
-      send(specific_action)
-      render :action => specific_action
+      if respond_to?(specific_action)
+        send(specific_action)
+        render :action => specific_action
+        return
+      end
     end
+
+    render :action => 'index'
   end
+  alias :assets :index
+
   def events
     @events = @results[:events]
     @calendar = Event.date_range(params[:year], params[:month]).map do |date|
@@ -118,6 +125,10 @@ class SearchController &lt; ApplicationController
   end
+  def people
+    #nothing, just to enable
+  end
+
   #######################################################
   # view the summary of one category
@@ -139,13 +150,6 @@ class SearchController &lt; ApplicationController
   end
   attr_reader :category
-  #def assets
-    #@results = { @asset => @finder.recent(@asset, LIST_LIMIT) }
-
-    #@asset_name = gettext(SEARCH_IN.find { |entry| entry.first == @asset }[1])
-    #@names = { @asset => @asset_name }
-  #end
-
   def directory
     @results = { @asset => @finder.find_by_initial(@asset, params[:initial]) }
@@ -153,7 +157,7 @@ class SearchController &lt; ApplicationController
     @asset_name = gettext(SEARCH_IN.find { |entry| entry.first == @asset }[1])
     @names = { @asset => @asset_name }
-    render :action => 'assets'
+    render :action => @asset
   end
   def tags
@@ -2,7 +2,7 @@ module AssetsHelper
   def generate_assets_menu()
-    options = { :controller => 'search', :action => 'assets', :category_path => (@category ? @category.explode_path : []) }
+    options = { :controller => 'search', :action => 'index', :category_path => (@category ? @category.explode_path : []) }
     [
       [ options.merge(:asset => 'articles'), "icon-menu-articles",   _('Articles') ],
       [ options.merge(:asset => 'people'), "icon-menu-people",     _('People') ],
@@ -0,0 +1,6 @@
+<div style='text-align: center'>
+  <%= link_to_unless_current(_('Recent'), :action => 'index') %>
+  &nbsp;
+  <%= (?a..?z).map { |initial| link_to_unless_current(('' << initial).upcase, :action => 'directory', :initial => ('' << initial)) }.join(' &nbsp; ') %>
+</div>
+<br style='clear:both'/>
@@ -14,7 +14,9 @@
       <% if @controller.action_name != 'assets' %>
         <h3>
           <%= @names[name] %>
-          <%= link_to _('(see more ...)'), params.merge(:action => 'index', :find_in => [ name ]) %>
+          <% if @results.size != 1 %>
+            <%= link_to _('(see more ...)'), params.merge(:action => 'index', :find_in => [ name ]) %>
+          <% end %>
         </h3>
       <% end %>
       <% partial = partial_for_class results.first.class %>
@@ -24,7 +26,7 @@
           <%= render :partial => partial, :object => hit %>
         <% end %>
         </ul>
-        <hr />
+        <br />
       </div><!-- end class="search-results-innerbox" -->
     </div><!-- end class="search-results-<%= name %>" -->
@@ -0,0 +1,18 @@
+<% if @query.blank? %>
+  <h1><%= _('People') %></h1>
+  <%= render :partial => 'directory' %>
+<% else %>
+  <h1> <%= h(@category ? (_('Search results for "%{query}" in "%{category}"') % { :query => @query, :category => @category.name}) : (_('Search results for "%s"') % @query)) %> </h1>
+  <div style='text-align: center'>
+    <%= link_to _('Browse by name'), :action => 'index', :asset => 'people' %>
+  </div>
+<% end %>
+
+
+<div style='text-align: center'>
+  <%= render :partial => 'search_form', :locals => { :form_title => @query.blank? ? _('Search') : _("Refine your search"), :simple_search => true } %>
+</div>
+
+<%= render :partial => 'display_results' %>
+
+<br style="clear:both" />
@@ -515,11 +515,6 @@ class SearchControllerTest &lt; Test::Unit::TestCase
     assert_response 403
   end
-  should 'expose asset name in instance variable' do
-    get :assets, :asset => 'products'
-    assert_equal 'Products', assigns(:asset_name)
-  end
-
   should 'not use design blocks' do
     get :index
     assert_no_tag :tag => 'div', :attributes => { :id => 'boxes', :class => 'boxes' }
@@ -907,7 +902,10 @@ class SearchControllerTest &lt; Test::Unit::TestCase
     end
   end
-  should 'test somehow the display of events as calendar'
+  should 'provide calendar for events' do
+    get :index, :find_in => [ 'events' ]
+    assert_equal 0, assigns(:calendar).size % 7
+  end
   ##################################################################
   ##################################################################
	@@ -2,7 +2,7 @@ module AssetsHelper		@@ -2,7 +2,7 @@ module AssetsHelper
2		2
3	def generate_assets_menu()	3	def generate_assets_menu()
4		4
5	- options = { :controller => 'search', :action => 'assets', :category_path => (@category ? @category.explode_path : []) }	5	+ options = { :controller => 'search', :action => 'index', :category_path => (@category ? @category.explode_path : []) }
6	[	6	[
7	[ options.merge(:asset => 'articles'), "icon-menu-articles", _('Articles') ],	7	[ options.merge(:asset => 'articles'), "icon-menu-articles", _('Articles') ],
8	[ options.merge(:asset => 'people'), "icon-menu-people", _('People') ],	8	[ options.merge(:asset => 'people'), "icon-menu-people", _('People') ],
@@ -0,0 +1,6 @@		@@ -0,0 +1,6 @@
	1	+<div style='text-align: center'>
	2	+ <%= link_to_unless_current(_('Recent'), :action => 'index') %>
	3	+
	4	+ <%= (?a..?z).map { \|initial\| link_to_unless_current(('' << initial).upcase, :action => 'directory', :initial => ('' << initial)) }.join('   ') %>
	5	+</div>
	6	+<br style='clear:both'/>
@@ -0,0 +1,18 @@		@@ -0,0 +1,18 @@
	1	+<% if @query.blank? %>
	2	+ <h1><%= _('People') %></h1>
	3	+ <%= render :partial => 'directory' %>
	4	+<% else %>
	5	+ <h1> <%= h(@category ? (_('Search results for "%{query}" in "%{category}"') % { :query => @query, :category => @category.name}) : (_('Search results for "%s"') % @query)) %> </h1>
	6	+ <div style='text-align: center'>
	7	+ <%= link_to _('Browse by name'), :action => 'index', :asset => 'people' %>
	8	+ </div>
	9	+<% end %>
	10	+
	11	+
	12	+<div style='text-align: center'>
	13	+ <%= render :partial => 'search_form', :locals => { :form_title => @query.blank? ? _('Search') : _("Refine your search"), :simple_search => true } %>
	14	+</div>
	15	+
	16	+<%= render :partial => 'display_results' %>
	17	+
	18	+<br style="clear:both" />