Downloads Documentation Community
Contribute
Contribute Demo






Show Sidebar
Login | Register


Ticket #652 (new task)

Opened 6 months ago

Last modified 2 weeks ago

Move tribe to person attribute

Reported by: bmamlin Assigned to: upul
Priority: minor Milestone: OpenMRS 1.4
Component: OpenMRS Code Base Keywords:
Cc: bwolfe, mseaton, jarbaugh

Description

When OpenMRS started, tribe was an attribute of the patient table. Over time, we developed the person_attribute table to hold implementation-specific attributes of people. Since not all implementation need/want tribe as part of the system, person_atttribute is a better place for it. One option would be to migrate tribe to person attributes over a couple OpenMRS updates (deprecating it in one and removing in another). Then a tribe module could be created to add the tribe functionality for those implementations who want it.

See also #624

Attachments

tribe-652-2.patch (83.4 kB) - added by bwolfe on 06/10/08 13:44:59.
testing again
openmrs-652.patch (87.7 kB) - added by upul on 07/07/08 09:03:12.
Latest patch with the dummy module page

Change History

05/27/08 12:04:17 changed by bwolfe

  • owner changed from somebody to upul.
  • milestone changed from OpenMRS 1.3 to OpenMRS 1.4.

06/06/08 12:08:58 changed by bwolfe

  • cc changed from bwolfe to bwolfe, mseaton, jarbaugh.

Awesome, thanks for this Upul. It looks like you've found everything.

My only suggestions are for the patch are:

  1. Leave the Manage Tribes page there with a note on it saying that you should download the Tribe module and install it now.

actually, thats all for the patch.

The module needs to extend the org.openmrs.patientDashboard.Header extensionPoint and add the tribe back to the patient dashboard.

#804 will need to be fixed soon after this change for James Arbaugh's needs and the Edit Tribe Permission hack that was put in.

06/10/08 13:44:59 changed by bwolfe

  • attachment tribe-652-2.patch added.

testing again

06/27/08 08:17:59 changed by bwolfe

I see in [4599] you added a note in the old location. However, I was meaning to leave a note in core code alluding to the fact that there is a module that they should use now. I think this is necessary because some people will download 1.4, see no tribes page and freak out. A text there that says "go here to download a new module, all your data will be imported..." will calm them down.

In 1.5 we can remove that link and text and just assume people will go and download the module.

07/07/08 09:03:12 changed by upul

  • attachment openmrs-652.patch added.

Latest patch with the dummy module page

08/14/08 08:57:56 changed by bwolfe

The mysql diff update file needs to get rid of the old permissions: 

delete from role_privilege where privilege like '%Tribes';

otherwise, the user pages and manage role pages don't work. :-)