Skip to content

Commit 091d73b

Browse files
committed
fix(protocol-contracts): remove unused function parameters in staking (N-06)
1 parent 9141432 commit 091d73b

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

protocol-contracts/staking/contracts/OperatorStaking.sol

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ contract OperatorStaking is ERC1363Upgradeable, ReentrancyGuardTransient, UUPSUp
323323
* @param controller The controller address.
324324
* @return Amount of shares pending redeem.
325325
*/
326-
function pendingRedeemRequest(uint256, address controller) public view virtual returns (uint256) {
326+
function pendingRedeemRequest(address controller) public view virtual returns (uint256) {
327327
OperatorStakingStorage storage $ = _getOperatorStakingStorage();
328328
return $._redeemRequests[controller].latest() - $._redeemRequests[controller].upperLookup(Time.timestamp());
329329
}
@@ -333,7 +333,7 @@ contract OperatorStaking is ERC1363Upgradeable, ReentrancyGuardTransient, UUPSUp
333333
* @param controller The controller address.
334334
* @return Amount of claimable shares.
335335
*/
336-
function claimableRedeemRequest(uint256, address controller) public view virtual returns (uint256) {
336+
function claimableRedeemRequest(address controller) public view virtual returns (uint256) {
337337
OperatorStakingStorage storage $ = _getOperatorStakingStorage();
338338
return $._redeemRequests[controller].upperLookup(Time.timestamp()) - $._sharesReleased[controller];
339339
}
@@ -360,7 +360,7 @@ contract OperatorStaking is ERC1363Upgradeable, ReentrancyGuardTransient, UUPSUp
360360
* @return The maximum redeemable shares.
361361
*/
362362
function maxRedeem(address ownerRedeem) public view virtual returns (uint256) {
363-
return claimableRedeemRequest(0, ownerRedeem);
363+
return claimableRedeemRequest(ownerRedeem);
364364
}
365365

366366
/**

protocol-contracts/staking/test/OperatorStaking.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -214,13 +214,13 @@ describe('OperatorStaking', function () {
214214
.connect(this.delegator1)
215215
.requestRedeem(await this.mock.balanceOf(this.delegator1), this.delegator1, this.delegator1);
216216

217-
await expect(this.mock.pendingRedeemRequest(0, this.delegator1)).to.eventually.eq(ethers.parseEther('1'));
218-
await expect(this.mock.claimableRedeemRequest(0, this.delegator1)).to.eventually.eq(0);
217+
await expect(this.mock.pendingRedeemRequest(this.delegator1)).to.eventually.eq(ethers.parseEther('1'));
218+
await expect(this.mock.claimableRedeemRequest(this.delegator1)).to.eventually.eq(0);
219219

220220
await time.increase(60);
221221

222-
await expect(this.mock.pendingRedeemRequest(0, this.delegator1)).to.eventually.eq(0);
223-
await expect(this.mock.claimableRedeemRequest(0, this.delegator1)).to.eventually.eq(ethers.parseEther('1'));
222+
await expect(this.mock.pendingRedeemRequest(this.delegator1)).to.eventually.eq(0);
223+
await expect(this.mock.claimableRedeemRequest(this.delegator1)).to.eventually.eq(ethers.parseEther('1'));
224224

225225
await expect(this.mock.connect(this.delegator1).redeem(ethers.parseEther('1'), this.delegator1, this.delegator1))
226226
.to.emit(this.token, 'Transfer')

0 commit comments

Comments
 (0)