You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by vi...@apache.org on 2008/05/15 22:36:37 UTC
svn commit: r656823 -
/geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java
Author: viet
Date: Thu May 15 13:36:37 2008
New Revision: 656823
URL: http://svn.apache.org/viewvc?rev=656823&view=rev
Log:
Fix for Geronimo-3996. Ensure that there are no null rendered params.
Modified:
geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java
Modified: geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java?rev=656823&r1=656822&r2=656823&view=diff
==============================================================================
--- geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java (original)
+++ geronimo/server/trunk/plugins/monitoring/mconsole-war/src/main/java/org/apache/geronimo/monitoring/console/MonitoringPortlet.java Thu May 15 13:36:37 2008
@@ -224,6 +224,12 @@
String retention = actionRequest.getParameter("retention");
Integer port = Integer.parseInt(actionRequest.getParameter("port"));
Integer protocol = Integer.parseInt(actionRequest.getParameter("protocol"));
+ if(snapshot == null) {
+ snapshot = "";
+ }
+ if(retention == null) {
+ retention = "";
+ }
String message = testConnection(name, ip, username, password, port, protocol);
actionResponse.setRenderParameter("message", message);
actionResponse.setRenderParameter("name", name);