Slightly harder changes following review

Author: area

contracts/colonyNetwork/ColonyNetwork.sol

@@ -195,12 +195,10 @@ contract ColonyNetwork is BasicMetaTransaction, ColonyNetworkStorage, Multicall,
   }
 
   function setColonyReputationDecayRateFromBridge(address _colony, uint256 _numerator, uint256 _denominator) public always
+    knownBridge(msgSender())
     onlyMiningChain
  {
-    uint256 bridgeChainId = bridgeData[msgSender()].chainId;
-    require(bridgeChainId != 0, "colony-network-not-known-bridge");
-
-    setColonyReputationDecayRateInternal(bridgeChainId, _colony, _numerator, _denominator);
+    setColonyReputationDecayRateInternal(bridgeData[msgSender()].chainId, _colony, _numerator, _denominator);
   }
 
   function setColonyReputationDecayRateInternal(uint256 _chainId, address _colony, uint256 _numerator, uint256 _denominator) internal {

contracts/colonyNetwork/ColonyNetworkMining.sol

@@ -86,16 +86,14 @@ contract ColonyNetworkMining is ColonyNetworkStorage, ScaleReputation {
 
   // Well this is a weird hack to need
   function newAddressArray() pure internal returns (address[] memory) {}
-  function setReputationRootHashFromBridge(bytes32 newHash, uint256 newNLeaves) onlyNotMiningChain stoppable public {
-    require(bridgeData[msgSender()].chainId != 0, "colony-network-not-known-bridge");
+  function setReputationRootHashFromBridge(bytes32 newHash, uint256 newNLeaves) onlyNotMiningChain stoppable knownBridge(msgSender()) public {
     reputationRootHash = newHash;
     reputationRootHashNLeaves = newNLeaves;
 
     emit ReputationRootHashSet(newHash, newNLeaves, newAddressArray(), 0);
   }
 
-  function bridgeCurrentRootHash(address _bridgeAddress) onlyMiningChain stoppable public {
-    require(bridgeData[_bridgeAddress].chainId != 0, "colony-network-not-known-bridge");
+  function bridgeCurrentRootHash(address _bridgeAddress) onlyMiningChain stoppable knownBridge(_bridgeAddress) public {
     bytes memory payload = abi.encodePacked(
       bridgeData[_bridgeAddress].setReputationRootHashBefore,
       abi.encodeWithSignature("setReputationRootHashFromBridge(bytes32,uint256)", reputationRootHash, reputationRootHashNLeaves),

contracts/colonyNetwork/ColonyNetworkStorage.sol

@@ -165,6 +165,11 @@ contract ColonyNetworkStorage is ColonyNetworkDataTypes, DSMath, CommonStorage,
     _;
   }
 
+  modifier knownBridge(address _bridgeAddress) {
+    require(bridgeData[_bridgeAddress].chainId != 0, "colony-network-not-known-bridge");
+    _;
+  }
+
   // Internal functions
 
   function toRootSkillId(uint256 _chainId) internal pure returns (uint256) {