Fixing bug that SimplePipe still try to connect pipe on pipeFailed

zhourenjian@gmail.com · zhourenjian@gmail.com · commit b2c3e43695dc · 2014-03-11T12:50:13.000Z
diff --git a/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeRequest.java b/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeRequest.java
@@ -448,7 +448,9 @@ public void onLoaded() {
 					return;
 				}
 				runnable.ajaxOut();
-				ajaxPipe(runnable);
+				if (runnable.isPipeLive()) { // false if #pipeFailed is called in #ajaxOut  
+					ajaxPipe(runnable);
+				}
 			}
 		});
 		request.send(serialize);
@@ -1007,7 +1009,6 @@ public void onLoaded() { // on case that no destroy event is sent to client
 	 * being parsed.
 	 */
 	public static String parseReceived(final String string) {
-		//System.out.println(string);
 		if (string == null) {
 			return null;
 		}
diff --git a/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeRunnable.java b/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeRunnable.java
@@ -202,6 +202,7 @@ public void ajaxFail() {
 	@Override
 	public void ajaxOut() {
 		if (pipeAlive) {
+			updateStatus(true);
 			pipeCreated();
 		} else {
 			pipeFailed();

Original file line number	Diff line number	Diff line change
`@@ -448,7 +448,9 @@ public void onLoaded() {`
`448`	`448`	`return;`
`449`	`449`	`}`
`450`	`450`	`runnable.ajaxOut();`
`451`		`- ajaxPipe(runnable);`
	`451`	`+ if (runnable.isPipeLive()) { // false if #pipeFailed is called in #ajaxOut`
	`452`	`+ ajaxPipe(runnable);`
	`453`	`+ }`
`452`	`454`	`}`
`453`	`455`	`});`
`454`	`456`	`request.send(serialize);`
`@@ -1007,7 +1009,6 @@ public void onLoaded() { // on case that no destroy event is sent to client`
`1007`	`1009`	`* being parsed.`
`1008`	`1010`	`*/`
`1009`	`1011`	`public static String parseReceived(final String string) {`
`1010`		`- //System.out.println(string);`
`1011`	`1012`	`if (string == null) {`
`1012`	`1013`	`return null;`
`1013`	`1014`	`}`