Merge pull request #136 from jjchee77/Bugfix-QrCodeEdgeCaseFailedGen

Bugfix: Edge case causing QR not to generate
AY2324S2-CS2103T-T17-3 · Mar 21, 2024 · e8d45e7 · e8d45e7
2 parents 781294c + 11c2848
commit e8d45e7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/main/java/seedu/address/model/ModelManager.java b/src/main/java/seedu/address/model/ModelManager.java
@@ -106,8 +106,8 @@ public void deletePerson(Person target) {
 
     @Override
     public void addPerson(Person person) {
-        addressBook.addPerson(person);
         person.generateQrCode();
+        addressBook.addPerson(person);
         updateFilteredPersonList(PREDICATE_SHOW_ALL_PERSONS);
     }