Yes, please do that. That's what it's for. :-) (Others, the wiki page is at https://wiki.openmrs.org/x/74Js if you're interested in watching progress.)
-Darius On Wed, Oct 19, 2011 at 7:47 PM, Rohit Manohar <[email protected]> wrote: > Can we update the wiki to our current understanding and progress (as we go > forward)? > > Regards, > > > On Wed, Oct 19, 2011 at 10:27 PM, Darius Jazayeri <[email protected] > > wrote: > >> Hi Rohit, >> >> If you include a link to the project or ticket in your email, that will >> give people more background for (the majority) of people who don't know >> offhand what the auditing project is. >> >> Generally when you create/update/void something, we have "SaveHandlers" >> that are invoked via AOP and they set the creator/dateCreated/changedBy/etc >> fields on the objects. (Or at least that's what we were doing until >> recently, when we added a hibernate interceptor to take care of some of >> those details.) So, for example if a user wants to soft-delete a Patient, >> they'd call patientService.voidPatient(patient, "Reason"). The user and date >> would be set by OpenMRS. >> >> Now, the goal of the audit project is that we would actually remove the >> audit fields. The fact that a patient is edited, and by whom, would be in >> the audit table--it wouldn't need to be stored in the patient table anymore. >> >> To answer your initial question, void reason should usually be provided by >> the user (e.g. the second argument to PatientService.voidPatient), and if >> not it should default to something like "deleted through API". The changedBy >> would be Context.getAuthenticatedUser(), and dateChanged would be now(). >> >> -Darius >> >> On Wed, Oct 19, 2011 at 6:59 PM, Rohit Manohar <[email protected]> wrote: >> >>> Restating, does an insert/update/delete to a table with auditing require >>> that all the audit attributes ( >>> https://wiki.openmrs.org/display/docs/Conventions) be specified by the >>> user? >>> >>> >>> On Wed, Oct 19, 2011 at 9:54 PM, Rohit Manohar <[email protected]>wrote: >>> >>>> Hi >>>> >>>> We are working on building a centralizes audit service. One of the >>>> fields required is 'void_reason'. Could someone elaborate how the hibernate >>>> interceptor is supposed to obtain this? >>>> >>>> Regards, >>>> -- >>>> Rohit Manohar >>>> Graduate Student >>>> North Carolina State University >>>> Raleigh, US. >>>> >>>> >>> >>> >>> -- >>> Rohit Manohar >>> Graduate Student >>> North Carolina State University >>> Raleigh, US. >>> >>> ------------------------------ >>> Click here to >>> unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from >>> OpenMRS Developers' mailing list >>> >> >> ------------------------------ >> Click here to >> unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from >> OpenMRS Developers' mailing list > > > > > -- > Rohit Manohar > Graduate Student > North Carolina State University > Raleigh, US. > > ------------------------------ > Click here to > unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from > OpenMRS Developers' mailing list > _________________________________________ To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to [email protected] with "SIGNOFF openmrs-devel-l" in the body (not the subject) of your e-mail. [mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]
