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

the Obs method getValueAsString(Locale locale) needs improvement

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Could
    • Resolution: Fixed
    • Affects Version/s: OpenMRS 1.6.2, OpenMRS 1.7.0
    • Fix Version/s: OpenMRS 1.9.0
    • Component/s: None
    • Complexity:
      Medium

      Description

      I have two complaints about this method.

      First, if the obs value is numeric, the string generated should respect the 'precise' property of the related ConceptNumeric.

      Second, the default date formats shouldn't include commas by default – these are the default column separators in csv exports, and it makes this method less useful if you're designing a data export. I think for dates yyyy-MM-dd would be more generally useful.

        Attachments

        1. MRS_API_1.patch
          5 kB
        2. OPENMRS_API_1883.patch
          5 kB

          Issue Links

            Activity

              People

              • Assignee:
                surangak Suranga Kasthurirathne
                Reporter:
                dthomas Dave Thomas
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: