search

raineorshine / solidity-repl

Ethereum Solidity REPL
ISC License
396 stars 33 forks source link

Add support for special variables like `msg` and `block` #7

Closed Dev43 closed 6 years ago

Dev43 commented 7 years ago

msg (node:14435) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: Cannot read property '1' of null (node:14435) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

raineorshine commented 6 years ago

Thanks. It would be helpful if it returned a tuple of values in msg.

raineorshine commented 6 years ago

http://solidity.readthedocs.io/en/v0.4.24/units-and-global-variables.html#special-variables-and-functions

gitcoinbot commented 6 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

This issue now has a funding of 40.0 DAI (40.0 USD @ $1.0/DAI) attached to it.

gitcoinbot commented 6 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work has been started.

These users each claimed they can complete the work by 7 months, 3 weeks ago. Please review their action plans below:

1) djmgit has started work.

Hi, I would like to take up this issue. I am going through the source code, I will be sending a PR soon.

Learn more on the Gitcoin Issue Details page.

vs77bb commented 6 years ago

Hi @djmgit Vivek from Gitcoin here, excited to see your progress! Do you have a WIP PR here?

djmgit commented 6 years ago

@raineorshine presently msg.data, msg.data, etc, evrything is showing result, same for the block command. What actually is wanted here? Do you want that on entering msg, it should return a tuple containing all the different values like msg.sender, msg.sender etc and same for block?

@vs77bb I have ran into some problem with my os, I will be sending a pr asap.

raineorshine commented 6 years ago

@djmgit Yes, although instead of a tuple I think it should display it similar to a struct i.e. to show both keys and values.

gitcoinbot commented 6 years ago

@djmgit Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

djmgit commented 6 years ago

@gitcoinbot i am working on this, will submit a pr asap.

gitcoinbot commented 6 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

Work for 40.0 DAI (40.0 USD @ $1.0/DAI) has been submitted by:

  1. @djmgit

@owocki please take a look at the submitted work:

gitcoinbot commented 6 years ago

Issue Status: 1. Open 2. Started 3. Submitted 4. Done

The funding of 40.0 DAI (40.0 USD @ $1.0/DAI) attached to this issue has been approved & issued to @djmgit.