Removing the storage of a voter email for checking if they have voted

controllers/vote.go

@@ -121,6 +121,10 @@ func (r *VoteRepo) AddVote(c echo.Context) error {
 		return fmt.Errorf("url not found")
 	}
 
+	if u1.Voted {
+		return fmt.Errorf("this url has expired")
+	}
+
 	err = c.Request().ParseForm()
 	if err != nil {
 		return err
@@ -134,7 +138,6 @@ func (r *VoteRepo) AddVote(c echo.Context) error {
 
 	ballot := &storage.Ballot{
 		Election: u1.GetElection(),
-		Voter:    u1.GetVoter(),
 		Choice:   m,
 	}
 

storage/storage.proto

@@ -14,7 +14,6 @@ message STV {
 message Ballot {
     string id = 1;
     string election = 2;
-    string voter = 3;
     map<uint64, string> choice = 4; // map[order, candidate id]
 }
 

store/store.go

@@ -41,12 +41,6 @@ func (store *Store) AddBallot(ballot *storage.Ballot) (*storage.Ballot, error) {
 		return nil, err
 	}
 
-	for _, e := range stv.GetBallots() {
-		if e.GetElection() == ballot.GetElection() && e.GetVoter() == ballot.GetVoter() {
-			return nil, fmt.Errorf("ballot already exists for AddBallot")
-		}
-	}
-
 beforeUUID:
 	ballot.Id = uuid.NewString()
 
@@ -59,18 +53,13 @@ beforeUUID:
 
 	for _, election := range stv.GetElections() {
 		if election.GetId() == ballot.GetElection() {
-			for _, voter := range stv.GetVoters() {
-				if voter.GetEmail() == ballot.GetVoter() {
-					stv.Ballots = append(stv.GetBallots(), ballot)
-
-					if err = store.backend.Write(stv); err != nil {
-						return nil, err
-					}
+			stv.Ballots = append(stv.GetBallots(), ballot)
 
-					return ballot, nil
-				}
+			if err = store.backend.Write(stv); err != nil {
+				return nil, err
 			}
-			return nil, fmt.Errorf("unable to find voter for AddBallot")
+
+			return ballot, nil
 		}
 	}