XForms Module

Marking TRAC-1994 as a duplicate of this ticket.

Jaime, can you attach a .patch file to this ticket for Daniel to review/apply?

Patch file which edits the scripts for 'scheduler_task_config' table to accomodate the column UUID

Daniel, can you review/apply this patch?

The xforms module will not load in openmrs 1.5 or 1.6 without this patch.

This patch makes the module throw exceptions for versions of openmrs lower than 1.6 which do not have the uuid column.

Hi Daniel,

Just FYI, for the HTML Form Entry module I actually needed to have 1.4.x, 1.5.x, and 1.6.x branches, to get things to work.

Thanks Darius for sharing that what you have done with the HTML Form Entry module.

How do you compare that approach with just doing only the problematic sql statements updates in the Activator, where i would know whether we have the uuid field or not?

If you're just talking about creating scheduler tasks, it makes sense to do that in the activator, yes.

(Because when they upgrade from 1.4 to 1.5 that should automatically upgrade scheduler_task_config without you having to do anything.)

You can't fix this in the activator because the module doesn't even load in 1.5. The sqldiff is run before the activator and the sqldiff is where the problem is. The uuid column is a "non-null unique" column and must be filled in when creating those rows.

Doing sql updates in the activator is not recommended anyway. The solution is to have a 1.4.x-compatible branch off of your current code in your module. Then in trunk its 1.5+ compatible, so you set the openmrs version in your config.xml to 1.5.0.

Marking as Needs Review so Daniel can weigh in on if he wants to create the 1.5.x-compatible branch or not.

Great! Marking this as fixed in the 3.8.0 release of the xforms module.