Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-2015

Move some functionality from SaveHandlers to hibernate interceptors

    XMLWordPrintable

Details

    • Undetermined

    Description

      Now that Ben has demonstrated feasibility in TRUNK-1930, we should consider moving more functionality from save handlers into hibernate interceptors. (Hibernate interceptors don't require specific conventions for method names, and are probably less error-prone.)

      This could definitely be done in an interceptor:

      • OpenmrsObjectSaveHandler assigns a UUID to any OpenmrsObject that doesn't have one.

      This might be possible to move to an interceptor (but it should probably be done in such a way that objects that parent and child objects get identical dateCreated values):

      • AuditableSaveHandler sets creator/dateCreated

      Gliffy Diagrams

        Attachments

          Issue Links

            caused

            Bug - A problem which impairs or prevents the functions of the product. TRUNK-2903 AuditableInterceptor fails with a NPE at startup if there are scheduled tasks to run at startup

            • Blocker - Blocks development and/or testing work, production could not run or other critical issues are being delayed.
            • Closed
            is related to

            Bug - A problem which impairs or prevents the functions of the product. TRUNK-2692 Editing provider via an HTML Form fails

            • Blocker - Blocks development and/or testing work, production could not run or other critical issues are being delayed.
            • Closed

            New Feature - A new feature of the product, which has yet to be developed. TRUNK-2588 Generic mechanism for doing Attributes and Attribute Types on many classes. (Review code, and Refactor.)

            • Blocker - Blocks development and/or testing work, production could not run or other critical issues are being delayed.
            • Closed

            Activity

              People

                raff Rafal Korytkowski
                darius Darius Jazayeri
                Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: