From 16a5475d2407f42079e149479dbf35b510f6fb1f Mon Sep 17 00:00:00 2001
From: Amir Bandeali <abandeali1@gmail.com>
Date: Sun, 27 Oct 2019 11:40:28 -0700
Subject: [PATCH] Add detachProtocolFeeCollector function (in order to have a
 separate timelock for detaching)

---
 .../contracts/src/MixinProtocolFees.sol         | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/contracts/exchange/contracts/src/MixinProtocolFees.sol b/contracts/exchange/contracts/src/MixinProtocolFees.sol
index da88503c1f..c6dc31debf 100644
--- a/contracts/exchange/contracts/src/MixinProtocolFees.sol
+++ b/contracts/exchange/contracts/src/MixinProtocolFees.sol
@@ -50,6 +50,23 @@ contract MixinProtocolFees is
     function setProtocolFeeCollectorAddress(address updatedProtocolFeeCollector)
         external
         onlyOwner
+    {
+        _setProtocolFeeCollectorAddress(updatedProtocolFeeCollector);
+    }
+
+    /// @dev Sets the protocolFeeCollector contract address to 0.
+    ///      Only callable by owner.
+    function detachProtocolFeeCollector()
+        external
+        onlyOwner
+    {
+        _setProtocolFeeCollectorAddress(address(0));
+    }
+
+    /// @dev Sets the protocolFeeCollector address and emits an event.
+    /// @param updatedProtocolFeeCollector The updated protocolFeeCollector contract address.
+    function _setProtocolFeeCollectorAddress(address updatedProtocolFeeCollector)
+        internal
     {
         emit ProtocolFeeCollectorAddress(protocolFeeCollector, updatedProtocolFeeCollector);
         protocolFeeCollector = updatedProtocolFeeCollector;