crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add assertion that one cannot specify both blockHash & fromBlock/toBlock to getLogsAsync

Browse Source
This commit is contained in:
Fabio Berger 2018-09-21 13:43:06 +01:00
parent 4e8ec2359d
commit e6840c60c7
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/web3-wrapper/src/web3_wrapper.ts
View File

@ -438,6 +438,10 @@ export class Web3Wrapper {
* @returns The corresponding log entries * @returns The corresponding log entries
*/ */
public async getLogsAsync(filter: FilterObject): Promise<LogEntry[]> { public async getLogsAsync(filter: FilterObject): Promise<LogEntry[]> {
if (!_.isUndefined(filter.blockHash) && (!_.isUndefined(filter.fromBlock) || !_.isUndefined(filter.toBlock))) {
throw new Error(`Cannot specify 'blockHash' as well as 'fromBlock'/'toBlock' in the filter supplied to 'getLogsAsync'`);
}
let fromBlock = filter.fromBlock; let fromBlock = filter.fromBlock;
if (_.isNumber(fromBlock)) { if (_.isNumber(fromBlock)) {
fromBlock = utils.numberToHex(fromBlock); fromBlock = utils.numberToHex(fromBlock);