Merge pull request #49 from TAMULib/sprint4-b03415-feature-proposal

tamu-sad-iii · web-flow · commit e365b171ca49 · 2018-03-30T08:50:34.000-05:00
No longer attempting to add same user twice
diff --git a/src/main/java/edu/tamu/app/model/FeatureProposal.java b/src/main/java/edu/tamu/app/model/FeatureProposal.java
@@ -104,7 +104,9 @@ public void setVoters(List<User> voters) {
     }
 
     public void addVoter(User voter) {
-        this.voters.add(voter);
+        if (!this.voters.contains(voter)) {
+            this.voters.add(voter);
+        }
     }
 
     public void removeVoter(User voter) {

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,9 @@ public void setVoters(List<User> voters) {`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`public void addVoter(User voter) {`
`107`		`- this.voters.add(voter);`
	`107`	`+ if (!this.voters.contains(voter)) {`
	`108`	`+ this.voters.add(voter);`
	`109`	`+ }`
`108`	`110`	`}`
`109`	`111`
`110`	`112`	`public void removeVoter(User voter) {`