Refactor search dynamic text

Signed-off-by: Fabio Teixeira <fabio1079@gmail.com> Signed-off-by: Luciano Prestes <lucianopcbr@gmail.com>

Refactor search dynamic text
Signed-off-by: Fabio Teixeira <fabio1079@gmail.com> Signed-off-by: Luciano Prestes <lucianopcbr@gmail.com>
Luciano Prestes · Gabriela Navarro
1 parent 8d0bfa0c
Showing 6 changed files with 59 additions and 10 deletions Show diff stats
features/software_catalog.feature
lib/ext/community.rb
lib/ext/search_controller.rb
lib/software_info.rb
public/software-catalog.js
views/search/_mpog_search_form.html.erb
@@ -37,7 +37,7 @@ Feature: Search software
   @selenium
   Scenario: Show software "One" when searching for "Software One"
     Given I go to /search/software_infos
-    And I fill in "search-input" with "Software One"
+    And I fill in "search-input" with "One"
     And I keyup on selector "#search-input"
     Then I should see "Software One"
     Then I should not see "Software Two"
@@ -2,6 +2,12 @@ require_dependency &#39;community&#39;
 class Community
+  SEARCHABLE_SOFTWARE_FIELDS = {
+    :name => 1,
+    :identifier => 2,
+    :nickname => 3,
+  }
+
   attr_accessible :visible
   has_one :software_info, :dependent=>:destroy
@@ -26,6 +26,7 @@ class SearchController
     results = results.paginate(:per_page => @per_page, :page => params[:page])
     @searches[@asset] = {:results => results}
     @search = results
+    @software_count = filter_software_infos_list.count
     render :layout=>false if request.xhr?
   end
@@ -65,7 +66,8 @@ class SearchController
   end
   def get_filtered_software_list
-    filtered_software_list = SoftwareInfo.like_search(params[:query])
+    params[:query] ||= ""
+    filtered_software_list = SoftwareInfo.search_by_query(params[:query])
     category_ids = get_filter_category_ids
 class SoftwareInfo < ActiveRecord::Base
   acts_as_having_settings :field => :settings
+  SEARCHABLE_SOFTWARE_FIELDS = {
+    :acronym => 1,
+    :finality => 2,
+  }
+
+  scope :search_by_query, lambda {|query = ""|
+    filtered_query = query.gsub(/[\|\(\)\\\/\s\[\]'"*%&!:]/,' ').split.map{|w| w += ":*"}.join('|')
+    search_fields = SoftwareInfo.pg_search_plugin_fields
+
+    if query.empty?
+      SoftwareInfo.all
+    else
+      joins(:community).where("to_tsvector('simple', #{search_fields}) @@ to_tsquery('#{filtered_query}')")
+    end
+  }
+
+  def self.pg_search_plugin_fields
+    searchable_fields = Community::SEARCHABLE_SOFTWARE_FIELDS.keys.map(&:to_s).sort.map {|f| "coalesce(#{Community.table_name}.#{f}, '')"}.join(" || ' ' || ")
+    searchable_fields += " || ' ' || "
+    searchable_fields += self::SEARCHABLE_SOFTWARE_FIELDS.keys.map(&:to_s).sort.map {|f| "coalesce(#{table_name}.#{f}, '')"}.join(" || ' ' || ")
+
+    searchable_fields
+  end
+
   SEARCH_FILTERS = []
   SEARCH_DISPLAYS = %w[full]
@@ -41,7 +65,7 @@ class SoftwareInfo &lt; ActiveRecord::Base
   # used on find_by_contents
   scope :like_search, lambda{ |name|
     joins(:community).where(
-      "name ILIKE ? OR acronym ILIKE ? OR finality ILIKE ?", 
+      "name ILIKE ? OR acronym ILIKE ? OR finality ILIKE ?",
       "%#{name}%", "%#{name}%", "%#{name}%"
     )
   }
@@ -95,20 +95,27 @@
   function update_search_page_on_ajax(response) {
-    close_loading();
     response = $(response);
     var search_list = $("#search-results");
     var selected_categories_field = $("#filter-categories-select-catalog");
     var pagination = $("#software-pagination");
+    var software_count = $("#software-count");
     var result_list = response.find("#search-results").html();
     var result_categories = response.find("#filter-categories-select-catalog").html();
     var result_pagination = response.find("#software-pagination").html();
+    var result_software_count = response.find("#software-count").html();
     search_list.html(result_list);
     selected_categories_field.html(result_categories);
     pagination.html(result_pagination);
+    software_count.html(result_software_count);
     show_head_message();
+
+    setTimeout(function(){
+      console.log("fgdjfgdsh");
+      close_loading();
+    }, 1000);
   }
@@ -127,10 +134,17 @@
   function update_page_by_text_filter() {
     var text = this.value;
+    dispatch_search_ajax(update_search_page_on_ajax, false);
+  }
-    if (text.length >= 3) {
-      dispatch_search_ajax(update_search_page_on_ajax, false);
-    }
+  function search_input_keyup() {
+    var timer = null;
+
+    $("#search-input").keyup(function() {
+        timer = setTimeout(update_page_by_text_filter, 400);
+    }).keydown(function() {
+        clearTimeout(timer);
+    });
   }
   function set_events() {
@@ -142,7 +156,8 @@
     $(".project-software").click(selectProjectSoftwareCheckbox);
     $("#software_display").change(update_page_by_ajax_on_select_change);
     $("#sort").change(update_page_by_ajax_on_select_change);
-    $("#search-input").keyup(update_page_by_text_filter);
+
+    search_input_keyup();
   }
@@ -11,14 +11,16 @@
       <span class="formfield">
         <%= text_field_tag 'query', @query, :id => 'search-input', :size => 50, :placeholder=>_("Type words about the software_info you're looking for") %>
       </span>
+
+      <%= submit_button(:search, _('Filter')) %>
     </div>
     <%= render :partial => 'search_form_extra_fields' %>
     <%= render :partial => 'catalog_filter' %>
     <!-- #display-options sera substituido pelo html passado pela equipe de design -->
     <div id="display-options" class="">
-      <div>
-        <strong><%= SoftwareInfo.count %> Softwares</strong>
+      <div id="software-count">
+        <strong><%= "#{@software_count} Software(s)" %> </strong>
       </div>
       <div>
	@@ -37,7 +37,7 @@ Feature: Search software		@@ -37,7 +37,7 @@ Feature: Search software
37	@selenium	37	@selenium
38	Scenario: Show software "One" when searching for "Software One"	38	Scenario: Show software "One" when searching for "Software One"
39	Given I go to /search/software_infos	39	Given I go to /search/software_infos
40	- And I fill in "search-input" with "Software One"	40	+ And I fill in "search-input" with "One"
41	And I keyup on selector "#search-input"	41	And I keyup on selector "#search-input"
42	Then I should see "Software One"	42	Then I should see "Software One"
43	Then I should not see "Software Two"	43	Then I should not see "Software Two"
	@@ -2,6 +2,12 @@ require_dependency 'community'		@@ -2,6 +2,12 @@ require_dependency 'community'
2		2
3	class Community	3	class Community
4		4
		5	+ SEARCHABLE_SOFTWARE_FIELDS = {
		6	+ :name => 1,
		7	+ :identifier => 2,
		8	+ :nickname => 3,
		9	+ }
		10	+
5	attr_accessible :visible	11	attr_accessible :visible
6		12
7	has_one :software_info, :dependent=>:destroy	13	has_one :software_info, :dependent=>:destroy
1	class SoftwareInfo < ActiveRecord::Base	1	class SoftwareInfo < ActiveRecord::Base
2	acts_as_having_settings :field => :settings	2	acts_as_having_settings :field => :settings
3		3
		4	+ SEARCHABLE_SOFTWARE_FIELDS = {
		5	+ :acronym => 1,
		6	+ :finality => 2,
		7	+ }
		8	+
		9	+ scope :search_by_query, lambda {\|query = ""\|
		10	+ filtered_query = query.gsub(/[\\|\(\)\\\/\s\[\]'"%&!:]/,' ').split.map{\|w\| w += ":"}.join('\|')
		11	+ search_fields = SoftwareInfo.pg_search_plugin_fields
		12	+
		13	+ if query.empty?
		14	+ SoftwareInfo.all
		15	+ else
		16	+ joins(:community).where("to_tsvector('simple', #{search_fields}) @@ to_tsquery('#{filtered_query}')")
		17	+ end
		18	+ }
		19	+
		20	+ def self.pg_search_plugin_fields
		21	+ searchable_fields = Community::SEARCHABLE_SOFTWARE_FIELDS.keys.map(&:to_s).sort.map {\|f\| "coalesce(#{Community.table_name}.#{f}, '')"}.join(" \|\| ' ' \|\| ")
		22	+ searchable_fields += " \|\| ' ' \|\| "
		23	+ searchable_fields += self::SEARCHABLE_SOFTWARE_FIELDS.keys.map(&:to_s).sort.map {\|f\| "coalesce(#{table_name}.#{f}, '')"}.join(" \|\| ' ' \|\| ")
		24	+
		25	+ searchable_fields
		26	+ end
		27	+
4	SEARCH_FILTERS = []	28	SEARCH_FILTERS = []
5	SEARCH_DISPLAYS = %w[full]	29	SEARCH_DISPLAYS = %w[full]
6		30
	@@ -41,7 +65,7 @@ class SoftwareInfo < ActiveRecord::Base		@@ -41,7 +65,7 @@ class SoftwareInfo < ActiveRecord::Base
41	# used on find_by_contents	65	# used on find_by_contents
42	scope :like_search, lambda{ \|name\|	66	scope :like_search, lambda{ \|name\|
43	joins(:community).where(	67	joins(:community).where(
44	- "name ILIKE ? OR acronym ILIKE ? OR finality ILIKE ?",	68	+ "name ILIKE ? OR acronym ILIKE ? OR finality ILIKE ?",
45	"%#{name}%", "%#{name}%", "%#{name}%"	69	"%#{name}%", "%#{name}%", "%#{name}%"
46	)	70	)
47	}	71	}