NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #424 from eugene-babichenko/transfer-ownership-event

Browse Source 
Add OwnershipTransferred event to Ownable contract and its derivatives
This commit is contained in:
Francisco Giordano
2017-09-07 11:59:29 -03:00
committed by GitHub
parent fdc8fbaa45 5035718e4b
commit 84be318ca9
3 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
contracts/ownership/Claimable.sol
View File

@ -32,6 +32,7 @@ contract Claimable is Ownable {
* @dev Allows the pendingOwner address to finalize the transfer.
*/
function claimOwnership() onlyPendingOwner {
OwnershipTransferred(owner, pendingOwner);
owner = pendingOwner;
pendingOwner = 0x0;
}

1
contracts/ownership/DelayedClaimable.sol
View File

@ -33,6 +33,7 @@ contract DelayedClaimable is Claimable {
*/
function claimOwnership() onlyPendingOwner {
require((block.number <= end) && (block.number >= start));
OwnershipTransferred(owner, pendingOwner);
owner = pendingOwner;
pendingOwner = 0x0;
end = 0;

4
contracts/ownership/Ownable.sol
View File

@ -10,6 +10,9 @@ contract Ownable {
address public owner;
event OwnershipTransferred(address indexed previousOwner, address indexed newOwner);
/**
* @dev The Ownable constructor sets the original `owner` of the contract to the sender
* account.
@ -34,6 +37,7 @@ contract Ownable {
*/
function transferOwnership(address newOwner) onlyOwner {
require(newOwner != address(0));
OwnershipTransferred(owner, newOwner);
owner = newOwner;
}