BasicSpell.doCutRewardsFee uses depositFee instead of withdraw fee
BasicSpell.doCutRewardsFee uses depositFee instead of withdraw fee\\n```\\n    function doCutRewardsFee(address token) internal {\\n        if (bank.config().treasury() == address(0)) revert NO_TREASURY_SET();\\n\\n\\n        uint256 balance = IERC20Upgradeable(token).balanceOf(address(this));\\n        if (balance > 0) {\\n            uint256 fee = (balance * bank.config().depositFee()) / DENOMINATOR;\\n            IERC20Upgradeable(token).safeTransfer(\\n                bank.config().treasury(),\\n                fee\\n            );\\n\\n\\n            balance -= fee;\\n            IERC20Upgradeable(token).safeTransfer(bank.EXECUTOR(), balance);\\n        }\\n    }\\n```\\n\\nThis function is called in order to get fee from ICHI rewards, collected by farming. But currently it takes `bank.config().depositFee()` instead of `bank.config().withdrawFee()`.
Issue BasicSpell.doCutRewardsFee uses depositFee instead of withdraw fee\\nTake withdraw fee from rewards.
Wrong fee amount is taken.
```\\n    function doCutRewardsFee(address token) internal {\\n        if (bank.config().treasury() == address(0)) revert NO_TREASURY_SET();\\n\\n\\n        uint256 balance = IERC20Upgradeable(token).balanceOf(address(this));\\n        if (balance > 0) {\\n            uint256 fee = (balance * bank.config().depositFee()) / DENOMINATOR;\\n            IERC20Upgradeable(token).safeTransfer(\\n                bank.config().treasury(),\\n                fee\\n            );\\n\\n\\n            balance -= fee;\\n            IERC20Upgradeable(token).safeTransfer(bank.EXECUTOR(), balance);\\n        }\\n    }\\n```\\n