Skip to content

e+05 Resolution#600

Open
Olawemi5 wants to merge 2 commits into
OuhscBbmc:mainfrom
Olawemi5:main
Open

e+05 Resolution#600
Olawemi5 wants to merge 2 commits into
OuhscBbmc:mainfrom
Olawemi5:main

Conversation

@Olawemi5

@Olawemi5 Olawemi5 commented Apr 16, 2026

Copy link
Copy Markdown

Summary

This fixes REDCap record import serialization for larger numeric record IDs.

Before this change, IDs such as 100000, 200000, and 400000 could be serialized in scientific notation during CSV upload, which could cause REDCap import failures or incorrect record identification.

What Changed

  • Added an internal CSV serialization helper for write/import operations.
  • Forced non-scientific numeric formatting during upload serialization.
  • Updated redcap_write_oneshot() to use the helper.
  • Added a focused regression test covering large record ID serialization.

Why This Is Small and Safe

This change is limited to the CSV generation path used for record writes. It does not change the API contract, request shape, or read behavior.

Example

Values like these are now serialized as plain values instead of scientific notation:

  • 100000
  • 200000
  • 400000

@Olawemi5 Olawemi5 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like these changes checked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant