diff --git a/contracts/accountants/HealthCheckAccountant.sol b/contracts/accountants/HealthCheckAccountant.sol index 398af79..8116870 100644 --- a/contracts/accountants/HealthCheckAccountant.sol +++ b/contracts/accountants/HealthCheckAccountant.sol @@ -565,7 +565,7 @@ contract HealthCheckAccountant { * @dev This function provides the maximum performance fee that the accountant can charge. * @return The maximum performance fee. */ - function performanceFeeThreshold() external view virtual returns (uint16) { + function performanceFeeThreshold() external pure virtual returns (uint16) { return PERFORMANCE_FEE_THRESHOLD; } @@ -574,7 +574,7 @@ contract HealthCheckAccountant { * @dev This function provides the maximum management fee that the accountant can charge. * @return The maximum management fee. */ - function managementFeeThreshold() external view virtual returns (uint16) { + function managementFeeThreshold() external pure virtual returns (uint16) { return MANAGEMENT_FEE_THRESHOLD; } } diff --git a/contracts/accountants/RefundAccountant.sol b/contracts/accountants/RefundAccountant.sol index 3fd714e..8eb5738 100644 --- a/contracts/accountants/RefundAccountant.sol +++ b/contracts/accountants/RefundAccountant.sol @@ -64,7 +64,12 @@ contract RefundAccountant is HealthCheckAccountant { address strategy, uint256 gain, uint256 loss - ) public override returns (uint256 totalFees, uint256 totalRefunds) { + ) + public + virtual + override + returns (uint256 totalFees, uint256 totalRefunds) + { (totalFees, totalRefunds) = super.report(strategy, gain, loss); Refund memory refundConfig = refund[msg.sender][strategy]; @@ -99,7 +104,7 @@ contract RefundAccountant is HealthCheckAccountant { address _strategy, bool _refund, uint256 _amount - ) external onlyFeeManager { + ) external virtual onlyFeeManager { require(vaults[_vault], "not added"); require( IVault(_vault).strategies(_strategy).activation != 0,