Reminder: Create & find trunk-related issues here. Legacy Trac tickets are still available. Problems? Workflow feedback? Need a module project? Open a support ticket.

OpenMRS Trunk

Cohort Column broken in data exports

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Should Should
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: OpenMRS 1.4.0
  • Component/s: None
  • Description:

    Any cohort columns (cohort, saved search, or saved filter) always shows up blank in data exports.

    I haven't verified whether this bug exists in 1.3, but if so a fix should be backported.

  1. 1166.patch
    (4 kB)
    Burke Mamlin
    2008-12-12 18:05:37 EST

Activity

Hide
Darius Jazayeri added a comment - 2008-12-12 18:07:49 EST

Unit test format is not exactly right, since I don't have the generate test cases plugin working.

This fixes it back up to how it worked before we introduced reporting evaluation contexts. The data export should probably create one of those and keept it around for the duration of a single data export generation. But that's left as a TODO.

Ben, Justin, quick code review on this?

Show
Darius Jazayeri added a comment - 2008-12-12 18:07:49 EST Unit test format is not exactly right, since I don't have the generate test cases plugin working. This fixes it back up to how it worked before we introduced reporting evaluation contexts. The data export should probably create one of those and keept it around for the duration of a single data export generation. But that's left as a TODO. Ben, Justin, quick code review on this?
Hide
Darius Jazayeri added a comment - 2008-12-15 17:35:21 EST

Bug also exists in 1.3.x so needs to be backported there too.

Show
Darius Jazayeri added a comment - 2008-12-15 17:35:21 EST Bug also exists in 1.3.x so needs to be backported there too.
Hide
Ben Wolfe added a comment - 2008-12-15 19:08:49 EST

Your DataExportFunctions and a slightly modified DataExportTest have been committed to trunk in rev:6288

A backport to the 1.4.x and 1.3.x branches is still needed.

Show
Ben Wolfe added a comment - 2008-12-15 19:08:49 EST Your DataExportFunctions and a slightly modified DataExportTest have been committed to trunk in rev:6288 A backport to the 1.4.x and 1.3.x branches is still needed.
Hide
Darius Jazayeri added a comment - 2008-12-15 20:13:30 EST

Backported to 1.4.x in rev:6289. Backported to 1.3.x in rev:6290.

Show
Darius Jazayeri added a comment - 2008-12-15 20:13:30 EST Backported to 1.4.x in rev:6289. Backported to 1.3.x in rev:6290.

People

Dates

  • Created:
    2008-12-12 18:03:11 EST
    Updated:
    2010-07-08 16:20:43 EDT
    Resolved:
    2010-07-01 22:42:53 EDT