@0x/contracts-staking: Keep fees not associated with a pool.

0xProject · Sep 4, 2019 · 9192a2a · 9192a2a
1 parent 15b4dda
commit 9192a2a
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/contracts/staking/contracts/src/fees/MixinExchangeFees.sol b/contracts/staking/contracts/src/fees/MixinExchangeFees.sol
@@ -97,15 +97,12 @@ contract MixinExchangeFees is
             // There is a pool associated with `makerAddress`.
             // TODO(dorothy-zbornak): When we have epoch locks on delegating, we could
             // preclude pools that have no delegated stake, since they will never have
-            // stake in this epoch.
+            // stake in this epoch and are therefore not entitled to rewards.
             uint256 _feesCollectedThisEpoch = protocolFeesThisEpochByPool[poolId];
             protocolFeesThisEpochByPool[poolId] = _feesCollectedThisEpoch.safeAdd(amount);
             if (_feesCollectedThisEpoch == 0) {
                 activePoolsThisEpoch.push(poolId);
             }
-        } else {
-            // No pool associated with `makerAddress`. Refund the fee.
-            msg.sender.transfer(amount);
         }
     }