Skip to content

fix bug of event parse #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Oct 29, 2020
Merged

fix bug of event parse #178

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8b8978d
fix bug of event parse
dalaocu Oct 28, 2020
9de7f43
Merge branch 'dev' of https://github.com/FISCO-BCOS/java-sdk into eve…
dalaocu Oct 29, 2020
00b03c9
change return code of call
dalaocu Oct 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .../src/main/java/org/fisco/bcos/sdk/transaction/codec/decode/TransactionDecoderService.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ public Map<String, List<List<Object>>> decodeEvents(String abi, List<Logs> logs)
(name, events) -> {
for (ABIDefinition abiDefinition : events) {
ABIObjectFactory abiObjectFactory = new ABIObjectFactory();
ABIObject outputObject = abiObjectFactory.createInputObject(abiDefinition);
ABIObject outputObject = abiObjectFactory.createEventInputObject(abiDefinition);
ABICodecObject abiCodecObject = new ABICodecObject();
for (Logs log : logs) {
String eventSignature =
Expand Down
3 changes: 2 additions & 1 deletion ...on/src/main/java/org/fisco/bcos/sdk/transaction/manager/AssembleTransactionProcessor.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@
import org.fisco.bcos.sdk.transaction.pusher.TransactionPusherService;
import org.fisco.bcos.sdk.transaction.tools.ContractLoader;
import org.fisco.bcos.sdk.transaction.tools.JsonUtils;
import org.fisco.bcos.sdk.utils.Numeric;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

Expand Down Expand Up @@ -315,7 +316,7 @@ private CallResponse parseCallResponseStatus(Call.CallOutput callOutput)
throws TransactionBaseException {
CallResponse callResponse = new CallResponse();
RetCode retCode = ReceiptParser.parseCallOutput(callOutput, "");
callResponse.setReturnCode(retCode.getCode());
callResponse.setReturnCode(Numeric.decodeQuantity(callOutput.getStatus()).intValue());
callResponse.setReturnMessage(retCode.getMessage());
if (!retCode.getMessage().equals(PrecompiledRetCode.CODE_SUCCESS.getMessage())) {
throw new TransactionBaseException(retCode);
Expand Down
3 changes: 2 additions & 1 deletion sdk-transaction/src/main/java/org/fisco/bcos/sdk/transaction/model/dto/CommonResponse.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ public int getReturnCode() {
return returnCode;
}

/** @param returnCode the returnCode to set */
/**
* @param returnCode the returnCode to set */
public void setReturnCode(int returnCode) {
this.returnCode = returnCode;
}
Expand Down