You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by eh...@apache.org on 2007/04/13 02:36:31 UTC

svn commit: r528295 - /lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb

Author: ehatcher
Date: Thu Apr 12 17:36:31 2007
New Revision: 528295

URL: http://svn.apache.org/viewvc?view=rev&rev=528295
Log:
Ignore blank queries instead of adding a bum empty one, and change redirect not be hard-coded

Modified:
    lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb

Modified: lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb?view=diff&rev=528295&r1=528294&r2=528295
==============================================================================
--- lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb (original)
+++ lucene/solr/trunk/client/ruby/flare/vendor/plugins/flare/lib/flare/controller_extensions.rb Thu Apr 12 17:36:31 2007
@@ -51,8 +51,11 @@
 
 
       def add_query
-        @flare.queries << {:query => params[:search][:query]}
-        @flare.page = 1
+        query = params[:search][:query].strip
+        if query.size > 0
+          @flare.queries << {:query => query}
+          @flare.page = 1
+        end
         redirect_to :action => 'index'
       end
 
@@ -61,7 +64,7 @@
         @flare.queries[params[:index].to_i][:query] = params[:value]
         @flare.page = 1 # TODO: let the context adjust this automatically when its state changes
         render :update do |page|
-          page.redirect_to '/browse'
+          page.redirect_to :action => 'index'
         end
       end