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

@0x:contracts-utils Updated the OwnershipTransferred event to be closer to OpenZeppelin's event

Browse Source
This commit is contained in:
Alex Towle 2019-10-10 10:58:58 -07:00
parent 7815da7257
commit 65f2626544
3 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
contracts/utils/contracts/src/Ownable.sol
View File

@ -47,7 +47,7 @@ contract Ownable is
LibRichErrors.rrevert(LibOwnableRichErrors.TransferOwnerToZeroError());
} else {
owner = newOwner;
emit OwnershipTransferred(newOwner);
emit OwnershipTransferred(msg.sender, newOwner);
}
}

3
contracts/utils/contracts/src/interfaces/IOwnable.sol
View File

@ -22,8 +22,9 @@ pragma solidity ^0.5.9;
contract IOwnable {
/// @dev Emitted by Ownable when ownership is transferred.
/// @param previousOwner The previous owner of the contract.
/// @param newOwner The new owner of the contract.
event OwnershipTransferred(address newOwner);
event OwnershipTransferred(address indexed previousOwner, address indexed newOwner);
/// @dev Transfers ownership of the contract to a new address.
/// @param newOwner The address that will become the owner.

2
contracts/utils/test/ownable.ts
View File

@ -48,7 +48,7 @@ blockchainTests.resets('Ownable', env => {
receipt.logs,
IOwnableEvents.OwnershipTransferred,
);
expect(event).to.be.deep.eq({ newOwner: nonOwner });
expect(event).to.be.deep.eq({ previousOwner: owner, newOwner: nonOwner });
// Ensure that the owner was actually updated
const updatedOwner = await ownable.owner.callAsync();