Skip to content

Support record binding from API actions using JSON#7837

Merged
labkey-adam merged 10 commits into
developfrom
fb_json_records
Jul 14, 2026
Merged

Support record binding from API actions using JSON#7837
labkey-adam merged 10 commits into
developfrom
fb_json_records

Conversation

@labkey-adam

@labkey-adam labkey-adam commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Rationale

We want more actions to support form binding to Java records. https://github.com/LabKey/internal-issues/issues/1277

Related Pull Requests

Comment thread api/src/org/labkey/api/action/BaseViewAction.java Outdated

@labkey-matthewb labkey-matthewb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I've spent a lot of time chasing down errors that change depending on the exact code path. I'd like to see a small record/class test that shows that the errors are the same.

@labkey-adam

Copy link
Copy Markdown
Contributor Author

I've spent a lot of time chasing down errors that change depending on the exact code path. I'd like to see a small record/class test that shows that the errors are the same.

They are probably as aligned as they're going to get. See BindingTestCase.

Of course, the missing parameter error does not apply to form beans, since these members are always initialized to their default values. The conversion error is now consistent across both, though.

@labkey-adam labkey-adam merged commit 6d03722 into develop Jul 14, 2026
8 checks passed
@labkey-adam labkey-adam deleted the fb_json_records branch July 14, 2026 23:22
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.

2 participants