Merge pull request #3724 from why-el/no-instance-for-strong-params

Pass project_id instead of project instance to version_attributes
pull/3441/merge
Stefan Botzenhart 9 years ago
commit 5ff64caa84
  1. 2
      app/models/permitted_params.rb
  2. 2
      spec/models/permitted_params_spec.rb

@ -327,7 +327,7 @@ class PermittedParams
:status,
:sharing,
:custom_field_value,
version_settings_attributes: [:id, :display, :project])
version_settings_attributes: [:id, :display, :project_id])
end
def comment

@ -254,7 +254,7 @@ describe PermittedParams, type: :model do
acceptable_params_with_data.merge!(version_settings_attributes: {id: '1',
display: '2',
project: '3'})
project_id: '3'})
params = ActionController::Parameters.new(version: acceptable_params_with_data)

Loading…
Cancel
Save