From b59469c1511bcec93d570cbc769f0691938bf1bf Mon Sep 17 00:00:00 2001 From: Yevhenii Babichenko Date: Tue, 5 Sep 2017 18:32:07 +0300 Subject: [PATCH] fire OwnershipTransferred event when the owner of an Ownable contract instance actually changed --- contracts/ownership/Claimable.sol | 1 + contracts/ownership/DelayedClaimable.sol | 1 + contracts/ownership/Ownable.sol | 4 ++++ 3 files changed, 6 insertions(+) diff --git a/contracts/ownership/Claimable.sol b/contracts/ownership/Claimable.sol index 59c2158a2..ae8080270 100644 --- a/contracts/ownership/Claimable.sol +++ b/contracts/ownership/Claimable.sol @@ -34,5 +34,6 @@ contract Claimable is Ownable { function claimOwnership() onlyPendingOwner { owner = pendingOwner; pendingOwner = 0x0; + OwnershipTransferred(owner); } } diff --git a/contracts/ownership/DelayedClaimable.sol b/contracts/ownership/DelayedClaimable.sol index f83ed5046..ddffd0d0e 100644 --- a/contracts/ownership/DelayedClaimable.sol +++ b/contracts/ownership/DelayedClaimable.sol @@ -36,6 +36,7 @@ contract DelayedClaimable is Claimable { owner = pendingOwner; pendingOwner = 0x0; end = 0; + OwnershipTransferred(owner); } } diff --git a/contracts/ownership/Ownable.sol b/contracts/ownership/Ownable.sol index fcb3ef91f..666747362 100644 --- a/contracts/ownership/Ownable.sol +++ b/contracts/ownership/Ownable.sol @@ -10,6 +10,9 @@ contract Ownable { address public owner; + event OwnershipTransferred(address indexed newOwner); + + /** * @dev The Ownable constructor sets the original `owner` of the contract to the sender * account. @@ -35,6 +38,7 @@ contract Ownable { function transferOwnership(address newOwner) onlyOwner { require(newOwner != address(0)); owner = newOwner; + OwnershipTransferred(newOwner); } }