Skip to content
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.

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

public functions does not have a correct function name #34

Closed
ferOnti opened this issue Dec 19, 2016 · 6 comments
Closed

public functions does not have a correct function name #34

ferOnti opened this issue Dec 19, 2016 · 6 comments

Comments

@ferOnti
Copy link

ferOnti commented Dec 19, 2016

using the command line wrapper generator, the functions for public variables are generated with invalid function name. (this is happening with variables with type array of structs)

Example:

struct Thing {
    string id;
    string name;
}

Thing[]    public things;

the generate function is like:

   Function function = new Function("org.web3j.abi.datatypes.generated.Uint256", 
            Arrays.<Type>asList(param0), 

instead:

    Function function = new Function("things", 
            Arrays.<Type>asList(param0), 
@conor10
Copy link
Contributor

conor10 commented Jan 1, 2017

Please could you provide the ABI file?

conor10 added a commit that referenced this issue Jan 3, 2017
.

2. Added web3j-tests.jar to build artifacts to facilitate incorporating test classes into other web3j libraries.
3. Bumped version to 1.1.1.
@conor10
Copy link
Contributor

conor10 commented Jan 3, 2017

@conor10 conor10 closed this as completed Jan 3, 2017
@ferOnti
Copy link
Author

ferOnti commented Jan 3, 2017 via email

@conor10
Copy link
Contributor

conor10 commented Jan 4, 2017

No problem - this certainly sounds like useful functionality to have.

Feel free to raise an issue for this feature.

I think the main complexity is how best to support it without passing on the nonce management burden to the users of the library - ideally I'd like to hide this complexity, as I've always wanted to keep the API as straight forwards as possible.

It may make sense to create a separate nonce manager behind the scenes which tracks nonces locally for higher throughput applications. That way, only those users with high throughput needs would need to use it. The others could use the current, default implementation.

I'd be happy to accept a pull request providing this functionality :)

@ferOnti
Copy link
Author

ferOnti commented Jan 4, 2017 via email

@conor10
Copy link
Contributor

conor10 commented Jan 4, 2017

That's a decent transaction throughput to get.

Is there a quantity value in your pre-encoded request that is being specified with a leading zero, e.g. 0x0... - as these are not supported in the JSON-RPC API - see https://github.com/ethereum/wiki/wiki/JSON-RPC#hex-value-encoding.

denis554 added a commit to denis554/web3j that referenced this issue Mar 27, 2019
…yperledger/web3j#34.

2. Added web3j-tests.jar to build artifacts to facilitate incorporating test classes into other web3j libraries.
3. Bumped version to 1.1.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants