Support composite primary keys on show, edit, update and destroy actions

[filipegiusti]

Impressively, this seems to be all that's required to support composite primary keys in Rails 7.1

Description

Support composite primary keys on show, edit, update and destroy actions on Rails 7.1.

params.extract_values was introduced in Rails 7.1.0.beta1.

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

Screenshots & recording

Not really.

Manual review steps

We do have a model that uses 3 enums as primary keys and I was able to view it's show page, edit and update it after the change. To implement the change in our codebase we have overwriting set_record on each resource that has a composite primary key.

[codeclimate]

Code Climate has analyzed commit e1428c9 and detected 0 issues on this pull request.

View more on Code Climate.

[filipegiusti]

This PR probably needs a test and a check to only use params.extract_values on Rails 7.1 and above. I decided to create it right away in case others want to simple copy and paste the method in their base controller like

module Avo
  class BaseResourcesController < Avo::ResourcesController
    def set_record
      id = @resource.model_class.primary_key.is_a?(Array) ? params.extract_value(:id) : params[:id]
      @record = @resource.find_record(id, query: model_scope, params:)
      @resource.hydrate(record: @record)
    end
  end
end

[Paul-Bob]

Thanks for the contribution @filipegiusti

I let a comment with a change suggestion to avoid breaks on Rails versions that doesn't have the extract_value method.