Author: kkolinko Date: Thu Feb 14 05:08:58 2019 New Revision: 1853557 URL: http://svn.apache.org/viewvc?rev=1853557&view=rev Log: Improve display of summary values on the status page of Manager web application: separate terms and values with a whitespace.
Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/manager/StatusTransformer.java tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/manager/StatusTransformer.java URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/manager/StatusTransformer.java?rev=1853557&r1=1853556&r2=1853557&view=diff ============================================================================== --- tomcat/tc7.0.x/trunk/java/org/apache/catalina/manager/StatusTransformer.java (original) +++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/manager/StatusTransformer.java Thu Feb 14 05:08:58 2019 @@ -177,19 +177,31 @@ public class StatusTransformer { writer.print("<p>"); writer.print( args[0] ); + writer.print(' '); writer.print(formatSize(Long.valueOf(result[0]), true)); - writer.print(" " + args[1]); + writer.print(' '); + writer.print(args[1]); + writer.print(' '); writer.print(formatSize(Long.valueOf(result[1]), true)); - writer.print(" " + args[2]); + writer.print(' '); + writer.print(args[2]); + writer.print(' '); writer.print(formatSize(Long.valueOf(result[2]), true)); - writer.print(" " + args[3]); + writer.print(' '); + writer.print(args[3]); + writer.print(' '); writer.print(formatSize(Long.valueOf(result[3]), true)); - writer.print(" " + args[4]); + writer.print(' '); + writer.print(args[4]); + writer.print(' '); writer.print(Long.valueOf(result[6])); writer.print("<br>"); - writer.print(" " + args[5]); + writer.print(args[5]); + writer.print(' '); writer.print(formatTime(Long.valueOf(result[11] / 1000), true)); - writer.print(" " + args[6]); + writer.print(' '); + writer.print(args[6]); + writer.print(' '); writer.print(formatTime(Long.valueOf(result[12] / 1000), true)); writer.print("</p>"); } else if (mode == 1){ @@ -223,12 +235,17 @@ public class StatusTransformer { writer.print("<p>"); writer.print( args[0] ); + writer.print(' '); writer.print(formatSize( Long.valueOf(Runtime.getRuntime().freeMemory()), true)); - writer.print(" " + args[1] ); + writer.print(' '); + writer.print(args[1]); + writer.print(' '); writer.print(formatSize( Long.valueOf(Runtime.getRuntime().totalMemory()), true)); - writer.print(" " + args[2] ); + writer.print(' '); + writer.print(args[2]); + writer.print(' '); writer.print(formatSize( Long.valueOf(Runtime.getRuntime().maxMemory()), true)); writer.print("</p>"); @@ -305,14 +322,21 @@ public class StatusTransformer { writer.print("<p>"); writer.print( args[0] ); + writer.print(' '); writer.print(mBeanServer.getAttribute(tpName, "maxThreads")); - writer.print(" " + args[1]); + writer.print(' '); + writer.print(args[1]); + writer.print(' '); writer.print(mBeanServer.getAttribute(tpName, "currentThreadCount")); - writer.print(" " + args[2]); + writer.print(' '); + writer.print(args[2]); + writer.print(' '); writer.print(mBeanServer.getAttribute(tpName, "currentThreadsBusy")); try { Object value = mBeanServer.getAttribute(tpName, "keepAliveCount"); - writer.print(" " + args[3]); + writer.print(' '); + writer.print(args[3]); + writer.print(' '); writer.print(value); } catch (Exception e) { // Ignore @@ -336,19 +360,30 @@ public class StatusTransformer { } writer.print( args[4] ); + writer.print(' '); writer.print(formatTime(mBeanServer.getAttribute (grpName, "maxTime"), false)); - writer.print(" " + args[5]); + writer.print(' '); + writer.print(args[5]); + writer.print(' '); writer.print(formatTime(mBeanServer.getAttribute (grpName, "processingTime"), true)); - writer.print(" " + args[6]); + writer.print(' '); + writer.print(args[6]); + writer.print(' '); writer.print(mBeanServer.getAttribute(grpName, "requestCount")); - writer.print(" " + args[7]); + writer.print(' '); + writer.print(args[7]); + writer.print(' '); writer.print(mBeanServer.getAttribute(grpName, "errorCount")); - writer.print(" " + args[8]); + writer.print(' '); + writer.print(args[8]); + writer.print(' '); writer.print(formatSize(mBeanServer.getAttribute (grpName, "bytesReceived"), true)); - writer.print(" " + args[9]); + writer.print(' '); + writer.print(args[9]); + writer.print(' '); writer.print(formatSize(mBeanServer.getAttribute (grpName, "bytesSent"), true)); writer.print("</p>"); @@ -516,20 +551,20 @@ public class StatusTransformer { if (showRequest) { writer.write(filter(mBeanServer.getAttribute (pName, "method"))); - writer.write(" "); + writer.write(' '); writer.write(filter(mBeanServer.getAttribute (pName, "currentUri"))); String queryString = (String) mBeanServer.getAttribute (pName, "currentQueryString"); if ((queryString != null) && (!queryString.equals(""))) { - writer.write("?"); + writer.write('?'); writer.print(RequestUtil.filter(queryString)); } - writer.write(" "); + writer.write(' '); writer.write(filter(mBeanServer.getAttribute (pName, "protocol"))); } else { - writer.write("?"); + writer.write('?'); } writer.write("</td>"); } else { Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1853557&r1=1853556&r2=1853557&view=diff ============================================================================== --- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Thu Feb 14 05:08:58 2019 @@ -209,6 +209,10 @@ Fix messages used by Host Manager web application. Disambiguate message keys used for different messages. (kkolinko) </fix> + <fix> + Improve display of summary values on the status page of Manager web + application: separate terms and values with a whitespace. (kkolinko) + </fix> </changelog> </subsection> <subsection name="Other"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org