SOLR-4405: Admin UI - admin-extra files are not rendered into the core-menu

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

solr/CHANGES.txt

@@ -99,6 +99,8 @@ Bug Fixes
 * SOLR-4636: If opening a reader fails for some reason when opening a SolrIndexSearcher,
   a Directory can be left unreleased. (Mark Miller)
 
+* SOLR-4405: Admin UI - admin-extra files are not rendered into the core-menu (steffkes)
+
 Optimizations
 ----------------------
 

solr/example/solr/collection1/conf/admin-extra.html

@@ -15,17 +15,10 @@
  limitations under the License.
 -->
 
-<!-- The content of this page will be statically included into the top
+<!-- The content of this page will be statically included into the top-
-of the admin page.  Uncomment this as an example to see there the content
+right box of the cores overview page. Uncomment this as an example to 
-will show up.
+see there the content will show up.
-
-<hr>
-<i>This line will appear before the first table</i>
-<tr>
-<td colspan="2">
-This row will be appended to the end of the first table
-</td>
-</tr>
-<hr>
 
+<img src="img/ico/construction.png"> This line will appear at the top-
+right box on collection1's Overview
 -->

solr/example/solr/collection1/conf/admin-extra.menu-bottom.html

@@ -1 +1,25 @@
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements.  See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License.  You may obtain a copy of the License at
+
+     http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+
 <!-- admin-extra.menu-bottom.html -->
+<!--
+<li>
+  <a href="#" style="background-image: url(img/ico/construction.png);">
+    LAST ITEM
+  </a>
+</li>
+-->

solr/example/solr/collection1/conf/admin-extra.menu-top.html

@@ -1 +1,25 @@
+<!--
+ Licensed to the Apache Software Foundation (ASF) under one or more
+ contributor license agreements.  See the NOTICE file distributed with
+ this work for additional information regarding copyright ownership.
+ The ASF licenses this file to You under the Apache License, Version 2.0
+ (the "License"); you may not use this file except in compliance with
+ the License.  You may obtain a copy of the License at
+
+     http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+-->
+
 <!-- admin-extra.menu-top.html -->
+<!--
+<li>
+  <a href="#" style="background-image: url(img/ico/construction.png);">
+    FIRST ITEM
+  </a>
+</li>
+-->

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

@@ -48,27 +48,26 @@ sammy.get
     content_element
       .removeClass( 'single' );
     
-    var core_menu = this.active_core.closest( 'ul' );
+    if( !app.core_menu.data( 'admin-extra-loaded' ) )
-    if( !core_menu.data( 'admin-extra-loaded' ) )
     {
-      core_menu.data( 'admin-extra-loaded', new Date() );
+      app.core_menu.data( 'admin-extra-loaded', new Date() );
 
       $.get
       (
-        core_basepath + '/admin/file/?file=admin-extra.menu-top.html',
+        core_basepath + '/admin/file/?file=admin-extra.menu-top.html&contentType=text/html;charset=utf-8',
         function( menu_extra )
         {
-          core_menu
+          app.core_menu
             .prepend( menu_extra );
         }
       );
-            
+      
       $.get
       (
-        core_basepath + '/admin/file/?file=admin-extra.menu-bottom.html',
+        core_basepath + '/admin/file/?file=admin-extra.menu-bottom.html&contentType=text/html;charset=utf-8',
         function( menu_extra )
         {
-          core_menu
+          app.core_menu
             .append( menu_extra );
         }
       );