Edit/View mode for Active/Past visit : Diagnoses fragment

[DiyaIT1]

No description provided.

[nehav39]

The diagnosis fragment does not show the entered diagnoses for a closed visit.

[nehav39]

For this patient, a diagnosis of "cough" and "fever of unknown origin" was added.

[prithi69]

in the patient overview screen "back to top" is not working.

[amalafrozalam]

@hardikk08
Check src > webapp > resources > scripts > diagnoses > diagnoses.js

[amalafrozalam]

Above comment is incorrect. It's definitely in that file, but somewhere further along.

The fragment seems to be saving the information correct, but it does not seem to be retrieving it correctly. Perhaps the URL call in one of the factories is incorrect. Specifically, when the page is loaded and previous/prior diagnoses are searched for.

[nehav39]

@amalafrozalam - No code was written to retrieve and display diagnoses. It's not a bug. This was just not completed.

[amalafrozalam]

@nehagoel2 you tagged the wrong amal. Need to use my full username.

Okay, well that explains it. There is a retrieval of sorts going on, but I believe that issues #23 and #44 are all related.

[hardikk08]

@nehagoel2 I wish you had pointed that out earlier @amalafrozalam I can see it being posted in the database. So basically the code to retrieve it is missing. Would have been easy if we knew the logic flow of the entire module hence it is taking time to understand.

[nehav39]

Look at closed issue #26

[nehav39]

https://github.com/openmrs/openmrs-module-coreapps/tree/master/omod/src/main/webapp/fragments/diagnosis

[nehav39]

https://github.com/openmrs/openmrs-module-coreapps/blob/master/omod/src/main/java/org/openmrs/module/coreapps/fragment/controller/DiagnosesFragmentController.java

[nehav39]

@hardikk08 - great job so far on the fragment. Just a last push:

1. For a closed visit, the text entry box for entering diagnosis should be hidden so that no new diagnoses can be entered
2. The diagnoses being seen in the Previous diagnoses are not marked as Primary, Secondary, Confirmed or Unconfirmed. These should also be seen
3. Previous diagnosis heading is in Red - make it consistent in look and feel.

[hardikk08]

For the 2nd point...
This is a bit difficult to achieve, saying it after trying a lot of stuff and this feature will take time. Reason, obs for diagnoses is being posted just after we enter a diagnoses name hence clicking on primary or confirmed has no effect on what is being posted in the database! Hence the checkbox implementation is more of a dummy right now. The way it is happening in core apps is very complex and will require a lot of modification in our app.