Update permissions, kill nested terneries with fire

JoinColony · Jun 18, 2023 · b00abe5 · b00abe5
1 parent 1272c2f
commit b00abe5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 5 deletions.
diff --git a/contracts/colony/Colony.sol b/contracts/colony/Colony.sol
@@ -326,7 +326,7 @@ contract Colony is BasicMetaTransaction, Multicall, ColonyStorage, PatriciaTreeP
     sig = bytes4(keccak256("setExpenditureMetadata(uint256,uint256,uint256,string)"));
     colonyAuthority.setRoleCapability(uint8(ColonyRole.Arbitration), address(this), sig, true);
 
-    sig = bytes4(keccak256("setDomainReputationScaling(uint256,bool,uint256)"));
+    sig = bytes4(keccak256("setDomainReputationScaling(uint256,uint256)"));
     colonyAuthority.setRoleCapability(uint8(ColonyRole.Root), address(this), sig, true);
 
     // Set the default token weighting for the native token

diff --git a/contracts/colony/ColonyAuthority.sol b/contracts/colony/ColonyAuthority.sol
@@ -131,7 +131,7 @@ contract ColonyAuthority is CommonAuthority {
     addRoleCapability(ARBITRATION_ROLE, "setExpenditurePayout(uint256,uint256,uint256,uint256,address,uint256)");
 
     // Added in colony v xxxxx
-    addRoleCapability(ROOT_ROLE, "setDomainReputationScaling(uint256,bool,uint256)");
+    addRoleCapability(ROOT_ROLE, "setDomainReputationScaling(uint256,uint256)");
     addRoleCapability(ROOT_ROLE, "setReputationDecayRate(uint256,uint256)");
     addRoleCapability(ROOT_ROLE, "setReputationMiningCycleRewardReputationScaling(uint256)");
 

diff --git a/contracts/common/ScaleReputation.sol b/contracts/common/ScaleReputation.sol
@@ -28,9 +28,13 @@ contract ScaleReputation is DSMath {
     if (reputationAmount == 0 || scaleFactor == 0) { return 0; }
 
     int256 sgnAmount = (reputationAmount >= 0) ? int256(1) : -1;
-    int256 absAmount = (reputationAmount == type(int256).min)
-      ? type(int256).max // Off by one, but best we can do - probably gets capped anyway
-      : (reputationAmount >= 0) ? reputationAmount : -reputationAmount;
+    int256 absAmount;
+
+    if (reputationAmount == type(int256).min){
+      absAmount = type(int256).max; // Off by one, but best we can do - probably gets capped anyway
+    } else {
+      absAmount = reputationAmount >= 0 ? reputationAmount : -reputationAmount;
+    }
 
     // Guard against overflows during calculation with wmul
     if (type(uint256).max / scaleFactor < uint256(absAmount)) {