diff --git a/contracts/deploy/upgrade-all.ts b/contracts/deploy/upgrade-all.ts index 74433c413..dfd0d882c 100644 --- a/contracts/deploy/upgrade-all.ts +++ b/contracts/deploy/upgrade-all.ts @@ -98,11 +98,11 @@ const deployUpgradeAll: DeployFunction = async (hre: HardhatRuntimeEnvironment) await upgrade(disputeKitShutter, "reinitialize", [wETH.address]); await upgrade(disputeKitGated, "reinitialize", [wETH.address]); await upgrade(disputeKitGatedShutter, "reinitialize", [wETH.address]); - await upgrade(disputeTemplateRegistry, "initialize2", []); - await upgrade(evidence, "initialize2", []); + await upgrade(disputeTemplateRegistry, "reinitialize", []); + await upgrade(evidence, "reinitialize", []); await upgrade(core, "reinitialize", [wETH.address]); - await upgrade(policyRegistry, "initialize2", []); - await upgrade(sortition, "initialize4", []); + await upgrade(policyRegistry, "reinitialize", []); + await upgrade(sortition, "reinitialize", []); }; deployUpgradeAll.tags = ["UpgradeAll"]; diff --git a/contracts/src/arbitration/DisputeTemplateRegistry.sol b/contracts/src/arbitration/DisputeTemplateRegistry.sol index b9354aa0a..8d3d68d58 100644 --- a/contracts/src/arbitration/DisputeTemplateRegistry.sol +++ b/contracts/src/arbitration/DisputeTemplateRegistry.sol @@ -8,7 +8,7 @@ import "./interfaces/IDisputeTemplateRegistry.sol"; /// @title Dispute Template Registry /// @dev A contract to maintain a registry of dispute templates. contract DisputeTemplateRegistry is IDisputeTemplateRegistry, UUPSProxiable, Initializable { - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -40,14 +40,10 @@ contract DisputeTemplateRegistry is IDisputeTemplateRegistry, UUPSProxiable, Ini /// @dev Initializer /// @param _owner Owner of the contract. - function initialize(address _owner) external reinitializer(1) { + function initialize(address _owner) external initializer { owner = _owner; } - function initialize2() external reinitializer(2) { - // NOP - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/KlerosCore.sol b/contracts/src/arbitration/KlerosCore.sol index 912a57269..4dd30f61b 100644 --- a/contracts/src/arbitration/KlerosCore.sol +++ b/contracts/src/arbitration/KlerosCore.sol @@ -8,7 +8,7 @@ import {KlerosCoreBase, IDisputeKit, ISortitionModule, IERC20} from "./KlerosCor /// Core arbitrator contract for Kleros v2. /// Note that this contract trusts the PNK token, the dispute kit and the sortition module contracts. contract KlerosCore is KlerosCoreBase { - string public constant override version = "0.10.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Constructor * // @@ -43,7 +43,7 @@ contract KlerosCore is KlerosCoreBase { bytes memory _sortitionExtraData, ISortitionModule _sortitionModuleAddress, address _wNative - ) external reinitializer(1) { + ) external initializer { __KlerosCoreBase_initialize( _owner, _guardian, @@ -59,10 +59,6 @@ contract KlerosCore is KlerosCoreBase { ); } - function reinitialize(address _wNative) external reinitializer(6) { - wNative = _wNative; - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/KlerosCoreNeo.sol b/contracts/src/arbitration/KlerosCoreNeo.sol index ebfd1bb72..8f0a413b3 100644 --- a/contracts/src/arbitration/KlerosCoreNeo.sol +++ b/contracts/src/arbitration/KlerosCoreNeo.sol @@ -9,7 +9,7 @@ import "@openzeppelin/contracts/token/ERC721/IERC721.sol"; /// Core arbitrator contract for Kleros v2. /// Note that this contract trusts the PNK token, the dispute kit and the sortition module contracts. contract KlerosCoreNeo is KlerosCoreBase { - string public constant override version = "0.10.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -70,10 +70,6 @@ contract KlerosCoreNeo is KlerosCoreBase { jurorNft = _jurorNft; } - function reinitialize(address _wNative) external reinitializer(6) { - wNative = _wNative; - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/PolicyRegistry.sol b/contracts/src/arbitration/PolicyRegistry.sol index acea8d01b..9c8d09f3e 100644 --- a/contracts/src/arbitration/PolicyRegistry.sol +++ b/contracts/src/arbitration/PolicyRegistry.sol @@ -7,7 +7,7 @@ import "../proxy/Initializable.sol"; /// @title PolicyRegistry /// @dev A contract to maintain a policy for each court. contract PolicyRegistry is UUPSProxiable, Initializable { - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Events * // @@ -47,14 +47,10 @@ contract PolicyRegistry is UUPSProxiable, Initializable { /// @dev Constructs the `PolicyRegistry` contract. /// @param _owner The owner's address. - function initialize(address _owner) external reinitializer(1) { + function initialize(address _owner) external initializer { owner = _owner; } - function initialize2() external reinitializer(2) { - // NOP - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/SortitionModule.sol b/contracts/src/arbitration/SortitionModule.sol index ae89335b0..bd0f6e007 100644 --- a/contracts/src/arbitration/SortitionModule.sol +++ b/contracts/src/arbitration/SortitionModule.sol @@ -7,7 +7,7 @@ import {SortitionModuleBase, KlerosCore, IRNG} from "./SortitionModuleBase.sol"; /// @title SortitionModule /// @dev A factory of trees that keeps track of staked values for sortition. contract SortitionModule is SortitionModuleBase { - string public constant override version = "0.9.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Constructor * // @@ -30,14 +30,10 @@ contract SortitionModule is SortitionModuleBase { uint256 _minStakingTime, uint256 _maxDrawingTime, IRNG _rng - ) external reinitializer(1) { + ) external initializer { __SortitionModuleBase_initialize(_owner, _core, _minStakingTime, _maxDrawingTime, _rng); } - function initialize4() external reinitializer(4) { - // NOP - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/SortitionModuleNeo.sol b/contracts/src/arbitration/SortitionModuleNeo.sol index 4d28f97b5..66dbba7e5 100644 --- a/contracts/src/arbitration/SortitionModuleNeo.sol +++ b/contracts/src/arbitration/SortitionModuleNeo.sol @@ -7,7 +7,7 @@ import {SortitionModuleBase, KlerosCore, IRNG, StakingResult} from "./SortitionM /// @title SortitionModuleNeo /// @dev A factory of trees that keeps track of staked values for sortition. contract SortitionModuleNeo is SortitionModuleBase { - string public constant override version = "0.9.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -48,10 +48,6 @@ contract SortitionModuleNeo is SortitionModuleBase { maxTotalStaked = _maxTotalStaked; } - function initialize4() external reinitializer(4) { - // NOP - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/devtools/KlerosCoreRuler.sol b/contracts/src/arbitration/devtools/KlerosCoreRuler.sol index 9350ebd57..42a428299 100644 --- a/contracts/src/arbitration/devtools/KlerosCoreRuler.sol +++ b/contracts/src/arbitration/devtools/KlerosCoreRuler.sol @@ -13,7 +13,7 @@ import "../../libraries/Constants.sol"; contract KlerosCoreRuler is IArbitratorV2, UUPSProxiable, Initializable { using SafeERC20 for IERC20; - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Enums / Structs * // @@ -175,11 +175,7 @@ contract KlerosCoreRuler is IArbitratorV2, UUPSProxiable, Initializable { /// @param _owner The owner's address. /// @param _pinakion The address of the token contract. /// @param _courtParameters Numeric parameters of General court (minStake, alpha, feeForJuror and jurorsForCourtJump respectively). - function initialize( - address _owner, - IERC20 _pinakion, - uint256[4] memory _courtParameters - ) external reinitializer(1) { + function initialize(address _owner, IERC20 _pinakion, uint256[4] memory _courtParameters) external initializer { owner = _owner; pinakion = _pinakion; @@ -210,10 +206,6 @@ contract KlerosCoreRuler is IArbitratorV2, UUPSProxiable, Initializable { ); } - function initialize2() external reinitializer(2) { - // NOP - } - // ************************************* // // * Governance * // // ************************************* // diff --git a/contracts/src/arbitration/dispute-kits/DisputeKitClassic.sol b/contracts/src/arbitration/dispute-kits/DisputeKitClassic.sol index af89d2816..dd2edbab8 100644 --- a/contracts/src/arbitration/dispute-kits/DisputeKitClassic.sol +++ b/contracts/src/arbitration/dispute-kits/DisputeKitClassic.sol @@ -11,7 +11,7 @@ import {DisputeKitClassicBase, KlerosCore} from "./DisputeKitClassicBase.sol"; /// - an incentive system: equal split between coherent votes, /// - an appeal system: fund 2 choices only, vote on any choice. contract DisputeKitClassic is DisputeKitClassicBase { - string public constant override version = "0.13.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Constructor * // @@ -32,14 +32,10 @@ contract DisputeKitClassic is DisputeKitClassicBase { KlerosCore _core, address _wNative, uint256 _jumpDisputeKitID - ) external reinitializer(1) { + ) external initializer { __DisputeKitClassicBase_initialize(_owner, _core, _wNative, _jumpDisputeKitID); } - function reinitialize(uint256 _jumpDisputeKitID) external reinitializer(10) { - jumpDisputeKitID = _jumpDisputeKitID; - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/dispute-kits/DisputeKitGated.sol b/contracts/src/arbitration/dispute-kits/DisputeKitGated.sol index 9097a8fd3..8abfdecac 100644 --- a/contracts/src/arbitration/dispute-kits/DisputeKitGated.sol +++ b/contracts/src/arbitration/dispute-kits/DisputeKitGated.sol @@ -27,7 +27,7 @@ interface IBalanceHolderERC1155 { /// - an incentive system: equal split between coherent votes, /// - an appeal system: fund 2 choices only, vote on any choice. contract DisputeKitGated is DisputeKitClassicBase { - string public constant override version = "0.13.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Constructor * // @@ -48,14 +48,10 @@ contract DisputeKitGated is DisputeKitClassicBase { KlerosCore _core, address _wNative, uint256 _jumpDisputeKitID - ) external reinitializer(1) { + ) external initializer { __DisputeKitClassicBase_initialize(_owner, _core, _wNative, _jumpDisputeKitID); } - function reinitialize(uint256 _jumpDisputeKitID) external reinitializer(10) { - jumpDisputeKitID = _jumpDisputeKitID; - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/dispute-kits/DisputeKitGatedShutter.sol b/contracts/src/arbitration/dispute-kits/DisputeKitGatedShutter.sol index 67d9ddbea..573f81f11 100644 --- a/contracts/src/arbitration/dispute-kits/DisputeKitGatedShutter.sol +++ b/contracts/src/arbitration/dispute-kits/DisputeKitGatedShutter.sol @@ -28,7 +28,7 @@ interface IBalanceHolderERC1155 { /// - an incentive system: equal split between coherent votes, /// - an appeal system: fund 2 choices only, vote on any choice. contract DisputeKitGatedShutter is DisputeKitClassicBase { - string public constant override version = "0.13.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -82,14 +82,10 @@ contract DisputeKitGatedShutter is DisputeKitClassicBase { KlerosCore _core, address _wNative, uint256 _jumpDisputeKitID - ) external reinitializer(1) { + ) external initializer { __DisputeKitClassicBase_initialize(_owner, _core, _wNative, _jumpDisputeKitID); } - function reinitialize(uint256 _jumpDisputeKitID) external reinitializer(10) { - jumpDisputeKitID = _jumpDisputeKitID; - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/dispute-kits/DisputeKitShutter.sol b/contracts/src/arbitration/dispute-kits/DisputeKitShutter.sol index b8bef461f..bbcb28b24 100644 --- a/contracts/src/arbitration/dispute-kits/DisputeKitShutter.sol +++ b/contracts/src/arbitration/dispute-kits/DisputeKitShutter.sol @@ -12,7 +12,7 @@ import {DisputeKitClassicBase, KlerosCore} from "./DisputeKitClassicBase.sol"; /// - an incentive system: equal split between coherent votes, /// - an appeal system: fund 2 choices only, vote on any choice. contract DisputeKitShutter is DisputeKitClassicBase { - string public constant override version = "0.13.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -66,14 +66,10 @@ contract DisputeKitShutter is DisputeKitClassicBase { KlerosCore _core, address _wNative, uint256 _jumpDisputeKitID - ) external reinitializer(1) { + ) external initializer { __DisputeKitClassicBase_initialize(_owner, _core, _wNative, _jumpDisputeKitID); } - function reinitialize(uint256 _jumpDisputeKitID) external reinitializer(10) { - jumpDisputeKitID = _jumpDisputeKitID; - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/dispute-kits/DisputeKitSybilResistant.sol b/contracts/src/arbitration/dispute-kits/DisputeKitSybilResistant.sol index 66553f02a..10cc67c5c 100644 --- a/contracts/src/arbitration/dispute-kits/DisputeKitSybilResistant.sol +++ b/contracts/src/arbitration/dispute-kits/DisputeKitSybilResistant.sol @@ -18,7 +18,7 @@ interface IProofOfHumanity { /// - an incentive system: equal split between coherent votes, /// - an appeal system: fund 2 choices only, vote on any choice. contract DisputeKitSybilResistant is DisputeKitClassicBase { - string public constant override version = "0.13.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -47,7 +47,7 @@ contract DisputeKitSybilResistant is DisputeKitClassicBase { IProofOfHumanity _poh, address _wNative, uint256 _jumpDisputeKitID - ) external reinitializer(1) { + ) external initializer { __DisputeKitClassicBase_initialize(_owner, _core, _wNative, _jumpDisputeKitID); poh = _poh; singleDrawPerJuror = true; diff --git a/contracts/src/arbitration/evidence/EvidenceModule.sol b/contracts/src/arbitration/evidence/EvidenceModule.sol index d1eb5f99c..8321e55ba 100644 --- a/contracts/src/arbitration/evidence/EvidenceModule.sol +++ b/contracts/src/arbitration/evidence/EvidenceModule.sol @@ -9,7 +9,7 @@ import "../../proxy/Initializable.sol"; /// @title Evidence Module contract EvidenceModule is IEvidence, Initializable, UUPSProxiable { - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Storage * // @@ -37,14 +37,10 @@ contract EvidenceModule is IEvidence, Initializable, UUPSProxiable { /// @dev Initializer. /// @param _owner The owner's address. - function initialize(address _owner) external reinitializer(1) { + function initialize(address _owner) external initializer { owner = _owner; } - function initialize2() external reinitializer(2) { - // NOP - } - // ************************ // // * Governance * // // ************************ // diff --git a/contracts/src/arbitration/university/KlerosCoreUniversity.sol b/contracts/src/arbitration/university/KlerosCoreUniversity.sol index b2519fdd7..afe7fc4d4 100644 --- a/contracts/src/arbitration/university/KlerosCoreUniversity.sol +++ b/contracts/src/arbitration/university/KlerosCoreUniversity.sol @@ -15,7 +15,7 @@ import "../../libraries/Constants.sol"; contract KlerosCoreUniversity is IArbitratorV2, UUPSProxiable, Initializable { using SafeERC20 for IERC20; - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Enums / Structs * // @@ -207,7 +207,7 @@ contract KlerosCoreUniversity is IArbitratorV2, UUPSProxiable, Initializable { uint256[4] memory _courtParameters, uint256[4] memory _timesPerPeriod, ISortitionModuleUniversity _sortitionModuleAddress - ) external reinitializer(1) { + ) external initializer { owner = _owner; instructor = _instructor; pinakion = _pinakion; diff --git a/contracts/src/arbitration/university/SortitionModuleUniversity.sol b/contracts/src/arbitration/university/SortitionModuleUniversity.sol index 63720eb18..66e815e4a 100644 --- a/contracts/src/arbitration/university/SortitionModuleUniversity.sol +++ b/contracts/src/arbitration/university/SortitionModuleUniversity.sol @@ -12,7 +12,7 @@ import "../../libraries/Constants.sol"; /// @title SortitionModuleUniversity /// @dev An adapted version of the SortitionModule contract for educational purposes. contract SortitionModuleUniversity is ISortitionModuleUniversity, UUPSProxiable, Initializable { - string public constant override version = "0.8.0"; + string public constant override version = "2.0.0"; // ************************************* // // * Enums / Structs * // @@ -87,7 +87,7 @@ contract SortitionModuleUniversity is ISortitionModuleUniversity, UUPSProxiable, /// @dev Initializer (constructor equivalent for upgradable contracts). /// @param _core The KlerosCore. - function initialize(address _owner, KlerosCoreUniversity _core) external reinitializer(1) { + function initialize(address _owner, KlerosCoreUniversity _core) external initializer { owner = _owner; core = _core; } diff --git a/contracts/src/gateway/ForeignGateway.sol b/contracts/src/gateway/ForeignGateway.sol index 712790d0d..366f3e6f0 100644 --- a/contracts/src/gateway/ForeignGateway.sol +++ b/contracts/src/gateway/ForeignGateway.sol @@ -81,7 +81,7 @@ contract ForeignGateway is IForeignGateway, UUPSProxiable, Initializable { address _veaOutbox, uint256 _homeChainID, address _homeGateway - ) external reinitializer(1) { + ) external initializer { owner = _owner; veaOutbox = _veaOutbox; homeChainID = _homeChainID; diff --git a/contracts/src/gateway/HomeGateway.sol b/contracts/src/gateway/HomeGateway.sol index d0062dc52..2a23913ef 100644 --- a/contracts/src/gateway/HomeGateway.sol +++ b/contracts/src/gateway/HomeGateway.sol @@ -72,7 +72,7 @@ contract HomeGateway is IHomeGateway, UUPSProxiable, Initializable { uint256 _foreignChainID, address _foreignGateway, IERC20 _feeToken - ) external reinitializer(1) { + ) external initializer { owner = _owner; arbitrator = _arbitrator; veaInbox = _veaInbox; diff --git a/contracts/src/proxy/Initializable.sol b/contracts/src/proxy/Initializable.sol index 47769a9f2..7bdf08eb4 100644 --- a/contracts/src/proxy/Initializable.sol +++ b/contracts/src/proxy/Initializable.sol @@ -93,7 +93,7 @@ abstract contract Initializable { * @dev A modifier that defines a protected initializer function that can be invoked at most once. In its scope, * `onlyInitializing` functions can be used to initialize parent contracts. * - * Similar to `reinitializer(1)`, except that functions marked with `initializer` can be nested in the context of a + * Similar to `initializer()`, except that functions marked with `initializer` can be nested in the context of a * constructor. * * Emits an {Initialized} event. diff --git a/contracts/src/proxy/mock/by-inheritance/UpgradedByInheritance.sol b/contracts/src/proxy/mock/by-inheritance/UpgradedByInheritance.sol index c5f796995..b443051a8 100644 --- a/contracts/src/proxy/mock/by-inheritance/UpgradedByInheritance.sol +++ b/contracts/src/proxy/mock/by-inheritance/UpgradedByInheritance.sol @@ -20,7 +20,7 @@ contract UpgradedByInheritanceV1 is UUPSProxiable, Initializable { _disableInitializers(); } - function initialize(address _owner) external virtual reinitializer(1) { + function initialize(address _owner) external virtual initializer { owner = _owner; counter = 1; } @@ -61,7 +61,7 @@ contract UpgradedByInheritanceV3Bad is UpgradedByInheritanceV2 { _disableInitializers(); } - function initializeV3() external reinitializer(1) { + function initializeV3() external initializer { // Wrong reinitializer version. } } diff --git a/contracts/src/proxy/mock/by-rewrite/UpgradedByRewrite.sol b/contracts/src/proxy/mock/by-rewrite/UpgradedByRewrite.sol index f5e5eae77..d8930796d 100644 --- a/contracts/src/proxy/mock/by-rewrite/UpgradedByRewrite.sol +++ b/contracts/src/proxy/mock/by-rewrite/UpgradedByRewrite.sol @@ -23,7 +23,7 @@ contract UpgradedByRewrite is UUPSProxiable, Initializable { _disableInitializers(); } - function initialize(address _owner) external virtual reinitializer(1) { + function initialize(address _owner) external virtual initializer { owner = _owner; counter = 1; }