feat: inline/simplify receive modifiers

packages/contracts/contracts/L1/teleportr/TeleportrDeposit.sol

@@ -11,51 +11,36 @@ contract TeleportrDeposit {
     address public owner;
     uint256 public maxDepositAmount;
     uint256 public maxBalance;
-    bool public canReceiveDeposit;
 
     // Events
     event OwnerSet(address indexed oldOwner, address indexed newOwner);
     event MaxDepositAmountSet(uint256 previousAmount, uint256 newAmount);
-    event CanReceiveDepositSet(bool canReceiveDeposit);
     event MaxBalanceSet(uint256 previousBalance, uint256 newBalance);
     event BalanceWithdrawn(address indexed owner, uint256 balance);
     event EtherReceived(address indexed emitter, uint256 amount);
 
     // Modifiers
-    modifier isLowerThanMaxDepositAmount() {
-        require(msg.value <= maxDepositAmount, "Deposit amount is too big");
-        _;
-    }
     modifier isOwner() {
         require(msg.sender == owner, "Caller is not owner");
         _;
     }
-    modifier canReceive() {
-        require(canReceiveDeposit == true, "Contract is not allowed to receive ether");
-        _;
-    }
-    modifier isLowerThanMaxBalance() {
-        require(address(this).balance <= maxBalance, "Contract reached the max balance allowed");
-        _;
-    }
 
-    constructor(
-        uint256 _maxDepositAmount,
-        uint256 _maxBalance,
-        bool _canReceiveDeposit
-    ) {
+    constructor(uint256 _maxDepositAmount, uint256 _maxBalance) {
         owner = msg.sender;
         maxDepositAmount = _maxDepositAmount;
         maxBalance = _maxBalance;
-        canReceiveDeposit = _canReceiveDeposit;
         emit OwnerSet(address(0), msg.sender);
         emit MaxDepositAmountSet(0, _maxDepositAmount);
         emit MaxBalanceSet(0, _maxBalance);
-        emit CanReceiveDepositSet(_canReceiveDeposit);
     }
 
-    // Receive function which reverts if amount > maxDepositAmount and canReceiveDeposit = false
-    receive() external payable isLowerThanMaxDepositAmount canReceive isLowerThanMaxBalance {
+    // Receive function which reverts if the amount is greater than
+    // maxDepositAmount, or the amount would put the contract over its
+    // maxBalance.
+    receive() external payable {
+        require(msg.value <= maxDepositAmount, "Deposit amount is too big");
+        require(address(this).balance <= maxBalance, "Contract max balance exceeded");
+
         emit EtherReceived(msg.sender, msg.value);
     }
 
@@ -77,11 +62,6 @@ contract TeleportrDeposit {
         owner = newOwner;
     }
 
-    function setCanReceiveDeposit(bool _canReceiveDeposit) external isOwner {
-        emit CanReceiveDepositSet(_canReceiveDeposit);
-        canReceiveDeposit = _canReceiveDeposit;
-    }
-
     function setMaxBalance(uint256 _maxBalance) external isOwner {
         emit MaxBalanceSet(maxBalance, _maxBalance);
         maxBalance = _maxBalance;