From 1c78092b4fd01f15dc73c04b6b0e7604bce59878 Mon Sep 17 00:00:00 2001
From: nithin <nithin@fragments.org>
Date: Tue, 22 Feb 2022 09:35:39 -0500
Subject: [PATCH 1/2] updated maturity condition

---
 contracts/ACash.sol | 28 +++++++++++++---------------
 1 file changed, 13 insertions(+), 15 deletions(-)

diff --git a/contracts/ACash.sol b/contracts/ACash.sol
index edb15de2..c184983d 100644
--- a/contracts/ACash.sol
+++ b/contracts/ACash.sol
@@ -41,10 +41,10 @@ contract ACash is ERC20, Initializable, Ownable {
     AddressQueue.Queue public bondQueue;
 
     // the minimum maturity time in seconds for a bond below which it gets removed from the bond queue
-    uint256 private _minQueueMaturiySec;
+    uint256 public minMaturiySec;
 
     // the maximum maturity time in seconds for a bond above which it can't get added into the bond queue
-    uint256 private _maxQueueMaturiySec;
+    uint256 public maxMaturiySec;
 
     //---- ERC-20 parameters
     uint8 private immutable _decimals;
@@ -98,7 +98,7 @@ contract ACash is ERC20, Initializable, Ownable {
         uint256 mintAmt = 0;
         for (uint256 i = 0; i < trancheCount; i++) {
             uint256 trancheYield = _trancheYields[configHash][i];
-            if(trancheYield == 0){
+            if (trancheYield == 0) {
                 continue;
             }
 
@@ -121,8 +121,7 @@ contract ACash is ERC20, Initializable, Ownable {
     function advanceMintBond(IBondController newBond) public {
         require(address(newBond) != bondQueue.head(), "New bond already in queue");
         require(bondIssuer.isInstance(newBond), "Expect new bond to be minted by the minter");
-        require(newBond.maturityDate() > minQueueMaturityDate(), "New bond matures too soon");
-        require(newBond.maturityDate() <= maxQueueMaturityDate(), "New bond matures too late");
+        require(isActiveBond(newBond), "New bond not active");
 
         bondQueue.enqueue(address(newBond));
     }
@@ -133,7 +132,7 @@ contract ACash is ERC20, Initializable, Ownable {
         while (true) {
             IBondController latestBond = IBondController(bondQueue.tail());
 
-            if (address(latestBond) == address(0) || latestBond.maturityDate() > minQueueMaturityDate()) {
+            if (address(latestBond) == address(0) || isActiveBond(latestBond)) {
                 break;
             }
 
@@ -187,21 +186,20 @@ contract ACash is ERC20, Initializable, Ownable {
         feeStrategy = feeStrategy_;
     }
 
-    function setTolarableBondMaturiy(uint256 minQueueMaturiySec, uint256 maxQueueMaturiySec) external onlyOwner {
-        _minQueueMaturiySec = minQueueMaturiySec;
-        _maxQueueMaturiySec = maxQueueMaturiySec;
+    function setTolarableBondMaturiy(uint256 minMaturiySec_, uint256 maxMaturiySec_) external onlyOwner {
+        minMaturiySec = minMaturiySec_;
+        maxMaturiySec = maxMaturiySec_;
     }
 
     function setTrancheYields(bytes32 configHash, uint256[] memory yields) external onlyOwner {
         _trancheYields[configHash] = yields;
     }
 
-    function minQueueMaturityDate() public view returns (uint256) {
-        return block.timestamp + _minQueueMaturiySec;
-    }
-
-    function maxQueueMaturityDate() public view returns (uint256) {
-        return block.timestamp + _maxQueueMaturiySec;
+    // bond's maturity is within bounds
+    // only active bonds can be added to the queue. If a bond is inactive it gets kicked from the queue ..
+    function isActiveBond(IBondController bond) public view returns (bool) {
+        return (bond.maturityDate() > block.timestamp + minMaturiySec &&
+            bond.maturityDate() <= block.timestamp + maxMaturiySec);
     }
 
     /*

From 29b0eb15703582b1754b5899922f697f2b6c963c Mon Sep 17 00:00:00 2001
From: aalavandhan1984 <6264334+aalavandhan@users.noreply.github.com>
Date: Tue, 22 Feb 2022 15:35:24 -0500
Subject: [PATCH 2/2] code review comments

---
 contracts/ACash.sol | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/contracts/ACash.sol b/contracts/ACash.sol
index c184983d..1fa6619f 100644
--- a/contracts/ACash.sol
+++ b/contracts/ACash.sol
@@ -186,7 +186,7 @@ contract ACash is ERC20, Initializable, Ownable {
         feeStrategy = feeStrategy_;
     }
 
-    function setTolarableBondMaturiy(uint256 minMaturiySec_, uint256 maxMaturiySec_) external onlyOwner {
+    function setTolerableBondMaturiy(uint256 minMaturiySec_, uint256 maxMaturiySec_) external onlyOwner {
         minMaturiySec = minMaturiySec_;
         maxMaturiySec = maxMaturiySec_;
     }
@@ -198,8 +198,8 @@ contract ACash is ERC20, Initializable, Ownable {
     // bond's maturity is within bounds
     // only active bonds can be added to the queue. If a bond is inactive it gets kicked from the queue ..
     function isActiveBond(IBondController bond) public view returns (bool) {
-        return (bond.maturityDate() > block.timestamp + minMaturiySec &&
-            bond.maturityDate() <= block.timestamp + maxMaturiySec);
+        return (bond.maturityDate() >= block.timestamp + minMaturiySec &&
+            bond.maturityDate() < block.timestamp + maxMaturiySec);
     }
 
     /*