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.

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

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

Merged
merged 4 commits into from
Sep 18, 2024

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 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 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 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 committed Sep 18, 2024
    Configuration menu
    Copy the full SHA
    20679b3 View commit details
    Browse the repository at this point in the history