[LBAC-8] Failed to filer the patients who listed in the findPatients page before searching Created: 2018-07-09 Updated: 2018-07-10 Resolved: 2018-07-10 |
|
| Status: | Closed |
| Project: | Location Based Access Control |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 0.1.0 |
| Type: | Bug | Priority: | Should |
| Reporter: | Suthagar Kailayapathy | Assignee: | Suthagar Kailayapathy |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | gsoc2018 | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Complexity: | Low |
| Watchers: |
Daniel Kayiwa, Suthagar Kailayapathy
|
| Designated Committer: | Suthagar Kailayapathy |
| Description |
|
The find patients page will list some patients who have the recent encounters and recent search also. This list contains the patients who do not belong to the logged in location of the user. It should be investigated and need a fix to remove those patients from that list. Dev Notes Last viewed patient list if fetched through the getPatient() method in emrapi module. See here : https://github.com/openmrs/openmrs-module-emrapi/blob/master/api/src/main/java/org/openmrs/module/emrapi/utils/GeneralUtils.java#L264 So need to add a AOP Advice to restrict the getPatient() method based on the location.
|
| Comments |
| Comment by Suthagar Kailayapathy [ 2018-07-09 ] |
|
Daniel Kayiwa I started to work on this issue. |
| Comment by Suthagar Kailayapathy [ 2018-07-09 ] |
|
Daniel Kayiwa Could you please review it : https://github.com/openmrs/openmrs-module-locationbasedaccess/pull/10
|
| Comment by Daniel Kayiwa [ 2018-07-10 ] |
|
Merged at https://github.com/openmrs/openmrs-module-locationbasedaccess/commit/02659395f6af06a7845d2104922681e478489346 |