Skip to content
This repository has been archived by the owner on Nov 29, 2017. It is now read-only.

Migrate/Remove Plugin: dynamic_form #279

Closed
jasonkliu opened this issue Jun 26, 2014 · 2 comments
Closed

Migrate/Remove Plugin: dynamic_form #279

jasonkliu opened this issue Jun 26, 2014 · 2 comments
Assignees

Comments

@jasonkliu
Copy link
Contributor

README of dynamic_form plugin

DynamicForm holds a few helpers method to help you deal with your models, they are:

  • input(record, method, options = {})
  • form(record, options = {})
  • error_message_on(object, method, options={})
  • error_messages_for(record, options={})

It also adds f.error_messages and f.error_messages_on to your form builders.

@jasonkliu
Copy link
Contributor Author

Looked for occurrences with grep (all others don't return everything):

$ grep -riIn "error_messages_for" * --exclude-dir=vendor

app/views/categories/index.html.erb:1:<%= error_messages_for :category %>
app/views/notices/_error_messages.html.erb:1:  <%= error_messages_for 'sticky', 'announcement', 'link' %>
app/views/payforms/show.html.erb:11:     <div id="cols-header"><%= error_messages_for 'payform' %></div>
app/views/requested_shifts/_error_messages.html.erb:1:  <%= error_messages_for 'requested_shift' %>
app/views/sub_requests/_error_messages.html.erb:1:  <%= error_messages_for 'sub_request' %>
app/views/template_time_slots/_error_messages.html.erb:1:<%= error_messages_for 'template_time_slot' %>

I think that error_messages_for might be a built-in method (http://apidock.com/rails/ActionView/Helpers/ActiveRecordHelper/error_messages_for)...
Dynamic Form is now available as a gem here. I'm going to try to delete the plugin first, then if needed I'll re-add it as a gem.

@jasonkliu
Copy link
Contributor Author

Right, it's totally important to keep dynamic_form because it adds an important feature, which is
f.error_messages. The error_messages method is deprecated in Rails 3, but we can bring it back with the Dynamic Form gem. This issue is ready to close when the pull request #258 is accepted.

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

No branches or pull requests

1 participant