Merge pull request #11 from MaximeZmt/Work-Mzmt-FixAddAdmin

fix: do reinsert existing admin
c4dt · May 29, 2024 · d2080fd · d2080fd
2 parents 7242c71 + e3454c4
commit d2080fd
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/contracts/evoting/types/admin.go b/contracts/evoting/types/admin.go
@@ -48,6 +48,15 @@ func (adminList *AdminList) AddAdmin(userID string) error {
 		return xerrors.Errorf("Failed to convert SCIPER to int: %v", err)
 	}
 
+	index, err := adminList.GetAdminIndex(userID)
+	if err != nil {
+		return err
+	}
+
+	if index > -1 {
+		return xerrors.Errorf("The user %v is already an admin", userID)
+	}
+
 	adminList.AdminList = append(adminList.AdminList, sciperInt)
 
 	return nil