@ensuro/core 2.4.0 → 2.5.0

package/contracts/Reserve.sol

@@ -152,6 +152,11 @@ abstract contract Reserve is PolicyPoolComponent {
         } else {
           _assetEarnings(abi.decode(result, (int256)));
         }
+        /**
+         * WARNING: if you are doing a forced replacement of the AM and you want the new AM
+         * to inherit the storage (just fixing the code), make sure the code of the new AM doesn't clean
+         * the storage in the connect() method (as it is the recommended practice in normal changes of AM).
+         */
       } else {
         bytes memory result = am.functionDelegateCall(
           abi.encodeWithSelector(IAssetManager.deinvestAll.selector)

package/contracts/TimeScaled.sol

@@ -116,6 +116,10 @@ library TimeScaled {
     int256 amount,
     uint256 interestRate
   ) internal {
+    if (scaledAmount.amount == 0) {
+      add(scaledAmount, uint256(amount), interestRate);
+      return;
+    }
     updateScale(scaledAmount, interestRate);
     uint256 newScaledAmount = uint256(int256(getScaledAmount(scaledAmount, interestRate)) + amount);
     scaledAmount.scale = newScaledAmount

package/contracts/interfaces/IAssetManager.sol

@@ -38,6 +38,11 @@ interface IAssetManager is IERC165 {
 
   /**
    * @dev Function called when an asset manager is plugged into a reserve. Useful for initialization tasks
+   *
+   * Since the asset manager for a reserve can be changed and they use the stoage of the reserve contract, you
+   * can't assume the storage starts clean (all zeros). This is because a previous AM, using the same hash for the
+   * diamondStorage might have left some data. So, in the connect method you should initialize the state setting to
+   * zero what's expected to be zero.
    */
   function connect() external;
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ensuro/core",
   "description": "Ensuro - Decentralized insurance protocol",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "files": [
     "**/*.sol",
     "/build/contracts/*.json",