Example run-transactions-complete, not printing storage content

[blintgit]

Running the example mentioned in the title, there is no output after the "---Storage---" line. According to my understanding, there should be a null_radix stored at the created address. (running gives no exceptions)
Please correct me if I'm wrong! Thanks

[blintgit]

[update]

At the following part of the example at the runTx function I've noticed that we refresh the createdAddress variable at every transaction even though it doesn't register a new contract. Therefore it leaves us with the loss of our contract's address and the readStorage function won't print any output.

I've fixed it by checking the createdAddress property for null before saving the value.

if (returns.createdAddress) {
  createdAddress = results.createdAddress
  console.log('address created: ' +
      createdAddress.toString('hex'))
}

[holgerd77]

Hi @blintgit, maybe wait up a couple of more contributions before talking in the "we" form, this is a bit irritating. :-)

That said: good catch, can you prepare a PR with your proposed changes?

[blintgit]

Yes, I'm preparing the PR asap. Thanks

[hswick]

returns is not a defined variable, believe you meant results. However, this doesn't seem to fix the issue for me.

[rosulucian]

same as @hswick :
i added
if (!createdAddress) { createdAddress = results.createdAddress }

so the address would not reset but I stil get an empty trie and no storage is printed

[rosulucian]

I had another look at it and after calling runCode I tried getting the account so I can read it`s trie by calling either stateTrie.get(createdAddress) or res.runState.stateManager.getAccount(res.runState.address) but I only get an empty account, i.e. codehash is SHA3_NULL_S and stateRoot is SHA3_RLP_S.

Then I tried to get the storage trie through res.runState.stateManager._getStorageTrie(res.runState.address) and for some reason you can still find it in the state manager (idk why it has two caches?!) it just didn`t commit somehow.

[holgerd77]

Fixed by #550, will close.