Update TransferRestrictModule's moduleCheck to allow mint/burn when a…

…ddress is null

CHANGELOG.md

@@ -8,6 +8,10 @@ All notable changes to this project will be documented in this file.
 - Added **removeTimeTransferLimit** function to TimeTransferLimitsModule, allows removing time transfer limits for the given limitTime.
 - Added **batchSetTimeTransferLimit** and **batchRemoveTimeTransferLimit** functions to TimeTransferLimitsModule, allows setting and removing multiple time transfer limits at once.
 
+### Update
+
+- Updated the **moduleCheck** function to directly allow transfers when the _from or _to address is the null address, enabling mint and burn operations without additional restrictions.
+
 ## [4.1.5]
 
 ### Added

contracts/compliance/modular/modules/TransferRestrictModule.sol

@@ -172,6 +172,10 @@ contract TransferRestrictModule is AbstractModuleUpgradeable {
         uint256 /*_value*/,
         address _compliance
     ) external view override returns (bool) {
+        if (_from == address(0) || _to == address(0)) {
+            return true;
+        }
+
         if(_allowedUserAddresses[_compliance][_from]) {
             return true;
         }

test/compliances/module-transfer-restrict.test.ts

@@ -325,6 +325,38 @@ describe('Compliance Module: TransferRestrict', () => {
         expect(result).to.be.true;
       });
     });
+
+    describe('when sender is null address', () => {
+      it('should return true', async () => {
+        const context = await loadFixture(deployTransferRestrictFullSuite);
+        const to = context.accounts.anotherWallet.address;
+        const from = '0x0000000000000000000000000000000000000000';
+
+        await context.suite.compliance.callModuleFunction(
+          new ethers.utils.Interface(['function allowUser(address _userAddress)']).encodeFunctionData('allowUser', [from]),
+          context.suite.complianceModule.address,
+        );
+
+        const result = await context.suite.complianceModule.moduleCheck(from, to, 10, context.suite.compliance.address);
+        expect(result).to.be.true;
+      });
+    });
+
+    describe('when receiver is null address', () => {
+      it('should return true', async () => {
+        const context = await loadFixture(deployTransferRestrictFullSuite);
+        const to = '0x0000000000000000000000000000000000000000';
+        const from = context.accounts.anotherWallet.address;
+
+        await context.suite.compliance.callModuleFunction(
+          new ethers.utils.Interface(['function allowUser(address _userAddress)']).encodeFunctionData('allowUser', [from]),
+          context.suite.complianceModule.address,
+        );
+
+        const result = await context.suite.complianceModule.moduleCheck(from, to, 10, context.suite.compliance.address);
+        expect(result).to.be.true;
+      });
+    });
   });
 
   describe('.moduleMintAction', () => {