You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cz...@apache.org on 2013/10/21 13:11:43 UTC
svn commit: r1534095 - in /felix/trunk/webconsole/src/main:
java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java
resources/res/ui/config.js
Author: cziegeler
Date: Mon Oct 21 11:11:43 2013
New Revision: 1534095
URL: http://svn.apache.org/r1534095
Log:
FELIX-4187 : Configuration Plugin does a POST to get configuration details
Modified:
felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java
felix/trunk/webconsole/src/main/resources/res/ui/config.js
Modified: felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java?rev=1534095&r1=1534094&r2=1534095&view=diff
==============================================================================
--- felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java (original)
+++ felix/trunk/webconsole/src/main/java/org/apache/felix/webconsole/internal/configuration/ConfigManager.java Mon Oct 21 11:11:43 2013
@@ -22,6 +22,7 @@ import java.io.PrintWriter;
import java.util.Iterator;
import java.util.Locale;
import java.util.Map;
+
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
@@ -213,8 +214,13 @@ public class ConfigManager extends Simpl
protected void doGet( HttpServletRequest request, HttpServletResponse response )
throws ServletException, IOException
{
- // let's check for a JSON request
+ // check for "post" requests from previous versions
+ if ( "true".equals(request.getParameter("post")) ) {
+ this.doPost(request, response);
+ return;
+ }
final String info = request.getPathInfo();
+ // let's check for a JSON request
if ( info.endsWith( ".json" ) ) //$NON-NLS-1$
{
response.setContentType( "application/json" ); //$NON-NLS-1$
Modified: felix/trunk/webconsole/src/main/resources/res/ui/config.js
URL: http://svn.apache.org/viewvc/felix/trunk/webconsole/src/main/resources/res/ui/config.js?rev=1534095&r1=1534094&r2=1534095&view=diff
==============================================================================
--- felix/trunk/webconsole/src/main/resources/res/ui/config.js (original)
+++ felix/trunk/webconsole/src/main/resources/res/ui/config.js Mon Oct 21 11:11:43 2013
@@ -34,12 +34,12 @@ var editorMessage = false;
function configure(pid, create) {
var uri = pluginRoot + '/' + pid;
// we have to add a cache killer for IE8
- var postUri = uri + '?';
+ var postUri = uri + '?post=true&';
if ( create ) {
postUri += param.create + '=1&';
}
postUri = postUri + 'ts='+new Date().getMilliseconds();
- $.post(postUri, null, displayConfigForm, 'json');
+ $.get(postUri, null, displayConfigForm, 'json');
}
function displayConfigForm(obj) {