Can you change the review to be a "pre-commit" review? Then crucible gives us syntax hightlighting, in-line reviews, etc. https://source.openmrs.org/cru/CR-TRUNK-576
The "web service" to fetch the changes can either be a simple controller in core or it can be added to the webservices.rest module. Ben On Sun, Nov 13, 2011 at 11:41 PM, Ankur Gupta <[email protected]> wrote: > Hi, > > A description of the project can be found at : > https://wiki.openmrs.org/display/projects/Centralized+Auditing+%28Design+Page%29?moved=true > The changes(save/update/delete) made to any of the > tables in the “openmrs” database are saved in a common table(audit_log). > The entries made into table are date_changed,changed_by, voided_by, > change_description(in XML format). > We have completed the following: 1. Creating the audit class > (AuditLog.java) and its hibernate mapping (AuditLog.hbm.xml) 2. > Liquibase entry to create audit_log table in the database 3. Modifying > AuditableInterceptor.java, to add entries to the audit_log table on > save/update/delete 4. Creating a HibernateAuditLogDAO.java to provide the > hibernate session to the AuditableInterceptor.java > We have submitted this patch to the OpenMRS developers forum and are > waiting for their comments. > Since this is huge open source project, we do not expect them to reply > to us promptly. Meanwhile, we are looking on extending this to a service, > purging the existing model and integrating it into mainline. > > We are planning to implement the web service for this > functionality. The web service will provide facility to the admin to > retrieve the changes in the database. The changes can be retrieved in the > form of XML file. > > We have submitted the patch for review. It is currently under > review. We are waiting for the feedback for the same. > > Thank You > > Ankur Gupta > ------------------------------ > 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]
