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

Support deriving auto_expire_date from duration and duration units for Drug Orders

    XMLWordPrintable

Details

    Description

      Currently duration_units is a concept without any metadata which can help converting the duration to auto_expire_date of drug order.

      We need a standard mechanism by which these concepts can be interpreted to determine auto_expire_dates from the start date.

      Units we need to support

      • Second(s)
      • Minute(s)
      • Hour(s)
      • Day(s)
      • Week(s)
      • Month(s)
      • Year(s)
      • Time(s)

      Questions

      What is an existing standard that has well-known external codes for the duration units we need to support? If we can't find anything better, we can go with Y, M, W, D, H, M, S from http://en.wikipedia.org/wiki/ISO_8601#Durations

      Does the upgrade helper module need to help with this?

      How should we handle "Time(s)"? The method signature in Dev Notes isn't sufficient for this

      Dev Notes

      (from https://notes.openmrs.org/Design-Forum-2014-08-13)

      If the startDate has no time component and the duration is provided, then the autoExpireDate should be computed until just before midnight (23:59:59.999) e.g If the prescription is for 4 days from august 1st, the autoExpireDate should be august 4th at 23.59:59.999pm (and importantly, the dev working on this ticket should be aware that the underlying API is going to push stop dates of 00:00:00 to be 23:59:59), see also TRUNK-4472

      (from https://notes.openmrs.org/Design-Forum-2014-07-30 )

      We are going to use concept reference mappings to indicate the underlying meaning of the duration unit concepts.

      • Ask Andy Kanter to add a concept source representing the standard we choose. Find the duration unit concepts in the CIEL concept dictionary, and ask Andy Kanter to add reference mappings for them. (E.g. CIEL 1073 should be mapped to "ISO 8601 Duration:W")
      • Add instructions to the 1.10 wiki pages describing how someone not using the CIEL dictionary can add this source and mappings
      • Add a utility method like: Date addDuration(Date indexDate, double num, Concept durationUnit).
        • Under the hood I would use joda time (if we've already included that in OpenMRS core)
        • I'd expect this method to have hardcoded behaviors knowing about the units we care about
        • To Do: change this so we can handle Time(s). E.g. maybe it should be Date inferAutoExpireDate(DrugOrder)
        • Main point: just have a single utility method that handles exactly the case we care about – we're not trying to introduce a generic unit conversion framework at this point.

      Gliffy Diagrams

        Attachments

          Issue Links

            Activity

              People

                endeepak Deepak N
                endeepak Deepak N
                Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Time Tracking

                    Estimated:
                    Original Estimate - Not Specified
                    Not Specified
                    Remaining:
                    Remaining Estimate - 0 minutes
                    0m
                    Logged:
                    Time Spent - 2 days, 2 hours
                    2d 2h