Skip to content

Handle array values of belongs_to primary_key option

Rodrigo Muino Tomonari requested to merge github/fork/Slotos/belongs-to-composite-primary-key into main

Created by: Slotos

Motivation / Background

Resolves #50850 (closed)

Resolves #51127 by allowing users to set all query constraints on association definitions, leaving model in a state compatible with active_storage (and active_text) associations.

Detail

This PR merges handling of primary keys by belongs_to association and has_ associations into one, rendering aforementioned associations' behaviour expectedly symmetrical.

Additional information

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one change. Changes that are unrelated should be opened in separate PRs.
  • Commit message has a detailed description of what changed and why. If this PR fixes a related issue include it in the commit message. Ex: [Fix #issue-number]
  • Tests are added or updated if you fix a bug or add a feature.
  • CHANGELOG files are updated for the changed libraries if there is a behavior change or additional feature. Minor bug fixes and documentation changes should not be included.

Merge request reports