-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[Add] GetStorage for ContractState #3615
base: master
Are you sure you want to change the base?
[Add] GetStorage for ContractState #3615
Conversation
/// <param name="storageKey">Key in the storage map.</param> | ||
/// <returns>Storage value of the item.</returns> | ||
/// <exception cref="ArgumentNullException"></exception> | ||
public static StorageItem GetStorage(this ContractState contractState, DataCache snapshot, byte[] storageKey) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be rename to StorageGet
(an analogue of System.Storage.Get
)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to keep the naming a convention
of dotnet
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a command to call this method in the client? otherwise I don't see a real use case
Description
Get contract storage for a given contract. Returns all storage keys and values for
StorageMap
.Change Log
GetStorage
extension methodsType of change
How Has This Been Tested?
TestGetStorage
Checklist: