--- CircuitBreaker/src/dk/thoerup/curcuitbreaker/web/ViewCircuitBreaker.java	2009/10/21 07:56:37	460
+++ CircuitBreaker/src/dk/thoerup/circuitbreaker/web/ViewCircuitBreaker.java	2012/08/31 06:33:12	1844
@@ -1,13 +1,16 @@
-package dk.thoerup.curcuitbreaker.web;
+package dk.thoerup.circuitbreaker.web;
 
 import java.util.Date;
+import java.util.Formatter;
+import java.util.LinkedList;
 
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import dk.thoerup.curcuitbreaker.AccountingCircuitBreaker;
-import dk.thoerup.curcuitbreaker.CircuitBreaker;
-import dk.thoerup.curcuitbreaker.CircuitBreakerManager;
+import dk.thoerup.circuitbreaker.AccountingCircuitBreaker;
+import dk.thoerup.circuitbreaker.LoggingCircuitBreaker;
+import dk.thoerup.circuitbreaker.CircuitBreaker;
+import dk.thoerup.circuitbreaker.CircuitBreakerManager;
 
 public class ViewCircuitBreaker implements Command {
 	
@@ -45,6 +48,14 @@
 		return sb.toString();
 	}
 	
+	public static String formatDate(long date) {
+		if (date == 0L) {
+			return "-";
+		} else {
+			return new Date(date).toString();
+		}
+	}
+	
 	public String execute(HttpServletRequest req, HttpServletResponse resp) {
 		CircuitBreaker breaker = CircuitBreakerManager.getManager().getCircuitBreaker( req.getParameter("breaker"));
 		String uri = req.getRequestURI();
@@ -62,7 +73,7 @@
 		sb.append("<b>Attributes:</b>\n");
 		sb.append("<table border=\"1\" cellspacing=0>");		
 		sb.append("<tr><td>Name </td><td>").append(breaker.getName()).append("</td></tr>\n");
-		sb.append("<tr><td>State </td><td><b>").append(  breaker ).append("</b></td></tr>\n");
+		sb.append("<tr><td>State </td><td><b>").append(  getStateBar(breaker) ).append("</b></td></tr>\n");
 		sb.append("<tr><td>Type </td><td>").append(breaker.getClass().getSimpleName() ).append("</td></tr>\n");
 		sb.append("<tr><td>Notifier </td><td>").append(breaker.getNotifierName()).append("</td></tr>\n");
 		sb.append("<tr><td>Failure count </td><td>").append(breaker.getFailureCount()).append("</td></tr>\n");
@@ -72,16 +83,47 @@
 		
 		if (breaker instanceof AccountingCircuitBreaker) {
 			AccountingCircuitBreaker acb = (AccountingCircuitBreaker) breaker;
+	
+			double failure_percentage = ((double) acb.getTotalFailureCount()*100) / ((double) acb.getTotalCallCount() );
+			StringBuilder sbf = new StringBuilder();
+			Formatter formatter = new Formatter(sb);
+			formatter.format("%.2f", failure_percentage);
+			
+
 			sb.append("<tr><th colspan=\"2\">Accounting</th></tr>\n");
 			sb.append("<tr><td>Trip count</td><td>").append( acb.getTripCount() ).append("</td></tr>\n");
+			sb.append("<tr><td>Re-trip count</td><td>").append( acb.getRetripCount() ).append("</td></tr>\n");
 			sb.append("<tr><td>Block count</td><td>").append( acb.getBlockCount() ).append("</td></tr>\n");
 			sb.append("<tr><td>Total failure count</td><td>").append( acb.getTotalFailureCount() ).append("</td></tr>\n");
 			sb.append("<tr><td>Total Call count</td><td>").append( acb.getTotalCallCount() ).append("</td></tr>\n");
-			sb.append("<tr><td>Last trip</td><td>").append( new Date(acb.getLastTrip()).toString() ).append("</td></tr>\n");
-			sb.append("<tr><td>Last reset</td><td>").append( new Date(acb.getLastResetCounters()).toString() ).append("</td></tr>\n");			
+			sb.append("<tr><td>Failure Percentage</td><td>").append( sb.toString() ).append("%</td></tr>\n");
+			sb.append("<tr><td>Last failure</td><td>").append( formatDate(acb.getLastFailure()) ).append("</td></tr>\n");
+			sb.append("<tr><td>Last trip</td><td>").append( formatDate(acb.getLastTrip()) ).append("</td></tr>\n");
+			sb.append("<tr><td>Last re-trip</td><td>").append( formatDate(acb.getLastRetrip()) ).append("</td></tr>\n");
+			sb.append("<tr><td>Last reset</td><td>").append( formatDate(acb.getLastReset()) ).append("</td></tr>\n");
+			
+			sb.append("<tr><td>Last resetCounters</td><td>").append( formatDate(acb.getLastResetCounters()) ).append("</td></tr>\n");			
+		}
+
+		if (breaker instanceof LoggingCircuitBreaker) {
+			LoggingCircuitBreaker lcb = (LoggingCircuitBreaker) breaker;			
+			
+			LinkedList<LoggingCircuitBreaker.LogEntry> list = lcb.getLog();
+			String headerLink = String.format("<a href=\"%s?command=log&breaker=%s\">Logging</a>", req.getRequestURI(), breaker.getName() );
+
+			sb.append("<tr><th colspan=\"2\">").append(headerLink).append("</th></tr>\n");
+			
+			int max = list.size() > 10 ? 10 : list.size();			
+			for (int i=0; i<max; i++) {
+				LoggingCircuitBreaker.LogEntry entry = list.get(i);
+				sb.append("<tr><td colspan=\"2\">");
+				sb.append( entry.toString() );				
+				sb.append("</td></tr>");
+			}
 		}
 		
-		sb.append("</table><br>\n");
+		sb.append("</table>\n");
+		sb.append("Current time: ").append( new Date().toString() ).append("<br><br>\n");
 		
 		sb.append("<b>Operations:</b>\n");
 		if ( !readOnly ) {
@@ -90,6 +132,9 @@
 			if (breaker instanceof AccountingCircuitBreaker ) {
 				sb.append( actionBuilder(uri, "resetCounters", breaker.getName())).append("<br>\n");	
 			}
+			if (breaker instanceof LoggingCircuitBreaker ) {
+				sb.append( actionBuilder(uri, "clearLog", breaker.getName())).append("<br>\n");	
+			}
 		} else {
 			sb.append("<i>the CircuitBreakers can only be viewed</i>");
 		}