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

When using rails 5 and trying to do an advanced search, getting "unable to convert unpermitted parameters" #794

Closed
dlbork opened this issue Aug 1, 2018 · 5 comments · Fixed by #830
Labels
Milestone

Comments

@dlbork
Copy link

dlbork commented Aug 1, 2018

ActionView::Template::Error (unable to convert unpermitted parameters to hash):
     9:     - if count == @per_page.to_i
    10:       %em.current= count
    11:     - else
    12:       = link_to count, send("redraw_#{controller.controller_name}_path", per_page: count, query: params[:query], q: params[:q]), remote: true
app/views/shared/_paginate_with_per_page.html.haml:12:in `block in _app_views_shared__paginate_with_per_page_html_haml___2598546186346708051_70070363839580'
app/views/shared/_paginate_with_per_page.html.haml:8:in `each'
app/views/shared/_paginate_with_per_page.html.haml:8:in `_app_views_shared__paginate_with_per_page_html_haml___2598546186346708051_70070363839580'
app/views/contacts/index.html.haml:17:in `_app_views_contacts_index_html_haml___2337017868419751336_88777960'
app/controllers/entities/contacts_controller.rb:16:in `index'
@CloCkWeRX
Copy link
Member

Which version(s)? of FFCRM?

Are you able to provide the post input?

@CloCkWeRX CloCkWeRX added the Bug label Nov 11, 2018
@javierhuertay
Copy link

@steveyken
Copy link
Member

I wonder if params[:query] and params[:q] have been permitted?

@CloCkWeRX
Copy link
Member

@CloCkWeRX
Copy link
Member

params[:q] appears not to be permitted in one test case I saw recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants