You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by st...@apache.org on 2013/03/14 17:40:38 UTC
svn commit: r1456521 - in /lucene/dev/trunk/solr: CHANGES.txt
webapp/web/js/scripts/analysis.js
Author: steffkes
Date: Thu Mar 14 16:40:37 2013
New Revision: 1456521
URL: http://svn.apache.org/r1456521
Log:
SOLR-4371: Admin UI - Analysis Screen shows empty result
Modified:
lucene/dev/trunk/solr/CHANGES.txt
lucene/dev/trunk/solr/webapp/web/js/scripts/analysis.js
Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1456521&r1=1456520&r2=1456521&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Thu Mar 14 16:40:37 2013
@@ -87,6 +87,8 @@ Bug Fixes
* SOLR-4543: setting shardHandlerFactory in solr.xml/solr.properties does not work.
(Ryan Ernst, Robert Muir via Erick Ericson)
+* SOLR-4371: Admin UI - Analysis Screen shows empty result (steffkes)
+
Other Changes
----------------------
Modified: lucene/dev/trunk/solr/webapp/web/js/scripts/analysis.js
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/webapp/web/js/scripts/analysis.js?rev=1456521&r1=1456520&r2=1456521&view=diff
==============================================================================
--- lucene/dev/trunk/solr/webapp/web/js/scripts/analysis.js (original)
+++ lucene/dev/trunk/solr/webapp/web/js/scripts/analysis.js Thu Mar 14 16:40:37 2013
@@ -247,9 +247,12 @@ sammy.get
'submit',
function( event )
{
- var params = compute_analysis_params();
+ var params = $.param( compute_analysis_params() )
+ .replace( /[\w\.]+=\+*(&)/g, '$1' ) // remove empty parameters
+ .replace( /(&)+/, '$1' ) // reduce multiple ampersands
+ .replace( /^&/, '' ); // remove leading ampersand
- context.redirect( context.path.split( '?' ).shift() + '?' + $.param( params ) );
+ context.redirect( context.path.split( '?' ).shift() + '?' + params );
return false;
}
)