SOLR-4045: SOLR admin page returns HTTP 404 on core names containing a '.' (dot)

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1429259 13f79535-47bb-0310-9956-ffa450edef68

solr/CHANGES.txt

@@ -445,6 +445,8 @@ Bug Fixes
 * SOLR-4257: PeerSync updates and Log Replay updates should not wait for
   a ZooKeeper connection in order to proceed. (yonik)
 
+* SOLR-4045: SOLR admin page returns HTTP 404 on core names containing
+  a '.' (dot) (steffkes)
 
 Other Changes
 ----------------------

solr/webapp/web/js/scripts/analysis.js

@@ -18,7 +18,7 @@
 // #/:core/analysis
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(analysis)$/,
+  new RegExp( app.core_regex_base + '\\/(analysis)$' ),
   function( context )
   {
     var active_core = this.active_core;

solr/webapp/web/js/scripts/app.js

@@ -100,7 +100,7 @@ var sammy = $.sammy
         {
           var selector = '~' === this.params.splat[0][0]
                        ? '#' + this.params.splat[0].replace( /^~/, '' ) + '.global'
-                       : '#menu-selector #' + this.params.splat[0];
+                       : '#menu-selector #' + this.params.splat[0].replace( /\./g, '__' );
 
           var active_element = $( selector, menu_wrapper );
                     
@@ -152,6 +152,8 @@ var solr_admin = function( app_config )
 
   this.timeout = null;
 
+  this.core_regex_base = '^#\\/([\\w\\d-\\.]+)';
+
   show_global_error = function( error )
   {
     var main = $( '#main' );
@@ -205,7 +207,7 @@ var solr_admin = function( app_config )
               classes.push( 'default' );
             }
 
-            var core_tpl = '<li id="' + core_name + '" '
+            var core_tpl = '<li id="' + core_name.replace( /\./g, '__' ) + '" '
                          + '    class="' + classes.join( ' ' ) + '"'
                          + '    data-basepath="' + core_path + '"'
                          + '    schema="' + schema + '"'

solr/webapp/web/js/scripts/dashboard.js

@@ -39,7 +39,7 @@ var set_healthcheck_status = function( status )
 // #/:core
 sammy.get
 (
-  /^#\/([\w\d-]+)$/,
+  new RegExp( app.core_regex_base + '$' ),
   function( context )
   {
     var core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/dataimport.js

@@ -101,7 +101,7 @@ sammy.bind
 // #/:core/dataimport
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(dataimport)$/,
+  new RegExp( app.core_regex_base + '\\/(dataimport)$' ),
   function( context )
   {
     sammy.trigger
@@ -129,7 +129,7 @@ sammy.get
 // #/:core/dataimport
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(dataimport)\//,
+  new RegExp( app.core_regex_base + '\\/(dataimport)\\/' ),
   function( context )
   {
     var core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/file.js

@@ -18,7 +18,7 @@
 // #/:core/schema, #/:core/config
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(schema|config)$/,
+  new RegExp( app.core_regex_base + '\\/(schema|config)$' ),
   function( context )
   {
     var core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/plugins.js

@@ -382,7 +382,7 @@ sammy.bind
 // #/:core/plugins/$type
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(plugins)\/(\w+)$/,
+  new RegExp( app.core_regex_base + '\\/(plugins)\\/(\\w+)$' ),
   function( context )
   {
     core_basepath = this.active_core.attr( 'data-basepath' );
@@ -405,7 +405,7 @@ sammy.get
 // #/:core/plugins
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(plugins)$/,
+  new RegExp( app.core_regex_base + '\\/(plugins)$' ),
   function( context )
   {
     core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/query.js

@@ -18,7 +18,7 @@
 // #/:core/query
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(query)$/,
+  new RegExp( app.core_regex_base + '\\/(query)$' ),
   function( context )
   {
     var core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/replication.js

@@ -490,7 +490,7 @@ var replication_fetch_status = function()
 // #/:core/replication
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(replication)$/,
+  new RegExp( app.core_regex_base + '\\/(replication)$' ),
   function( context )
   {
     core_basepath = this.active_core.attr( 'data-basepath' );

solr/webapp/web/js/scripts/schema-browser.js

@@ -678,7 +678,7 @@ sammy.bind
 // #/:core/schema-browser
 sammy.get
 (
-  /^#\/([\w\d-]+)\/(schema-browser)$/,
+  new RegExp( app.core_regex_base + '\\/(schema-browser)$' ),
   function( context )
   {
     var core_basepath = this.active_core.attr( 'data-basepath' );