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 ry...@apache.org on 2007/12/08 08:35:14 UTC
svn commit: r602342 -
/lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java
Author: ryan
Date: Fri Dec 7 23:35:13 2007
New Revision: 602342
URL: http://svn.apache.org/viewvc?rev=602342&view=rev
Log:
SOLR-350, removing old option to set core from query params (Responding to Hoss' comments on SOLR-428 let me see this)
Modified:
lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java
Modified: lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java?rev=602342&r1=602341&r2=602342&view=diff
==============================================================================
--- lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java (original)
+++ lucene/solr/trunk/src/webapp/src/org/apache/solr/servlet/SolrDispatchFilter.java Fri Dec 7 23:35:13 2007
@@ -182,8 +182,8 @@
SolrCore core = singlecore;
if( core == null ) {
// try to get the corename as a request parameter first
- String corename = request.getParameter("core");
- if( corename == null && path.startsWith( "/@" ) ) { // multicore
+ String corename = null;
+ if( path.startsWith( "/@" ) ) { // multicore
idx = path.indexOf( '/', 2 );
if( idx < 1 ) {
throw new SolrException( SolrException.ErrorCode.BAD_REQUEST,
@@ -191,8 +191,7 @@
}
corename = path.substring( 2, idx );
path = path.substring( idx );
- }
- if (corename != null) {
+
core = multicore.getCore( corename );
if( core == null ) {
throw new SolrException( SolrException.ErrorCode.BAD_REQUEST,