search

ctsit / search_and_populate_data_from_another_project

REDCap Module to search another project for data to populate data into the current form
Other
5 stars 2 forks source link

fix php call causing breakage as early as redcap v10.5.1 #20

Closed ChemiKyle closed 3 years ago

ChemiKyle commented 3 years ago

hardcode arm arg in enableDataSearchAutocomplete call

~TODO: README update~

michael-bentz commented 3 years ago

I forgot, where can I find a sample project to test this?

ChemiKyle commented 3 years ago

You'll want to use the Cases and Person Search projects from redcap-phone, use Person Search as the source project and Cases as the target with the mapping files under examples/*.json. All the module set up will be in the target project only, though you'll need to provide some seed data to Person Search (manually entering a few is fine but you can script it if you like by pulling from the STP DB's person table and sample_frac'ing).
We can do a video call if you need some help setting up.

Note that the bug this fixes occurs when trying to pipe data, searching still works.

michael-bentz commented 3 years ago

Was multi-arm supported in previous versions of REDCap and break with a newer release? Maybe the pr can renamed to "Remove multi-arm support" instead.

ChemiKyle commented 3 years ago

No, arms were never supported, not mentioning that was an oversight.

michael-bentz commented 3 years ago

I understand, this is a fix to get the module up and running. Multi-arm support can either be added or removed entirely later via: https://github.com/ctsit/search_and_populate_data_from_another_project/blob/37613f41893742102248199ddf1a9f6e8d22b79b/js/custom_data_search.js#L22