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

Copy not functioning with shared keys #3

Closed jrpence closed 4 years ago

jrpence commented 4 years ago

If the target keys and the source keys were identical between the source and target then the copy within the array would hit an error. The recursive search through the array would also have an issue, that wouldn't resolve the search. By separation of the source and target arrays the reads, writes, and unsetting of data no longer interferes with themselves.

ChemiKyle commented 4 years ago

Good catch, @jrpence! I can confirm I'm able to recreate the bug and that this PR fixes it.

ChemiKyle commented 4 years ago

Most of the diff is just noise from style differences, merging.