Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: EXPOSED-547 idParam() registers composite id value with a single placeholder #2242

Merged
merged 4 commits into from
Sep 18, 2024
Merged

fix: EXPOSED-547 idParam() registers composite id value with a single placeholder #2242

merged 4 commits into from
Sep 18, 2024
+61 −26

Commits on Sep 18, 2024

  1. fix: idParam with CompositeID value registers single argument 

    Current idParam registers the value from an entity id column with a single placeholder
marker. This fix ensures that a placeholder is registered for every component
value stored by CompositeID and that this argument registeration is overriden
and handled by the mapping operator as needed.
    @bog-walk
    bog-walk committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    cee521f View commit details
    Browse the repository at this point in the history

  2. fix: EXPOSED-547 idParam() registers composite id value with a single… 

    … placeholder

Replace creation of new QueryParameters with direct calls to registerArgument()
    @bog-walk
    bog-walk committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    09e882b View commit details
    Browse the repository at this point in the history

  3. fix: EXPOSED-547 idParam() registers composite id value with a single… 

    … placeholder

- Refactor test usage of idParam()
    @bog-walk
    bog-walk committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    2565626 View commit details
    Browse the repository at this point in the history

  4. fix: EXPOSED-547 idParam() registers composite id value with a single… 

    … placeholder

- Fix type mismatch in tests due to alias.
    @bog-walk
    bog-walk committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    20679b3 View commit details
    Browse the repository at this point in the history