Fix being able to vote on your own polls (#25015)
This commit is contained in:
		
							parent
							
								
									844d8e101e
								
							
						
					
					
						commit
						b030441368
					
				
					 2 changed files with 6 additions and 1 deletions
				
			
		| 
						 | 
					@ -3,8 +3,8 @@
 | 
				
			||||||
class VoteValidator < ActiveModel::Validator
 | 
					class VoteValidator < ActiveModel::Validator
 | 
				
			||||||
  def validate(vote)
 | 
					  def validate(vote)
 | 
				
			||||||
    vote.errors.add(:base, I18n.t('polls.errors.expired')) if vote.poll_expired?
 | 
					    vote.errors.add(:base, I18n.t('polls.errors.expired')) if vote.poll_expired?
 | 
				
			||||||
 | 
					 | 
				
			||||||
    vote.errors.add(:base, I18n.t('polls.errors.invalid_choice')) if invalid_choice?(vote)
 | 
					    vote.errors.add(:base, I18n.t('polls.errors.invalid_choice')) if invalid_choice?(vote)
 | 
				
			||||||
 | 
					    vote.errors.add(:base, I18n.t('polls.errors.self_vote')) if self_vote?(vote)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    vote.errors.add(:base, I18n.t('polls.errors.already_voted')) if additional_voting_not_allowed?(vote)
 | 
					    vote.errors.add(:base, I18n.t('polls.errors.already_voted')) if additional_voting_not_allowed?(vote)
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
| 
						 | 
					@ -27,6 +27,10 @@ class VoteValidator < ActiveModel::Validator
 | 
				
			||||||
    vote.choice.negative? || vote.choice >= vote.poll.options.size
 | 
					    vote.choice.negative? || vote.choice >= vote.poll.options.size
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					  def self_vote?(vote)
 | 
				
			||||||
 | 
					    vote.account_id == vote.poll.account_id
 | 
				
			||||||
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def already_voted_for_same_choice_on_multiple_poll?(vote)
 | 
					  def already_voted_for_same_choice_on_multiple_poll?(vote)
 | 
				
			||||||
    if vote.persisted?
 | 
					    if vote.persisted?
 | 
				
			||||||
      account_votes_on_same_poll(vote).where(choice: vote.choice).where.not(poll_votes: { id: vote }).exists?
 | 
					      account_votes_on_same_poll(vote).where(choice: vote.choice).where.not(poll_votes: { id: vote }).exists?
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1446,6 +1446,7 @@ en:
 | 
				
			||||||
      expired: The poll has already ended
 | 
					      expired: The poll has already ended
 | 
				
			||||||
      invalid_choice: The chosen vote option does not exist
 | 
					      invalid_choice: The chosen vote option does not exist
 | 
				
			||||||
      over_character_limit: cannot be longer than %{max} characters each
 | 
					      over_character_limit: cannot be longer than %{max} characters each
 | 
				
			||||||
 | 
					      self_vote: You cannot vote in your own polls
 | 
				
			||||||
      too_few_options: must have more than one item
 | 
					      too_few_options: must have more than one item
 | 
				
			||||||
      too_many_options: can't contain more than %{max} items
 | 
					      too_many_options: can't contain more than %{max} items
 | 
				
			||||||
  preferences:
 | 
					  preferences:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue