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

[issue#671] Fix smart contract code generators by generating two more… #672

Conversation

franz-see
Copy link

… constructors on the SolidityFunctionWrapper generated classes:

  • protected (String contractAddress, Web3j web3j, Credentials credentials, ContractGasProvider gasProvider) {}
  • protected (String contractAddress, Web3j web3j, TransactionManager transactionManager, ContractGasProvider gasProvider) {}

… constructors on the SolidityFunctionWrapper generated classes:

 * protected <Contract>(String contractAddress, Web3j web3j, Credentials credentials, ContractGasProvider gasProvider) {}
 * protected <Contract>(String contractAddress, Web3j web3j, TransactionManager transactionManager, ContractGasProvider gasProvider) {}
@codecov
Copy link

codecov bot commented Aug 12, 2018

Codecov Report

Merging #672 into release/3.6 will decrease coverage by 0.1%.
The diff coverage is 90.9%.

Impacted file tree graph

@@               Coverage Diff               @@
##             release/3.6   #672      +/-   ##
===============================================
- Coverage          77.11%    77%   -0.11%     
  Complexity          1794   1794              
===============================================
  Files                239    239              
  Lines               6667   6679      +12     
  Branches             996    996              
===============================================
+ Hits                5141   5143       +2     
- Misses              1275   1286      +11     
+ Partials             251    250       -1
Impacted Files Coverage Δ Complexity Δ
core/src/main/java/org/web3j/tx/Contract.java 74.66% <0%> (-1.01%) 46 <0> (ø)
...ava/org/web3j/codegen/SolidityFunctionWrapper.java 96.01% <100%> (+0.07%) 115 <1> (+1) ⬆️
...n/java/org/web3j/protocol/core/filters/Filter.java 53.12% <0%> (-14.07%) 8% <0%> (-2%)
...ore/src/main/java/org/web3j/utils/Observables.java 88% <0%> (+4%) 11% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe8c359...137c66a. Read the comment docs.

@yuriymyronovych
Copy link
Contributor

@franz-see looks great!

@snazha-blkio snazha-blkio changed the base branch from master to release/3.6 October 3, 2018 10:50
@snazha-blkio snazha-blkio changed the base branch from release/3.6 to release/4.0 October 18, 2018 20:32
@snazha-blkio snazha-blkio added this to the 4.0 milestone Oct 19, 2018
@iikirilov
Copy link
Contributor

This has been addressed in #725

@stewartsvg
Copy link

Hi, the new 4.0.1 web3j CLI tool smart contract wrappers generate a ContractGasProvider, but the latest Android version (3.3.1) of web3j do not contain this

@stewartsvg
Copy link

using web3j CLI version 3.5.0 works, is that the only workaround I can use for now? Sorry if this is not the right place to ask this question

@iikirilov
Copy link
Contributor

@stewartsvg please use web3j 3.5.0 or higher

we are in progress to release the android branch - I believe a snapshot is available

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

Successfully merging this pull request may close these issues.

None yet

6 participants