Skip to content

Commit

Permalink
rename _isProposalOpen arg to isOpen
Browse files Browse the repository at this point in the history
  • Loading branch information
Rekard0 committed Jan 29, 2025
1 parent c165feb commit 97c82c3
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions packages/contracts/src/MajorityVotingBase.sol
Original file line number Diff line number Diff line change
Expand Up @@ -643,15 +643,12 @@ abstract contract MajorityVotingBase is

/// @notice An internal function that checks if the proposal succeeded or not.
/// @param _proposalId The ID of the proposal.
/// @param _isProposalOpen Weather the proposal is open or not.
/// @param _isOpen Weather the proposal is open or not.
/// @return Returns `true` if the proposal succeeded depending on the thresholds and voting modes.
function _hasSucceeded(
uint256 _proposalId,
bool _isProposalOpen
) internal view virtual returns (bool) {
function _hasSucceeded(uint256 _proposalId, bool _isOpen) internal view virtual returns (bool) {
Proposal storage proposal_ = proposals[_proposalId];

if (_isProposalOpen) {
if (_isOpen) {
// If the proposal is still open and the voting mode is VoteReplacement,
// success cannot be determined until the voting period ends.
if (proposal_.parameters.votingMode == VotingMode.VoteReplacement) {
Expand Down

0 comments on commit 97c82c3

Please sign in to comment.