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

Add OwnershipTransferred event to Ownable contract and its derivatives
OpenZeppelin · Sep 7, 2017 · 84be318 · 84be318
2 parents fdc8fba + 5035718
commit 84be318
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/contracts/ownership/Claimable.sol b/contracts/ownership/Claimable.sol
@@ -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;
   }

diff --git a/contracts/ownership/DelayedClaimable.sol b/contracts/ownership/DelayedClaimable.sol
@@ -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;

diff --git a/contracts/ownership/Ownable.sol b/contracts/ownership/Ownable.sol
@@ -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;
   }