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

the Obs method getValueAsString(Locale locale) needs improvement

    XMLWordPrintable

    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: