Skip to content

Commit 8f46ea4

Browse files
committed
fixed bug in dbgatway where id -1 was getting added to database
1 parent 7bf48c1 commit 8f46ea4

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

src/data/Users.csv

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1-
username,password,isAdmin,flashcardSetsIds
2-
testUser,password,false,0,1
3-
jempio,ilovejempio,false,
1+
username,password,isAdmin,flashcardSetIds
2+
jempio,ilovejempio,false
3+
ducas,badpassword,false
4+
testUser,password,false,0,1,2

src/main/java/dataAccess/DBGateway.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,9 @@ public void saveUser(CommonUserDsRequestModel user) {
7171

7272

7373
public int saveFlashcardSet(FlashcardSetDsRequestModel flashcardSet) {
74+
int id = this.flashcardSetGateway.saveFlashcardSet(flashcardSet);
7475
this.userGateway.saveFlashcardSetID(flashcardSet.getOwnerUsername(), flashcardSet.getFlashcardSetId());
75-
return this.flashcardSetGateway.saveFlashcardSet(flashcardSet);
76+
return id;
7677
}
7778

7879
public void deleteFlashcardSet(String ownerUsername, int flashcardSetID) {
@@ -81,8 +82,9 @@ public void deleteFlashcardSet(String ownerUsername, int flashcardSetID) {
8182
}
8283

8384
public int saveFlashcard(FlashcardDsRequestModel flashcard) {
85+
int id = this.flashcardGateway.saveFlashcard(flashcard);
8486
this.flashcardSetGateway.saveFlashcardID(flashcard.getBelongsToId(), flashcard.getFlashcardId());
85-
return this.flashcardGateway.saveFlashcard(flashcard);
87+
return id;
8688
}
8789

8890
public void deleteFlashcard(int flashcardSetId, int flashcardId) {

0 commit comments

Comments
 (0)