[FAB-3263] remove ChaincodeBase.getChaincodeId()

Luis Sanchez · Luis Sanchez · commit 64581267895c · 2017-04-26T12:17:43.000-04:00
No longer needed.

Change-Id: I991cf3d16b629030cfd92482ea1ebcdd843ff76f
Signed-off-by: Luis Sanchez &lt;sanchezl@us.ibm.com&gt;
diff --git a/core/chaincode/shim/java/src/main/java/org/hyperledger/fabric/shim/ChaincodeBase.java b/core/chaincode/shim/java/src/main/java/org/hyperledger/fabric/shim/ChaincodeBase.java
@@ -58,7 +58,6 @@ public abstract class ChaincodeBase implements Chaincode {
 	
     private static Log logger = LogFactory.getLog(ChaincodeBase.class);
     
-    public abstract String getChaincodeID();
     public static final String DEFAULT_HOST = "127.0.0.1";
     public static final int DEFAULT_PORT = 7051;
 
@@ -69,26 +68,32 @@ public abstract class ChaincodeBase implements Chaincode {
     private String rootCertFile = "/etc/hyperledger/fabric/peer.crt";
 
     private Handler handler;
-    private String id = getChaincodeID();
+    private String id;
 
     private final static String CORE_CHAINCODE_ID_NAME = "CORE_CHAINCODE_ID_NAME";
     private final static String CORE_PEER_ADDRESS = "CORE_PEER_ADDRESS";
     private final static String CORE_PEER_TLS_ENABLED = "CORE_PEER_TLS_ENABLED";
     private final static String CORE_PEER_TLS_SERVERHOSTOVERRIDE = "CORE_PEER_TLS_SERVERHOSTOVERRIDE";
     private static final String CORE_PEER_TLS_ROOTCERT_FILE = "CORE_PEER_TLS_ROOTCERT_FILE";
 
-    // Start entry point for chaincodes bootstrap.
-    public void start(String[] args) {
-	processEnvironmentOptions();
-	processCommandLineOptions(args);
-	new Thread(() -> {
-	    logger.trace("chaincode started");
-	    final ManagedChannel connection = newPeerClientConnection();
-	    logger.trace("connection created");
-	    chatWithPeer(connection);
-	    logger.trace("chatWithPeer DONE");
-	}).start();
-    }
+	/**
+	 * Start chaincode
+	 * @param args command line arguments
+	 */
+	public void start(String[] args) {
+		processEnvironmentOptions();
+		processCommandLineOptions(args);
+		if(this.id == null) {
+			logger.error(String.format("The chaincode id must be specified using either the -i or --i command line options or the %s environment variable.", CORE_CHAINCODE_ID_NAME));
+		}
+		new Thread(() -> {
+			logger.trace("chaincode started");
+			final ManagedChannel connection = newPeerClientConnection();
+			logger.trace("connection created");
+			chatWithPeer(connection);
+			logger.trace("chatWithPeer DONE");
+		}).start();
+	}
 
     private void processCommandLineOptions(String[] args) {
 	Options options = new Options();
diff --git a/examples/chaincode/java/Example/src/main/java/example/Example.java b/examples/chaincode/java/Example/src/main/java/example/Example.java
@@ -17,10 +17,10 @@
 
 package example;
 
-import org.hyperledger.fabric.shim.ChaincodeBase;
-import org.hyperledger.fabric.shim.ChaincodeStub;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
+import org.hyperledger.fabric.shim.ChaincodeBase;
+import org.hyperledger.fabric.shim.ChaincodeStub;
 
 
 /**
@@ -101,11 +101,6 @@ public String query(ChaincodeStub stub, String function, String[] args) {
 		}
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "hello";
-	}
-
 	public static void main(String[] args) throws Exception {
 		System.out.println("Hello world! starting "+args);
 		log.info("starting");
diff --git a/examples/chaincode/java/LinkExample/src/main/java/example/LinkExample.java b/examples/chaincode/java/LinkExample/src/main/java/example/LinkExample.java
@@ -76,9 +76,4 @@ public static void main(String[] args) throws Exception {
 		new LinkExample().start(args);
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "link";
-	}
-
 }
diff --git a/examples/chaincode/java/MapExample/src/main/java/example/MapExample.java b/examples/chaincode/java/MapExample/src/main/java/example/MapExample.java
@@ -49,11 +49,6 @@ public String query(ChaincodeStub stub, String function, String[] args) {
 		return stub.getState(args[0]);
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "map";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new MapExample().start(args);
 	}
diff --git a/examples/chaincode/java/RangeExample/src/main/java/example/RangeExample.java b/examples/chaincode/java/RangeExample/src/main/java/example/RangeExample.java
@@ -71,11 +71,6 @@ public String query(ChaincodeStub stub, String function, String[] args) {
 
     }
 
-    @java.lang.Override
-    public String getChaincodeID() {
-        return "RangeExample";
-    }
-
     public static void main(String[] args) throws Exception {
         log.info("starting");
         new RangeExample().start(args);
diff --git a/examples/chaincode/java/SimpleSample/src/main/java/example/SimpleSample.java b/examples/chaincode/java/SimpleSample/src/main/java/example/SimpleSample.java
@@ -142,11 +142,6 @@ public Response query(ChaincodeStub stub, String function, String[] args) {
 
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "SimpleSample";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new SimpleSample().start(args);
 	}
diff --git a/examples/chaincode/java/chaincode_example02/src/main/java/example/Example02.java b/examples/chaincode/java/chaincode_example02/src/main/java/example/Example02.java
@@ -22,8 +22,6 @@
 import static org.hyperledger.fabric.shim.ChaincodeHelper.newInternalServerErrorResponse;
 import static org.hyperledger.fabric.shim.ChaincodeHelper.newSuccessResponse;
 
-import java.util.List;
-
 import javax.json.Json;
 
 import org.apache.commons.logging.Log;
@@ -155,11 +153,6 @@ private Response query(ChaincodeStub stub, String[] args) {
 
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "Example02";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new Example02().start(args);
 	}
diff --git a/examples/chaincode/java/chaincode_example04/src/main/java/example/Example04.java b/examples/chaincode/java/chaincode_example04/src/main/java/example/Example04.java
@@ -150,11 +150,6 @@ private Response doInvoke(ChaincodeStub stub, String[] args) {
 		return response;
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "Example04";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new Example04().start(args);
 	}
diff --git a/examples/chaincode/java/chaincode_example05/src/main/java/example/Example05.java b/examples/chaincode/java/chaincode_example05/src/main/java/example/Example05.java
@@ -129,11 +129,6 @@ private Response doInvoke(ChaincodeStub stub, String[] args) {
 		return newSuccessResponse(String.valueOf(sum).getBytes(UTF_8));
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "Example05";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new Example05().start(args);
 	}
diff --git a/examples/chaincode/java/chaincode_example06/src/main/java/example/Example06.java b/examples/chaincode/java/chaincode_example06/src/main/java/example/Example06.java
@@ -42,11 +42,6 @@ public Response invoke(ChaincodeStub stub) {
 			return newBadRequestResponse(format("Invalid arguments specified"));
 		}
 	}
-	
-	@Override
-	public String getChaincodeID() {
-		return "Example06";
-	}
 
 	public static void main(String[] args) throws Exception {
 		new Example06().start(args);
diff --git a/examples/chaincode/java/eventsender/src/main/java/example/EventSender.java b/examples/chaincode/java/eventsender/src/main/java/example/EventSender.java
@@ -19,7 +19,6 @@
 import static java.lang.String.format;
 import static java.nio.charset.StandardCharsets.UTF_8;
 import static java.util.stream.Collectors.joining;
-import static java.util.stream.Collectors.toList;
 import static org.hyperledger.fabric.shim.ChaincodeHelper.newBadRequestResponse;
 import static org.hyperledger.fabric.shim.ChaincodeHelper.newInternalServerErrorResponse;
 import static org.hyperledger.fabric.shim.ChaincodeHelper.newSuccessResponse;
@@ -82,11 +81,6 @@ private Response doQuery(ChaincodeStub stub) {
 		return newSuccessResponse(String.format("{\"NoEvents\":%d}", Integer.parseInt(stub.getStringState(EVENT_COUNT))));
 	}
 
-	@Override
-	public String getChaincodeID() {
-		return "EventSender";
-	}
-
 	public static void main(String[] args) throws Exception {
 		new EventSender().start(args);
 	}

Original file line number	Diff line number	Diff line change
`@@ -76,9 +76,4 @@ public static void main(String[] args) throws Exception {`
`76`	`76`	`new LinkExample().start(args);`
`77`	`77`	`}`
`78`	`78`
`79`		`- @Override`
`80`		`- public String getChaincodeID() {`
`81`		`- return "link";`
`82`		`- }`
`83`		`-`
`84`	`79`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,11 +49,6 @@ public String query(ChaincodeStub stub, String function, String[] args) {`
`49`	`49`	`return stub.getState(args[0]);`
`50`	`50`	`}`
`51`	`51`
`52`		`- @Override`
`53`		`- public String getChaincodeID() {`
`54`		`- return "map";`
`55`		`- }`
`56`		`-`
`57`	`52`	`public static void main(String[] args) throws Exception {`
`58`	`53`	`new MapExample().start(args);`
`59`	`54`	`}`
Original file line number	Diff line number	Diff line change
`@@ -142,11 +142,6 @@ public Response query(ChaincodeStub stub, String function, String[] args) {`
`142`	`142`
`143`	`143`	`}`
`144`	`144`
`145`		`- @Override`
`146`		`- public String getChaincodeID() {`
`147`		`- return "SimpleSample";`
`148`		`- }`
`149`		`-`
`150`	`145`	`public static void main(String[] args) throws Exception {`
`151`	`146`	`new SimpleSample().start(args);`
`152`	`147`	`}`
Original file line number	Diff line number	Diff line change
`@@ -150,11 +150,6 @@ private Response doInvoke(ChaincodeStub stub, String[] args) {`
`150`	`150`	`return response;`
`151`	`151`	`}`
`152`	`152`
`153`		`- @Override`
`154`		`- public String getChaincodeID() {`
`155`		`- return "Example04";`
`156`		`- }`
`157`		`-`
`158`	`153`	`public static void main(String[] args) throws Exception {`
`159`	`154`	`new Example04().start(args);`
`160`	`155`	`}`
Original file line number	Diff line number	Diff line change
`@@ -129,11 +129,6 @@ private Response doInvoke(ChaincodeStub stub, String[] args) {`
`129`	`129`	`return newSuccessResponse(String.valueOf(sum).getBytes(UTF_8));`
`130`	`130`	`}`
`131`	`131`
`132`		`- @Override`
`133`		`- public String getChaincodeID() {`
`134`		`- return "Example05";`
`135`		`- }`
`136`		`-`
`137`	`132`	`public static void main(String[] args) throws Exception {`
`138`	`133`	`new Example05().start(args);`
`139`	`134`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,11 +42,6 @@ public Response invoke(ChaincodeStub stub) {`
`42`	`42`	`return newBadRequestResponse(format("Invalid arguments specified"));`
`43`	`43`	`}`
`44`	`44`	`}`
`45`		`-`
`46`		`- @Override`
`47`		`- public String getChaincodeID() {`
`48`		`- return "Example06";`
`49`		`- }`
`50`	`45`
`51`	`46`	`public static void main(String[] args) throws Exception {`
`52`	`47`	`new Example06().start(args);`