Skip to content

Commit

Permalink
Merge pull request #59 from The-Poolz/rename
Browse files Browse the repository at this point in the history
renamed variable to avoid ambiguity
  • Loading branch information
Lomet authored Oct 11, 2023
2 parents f7cb7fa + ed4abc2 commit 8807eb0
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions contracts/VaultManager/VaultView.sol
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,10 @@ abstract contract VaultView is VaultState, IVaultManager {
uint count
) external view returns (uint balance) {
uint[] memory vaultIds = tokenToVaultIds[_tokenAddress];
uint totalVaults = vaultIds.length;
require(totalVaults > 0, "VaultManager: No vaults for this token");
uint totalVaultsOfToken = vaultIds.length;
require(totalVaultsOfToken > 0, "VaultManager: No vaults for this token");
require(count > 0, "VaultManager: Count must be greater than 0");
require(from + count <= totalVaults, "VaultManager: Invalid range");
require(from + count <= totalVaultsOfToken, "VaultManager: Invalid range");
for (uint i = from; i < from + count; i++) {
balance += Vault(vaultIdToVault[vaultIds[i]]).tokenBalance();
}
Expand Down

0 comments on commit 8807eb0

Please sign in to comment.