fix duplicate patient bug

fortitudoinc · Dec 19, 2019 · 7bcd401 · 7bcd401
1 parent 7881f10
commit 7bcd401
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/omod/pom.xml b/omod/pom.xml
@@ -31,7 +31,6 @@
 			<artifactId>aws-java-sdk</artifactId>
 			<version>1.11.327</version>
 		</dependency>
-
 		<dependency>
 			<groupId>org.openmrs.web</groupId>
 			<artifactId>openmrs-web</artifactId>

diff --git a/omod/src/main/java/org/openmrs/module/patientlist/advice/PatientAroundSaveAdvise.java b/omod/src/main/java/org/openmrs/module/patientlist/advice/PatientAroundSaveAdvise.java
@@ -74,7 +74,8 @@ public Object invoke(MethodInvocation invocation) throws Throwable {
 			}
 			// the proceed() method does not have to be called
 			Patient newPatient = (Patient) invocation.proceed();
-			System.out.println("AFTER SAVE - ADDING NEW PATIENT TO PATIENTLIST o: " + o.getClass());
+			System.out.println("AFTER SAVE - ADDING NEW PATIENT TO PATIENTLIST: " + newPatient.getGivenName() + " "
+			        + newPatient.getFamilyName());
 
 			addPatientToActiveList(patient);
 			log.debug("After " + invocation.getMethod().getName() + ".");
@@ -102,8 +103,8 @@ private boolean isOldPatientInActiveList() {
 
 		private boolean isPatientAlreadyRegistered(Patient patient) {
 			List<Patient> patients = Context.getPatientService().getAllPatients();
-			System.out.println("patientAlreadyRegistered: " + patient.getGivenName() + " " + patient.getFamilyName() + " "
-			        + patient.getGender() + " " + patient.getAge() + " "
+			System.out.println("in method patientAlreadyRegistered: " + patient.getGivenName() + " "
+			        + patient.getFamilyName() + " " + patient.getGender() + " " + patient.getAge() + " "
 			        + patient.getAttribute("Telephone Number").getValue());
 			for (Patient oldPat : patients) {
 				if ((oldPat.getGender().equals(patient.getGender()))