From 4a57e1c536c1112f49e1dc85468b5891bfedacc7 Mon Sep 17 00:00:00 2001
From: Suthagar23 <suthagar23@gmail.com>
Date: Wed, 20 Jun 2018 18:55:45 +0530
Subject: [PATCH] LABC-3 Added implementation for assigning the patients to the
 locations on registration

Removed personAttribute creation

Added fixes for PR comments

PR Review fixes

Added minor fix

Added some fixes according to the PR reviews

Minor fix

Changed the locationCheck method structure

minor fix

Changed method name

Added fixes for PR reviews

Added fixes for PR reviews

Changed method to compare

Changed Compare method

Minor improvements

Minor modifications

Minor fix
---
 .../PatientSearchAdviser.java                 | 77 +++++++++++++++++++
 omod/pom.xml                                  |  6 ++
 .../field/LocationsFragmentController.java    | 16 ++++
 omod/src/main/resources/config.xml            |  8 +-
 .../resources/webModuleApplicationContext.xml |  4 +-
 .../main/webapp/fragments/field/locations.gsp | 16 ++++
 pom.xml                                       | 30 +++++++-
 7 files changed, 154 insertions(+), 3 deletions(-)
 create mode 100644 api/src/main/java/org/openmrs/module/locationbasedaccess/PatientSearchAdviser.java
 create mode 100644 omod/src/main/java/org/openmrs/module/locationbasedaccess/fragment/controller/field/LocationsFragmentController.java
 create mode 100644 omod/src/main/webapp/fragments/field/locations.gsp

diff --git a/api/src/main/java/org/openmrs/module/locationbasedaccess/PatientSearchAdviser.java b/api/src/main/java/org/openmrs/module/locationbasedaccess/PatientSearchAdviser.java
new file mode 100644
index 0000000..87819b5
--- /dev/null
+++ b/api/src/main/java/org/openmrs/module/locationbasedaccess/PatientSearchAdviser.java
@@ -0,0 +1,77 @@
+/**
+ * This Source Code Form is subject to the terms of the Mozilla Public License,
+ * v. 2.0. If a copy of the MPL was not distributed with this file, You can
+ * obtain one at http://mozilla.org/MPL/2.0/. OpenMRS is also distributed under
+ * the terms of the Healthcare Disclaimer located at http://openmrs.org/license.
+ * <p>
+ * Copyright (C) OpenMRS Inc. OpenMRS is a registered trademark and the OpenMRS
+ * graphic logo is a trademark of OpenMRS Inc.
+ */
+
+package org.openmrs.module.locationbasedaccess;
+
+import org.aopalliance.aop.Advice;
+import org.aopalliance.intercept.MethodInvocation;
+import org.aopalliance.intercept.MethodInterceptor;
+import org.apache.commons.lang3.StringUtils;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+import org.openmrs.Patient;
+import org.openmrs.PersonAttributeType;
+import org.openmrs.api.context.Context;
+import org.springframework.aop.Advisor;
+import org.springframework.aop.support.StaticMethodMatcherPointcutAdvisor;
+import java.lang.reflect.Method;
+import java.util.ArrayList;
+import java.util.Iterator;
+import java.util.List;
+
+public class PatientSearchAdviser extends StaticMethodMatcherPointcutAdvisor implements Advisor {
+
+    private static final Log log = LogFactory.getLog(PatientSearchAdviser.class);
+    public static final  String LOCATION_ATTRIBUTE_GLOBAL_PROPERTY_NAME = "locationbasedaccess.locationAttributeUuid";
+
+    @Override
+    public boolean matches(Method method, Class targetClass) {
+        if (method.getName().equals("getPatients")) {
+            return true;
+        }
+        return false;
+    }
+
+    @Override
+    public Advice getAdvice() {
+        return new PatientSearchAdvise();
+    }
+
+    private class PatientSearchAdvise implements MethodInterceptor {
+
+        public Object invoke(MethodInvocation invocation) throws Throwable {
+            Integer sessionLocationId = Context.getUserContext().getLocationId();
+            String locationAttributeUuid = Context.getAdministrationService().getGlobalProperty(LOCATION_ATTRIBUTE_GLOBAL_PROPERTY_NAME);
+            List<Patient> patientList = (List<Patient>) invocation.proceed();
+            if (StringUtils.isNotBlank(locationAttributeUuid)) {
+                final PersonAttributeType personAttributeType = Context.getPersonService().getPersonAttributeTypeByUuid(locationAttributeUuid);
+                if (sessionLocationId != null) {
+                    String sessionLocationUuid = Context.getLocationService().getLocation(sessionLocationId).getUuid();
+
+                    for (Iterator<Patient> iterator = patientList.iterator(); iterator.hasNext(); ) {
+                        Patient patient = iterator.next();
+                        if (!compare(patient.getAttribute(personAttributeType).getValue(), sessionLocationUuid)) {
+                            iterator.remove();
+                        }
+                    }
+                } else {
+                    // If the sessionLocationId is null, then return a empty list
+                    log.debug("Search Patient : Null Session Location in the UserContext");
+                    return new ArrayList<Patient>();
+                }
+            }
+            return patientList;
+        }
+
+        private Boolean compare(String value1, String value2) {
+            return (StringUtils.isNotBlank(value1) && StringUtils.isNotBlank(value2)) && value1.equals(value2);
+        }
+    }
+}
diff --git a/omod/pom.xml b/omod/pom.xml
index c7768dc..65efaf9 100644
--- a/omod/pom.xml
+++ b/omod/pom.xml
@@ -19,6 +19,12 @@
             <version>${project.parent.version}</version>
             <scope>compile</scope>
         </dependency>
+		<dependency>
+			<groupId>org.openmrs.module</groupId>
+			<artifactId>appui-omod</artifactId>
+			<version>${appuiVersion}</version>
+			<scope>provided</scope>
+		</dependency>
 	</dependencies>
 
 	<build>
diff --git a/omod/src/main/java/org/openmrs/module/locationbasedaccess/fragment/controller/field/LocationsFragmentController.java b/omod/src/main/java/org/openmrs/module/locationbasedaccess/fragment/controller/field/LocationsFragmentController.java
new file mode 100644
index 0000000..0df4e3e
--- /dev/null
+++ b/omod/src/main/java/org/openmrs/module/locationbasedaccess/fragment/controller/field/LocationsFragmentController.java
@@ -0,0 +1,16 @@
+package org.openmrs.module.locationbasedaccess.fragment.controller.field;
+
+import org.openmrs.Location;
+import org.openmrs.ui.framework.fragment.FragmentModel;
+import org.openmrs.api.context.Context;
+import javax.servlet.http.HttpSession;
+import java.util.List;
+
+public class LocationsFragmentController {
+
+    public void controller(FragmentModel model,
+                           HttpSession session) {
+        List<Location> activeLocations = Context.getLocationService().getAllLocations();
+        model.addAttribute("activeLocations", activeLocations);
+    }
+}
diff --git a/omod/src/main/resources/config.xml b/omod/src/main/resources/config.xml
index df6c3ad..cdaa591 100644
--- a/omod/src/main/resources/config.xml
+++ b/omod/src/main/resources/config.xml
@@ -18,7 +18,7 @@
 	
 	<!-- Required Modules -->
 	<require_modules>
-
+		<require_module version="${uiframeworkVersion}">org.openmrs.module.uiframework</require_module>
 	</require_modules>
 	<!-- / Required Modules -->
 	
@@ -46,5 +46,11 @@
 		<file>messages_es.properties</file>
 	</messages>
 	<!-- /Internationalization -->
+
+	<advice>
+		<point>org.openmrs.api.PatientService</point>
+		<class>org.openmrs.module.locationbasedaccess.PatientSearchAdviser</class>
+	</advice>
+
 </module>
 
diff --git a/omod/src/main/resources/webModuleApplicationContext.xml b/omod/src/main/resources/webModuleApplicationContext.xml
index 5b53dec..d1bef5b 100644
--- a/omod/src/main/resources/webModuleApplicationContext.xml
+++ b/omod/src/main/resources/webModuleApplicationContext.xml
@@ -5,6 +5,8 @@
   		    http://www.springframework.org/schema/beans/spring-beans-3.0.xsd">
 
 	<!-- Add here beans related to the web context -->
-
+	<bean class="org.openmrs.ui.framework.StandardModuleUiConfiguration">
+		<property name="moduleId" value="${project.parent.artifactId}"/>
+	</bean>
 		
 </beans>
diff --git a/omod/src/main/webapp/fragments/field/locations.gsp b/omod/src/main/webapp/fragments/field/locations.gsp
new file mode 100644
index 0000000..4a9aca4
--- /dev/null
+++ b/omod/src/main/webapp/fragments/field/locations.gsp
@@ -0,0 +1,16 @@
+
+<div>
+    <div>
+        <p class="left">
+            <label>
+                Please select the location
+            </label>
+            <select id="${config.formFieldName}" name="${config.formFieldName}">
+                <% activeLocations.each { location -> %>
+                <option value="${location.uuid}">${location.name}</option>
+                <% } %>
+            </select>
+        </p>
+    </div>
+</div>
+
diff --git a/pom.xml b/pom.xml
index fd3d0ba..1382bd9 100644
--- a/pom.xml
+++ b/pom.xml
@@ -22,6 +22,10 @@
 
     <properties>
     	<openMRSVersion>2.0.5</openMRSVersion>
+		<uiframeworkVersion>3.9</uiframeworkVersion>
+		<uicommonsVersion>2.4.0</uicommonsVersion>
+		<appframeworkVersion>2.9</appframeworkVersion>
+        <appuiVersion>1.7</appuiVersion>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
      </properties>
 
@@ -64,7 +68,31 @@
 				<scope>provided</scope>
 			</dependency>
 
-            
+			<dependency>
+				<groupId>org.openmrs.module</groupId>
+				<artifactId>uiframework-api</artifactId>
+				<version>${uiframeworkVersion}</version>
+				<scope>provided</scope>
+			</dependency>
+
+			<dependency>
+				<groupId>org.openmrs.module</groupId>
+				<artifactId>uicommons-api</artifactId>
+				<version>${uicommonsVersion}</version>
+				<scope>provided</scope>
+			</dependency>
+			<dependency>
+				<groupId>org.openmrs.module</groupId>
+				<artifactId>appframework-api</artifactId>
+				<version>${appframeworkVersion}</version>
+				<scope>provided</scope>
+			</dependency>
+            <dependency>
+                <groupId>org.openmrs.module</groupId>
+                <artifactId>appui-omod</artifactId>
+                <version>${appuiVersion}</version>
+                <scope>provided</scope>
+            </dependency>
             <!-- For testing -->
             <dependency>
                 <groupId>org.openmrs.test</groupId>