Skip to content

Commit

Permalink
permit domain names nested params
Browse files Browse the repository at this point in the history
This resolves a bug where the user could not update their domains via the ui.
  • Loading branch information
Shana Moore committed Jan 17, 2024
1 parent 94ce542 commit b083885
Showing 1 changed file with 14 additions and 14 deletions.
28 changes: 14 additions & 14 deletions app/controllers/proprietor/accounts_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -93,20 +93,20 @@ def ensure_admin!

# Never trust parameters from the scary internet, only allow the permitted parameters through.
def edit_account_params
params.require(:account).permit(:name,
:cname,
:title,
:is_public,
:search_only,
*@account.live_settings.keys,
admin_emails: [],
full_account_cross_searches_attributes: [:id,
:_destroy,
:full_account_id,
full_account_attributes: [:id]],
solr_endpoint_attributes: %i[id url],
fcrepo_endpoint_attributes: %i[id url base_path],
data_cite_endpoint_attributes: %i[mode prefix username password])
params.require(:account).permit(
:name,
:cname,
:title,
:is_public,
:search_only,
*@account.live_settings.keys,
admin_emails: [],
full_account_cross_searches_attributes: [:id, :_destroy, :full_account_id, full_account_attributes: [:id]],
solr_endpoint_attributes: %i[id url],
fcrepo_endpoint_attributes: %i[id url base_path],
data_cite_endpoint_attributes: %i[mode prefix username password],
domain_names_attributes: %i[id tenant cname is_active _destroy]
)
end

def account_params
Expand Down

0 comments on commit b083885

Please sign in to comment.