Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Added documentation to the FormBuilder class, should help | wangjohn | 2013-04-06 | 1 | -2/+55 |
| | | | | | clarify issue #10115. Also made the field_helpers an explicit list of methods. | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-03-30 | 1 | -16/+0 |
|\ | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb activerecord/test/cases/adapter_test.rb guides/source/testing.md [ci skip] | ||||
| * | Remove duplicated accepts_nested_attributes description | ma2gedev | 2013-03-22 | 1 | -16/+0 |
| | | | | | | | | there are just the same description about accepts_nested_attributes [ci skip] | ||||
* | | Allow pass multipart option to form_for | Grzegorz Derebecki | 2013-03-17 | 1 | -1/+1 |
|/ | |||||
* | Missing or unneeded require extract_options | Akira Matsuda | 2013-02-01 | 1 | -1/+0 |
| | |||||
* | Fix JavaScript syntax in code comment [ci skip] | Ryunosuke SATO | 2013-01-29 | 1 | -4/+4 |
| | |||||
* | Merge pull request #8623 from virusman/form_helpers_collectionproxy_fix | Jon Leighton | 2013-01-11 | 1 | -1/+1 |
|\ | | | | | Fixed nested fields bug when called with AssociationProxy | ||||
| * | Test nested fields with AssociationProxy mockup & fix AssociationProxy ↵ | virusman | 2012-12-27 | 1 | -1/+1 |
| | | | | | | | | support in form helper | ||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-01-09 | 1 | -4/+4 |
|\ \ | | | | | | | | | | | | | Conflicts: guides/source/getting_started.md | ||||
| * | | PUT => PATCH | Akira Matsuda | 2013-01-03 | 1 | -4/+4 |
| |/ | |||||
* | | Refactor the logic that checks whether or not to emit the hidden id field | Carlos Antonio da Silva | 2013-01-06 | 1 | -7/+6 |
| | | | | | | | | | | By checking for object.persisted? first, we avoid the hash lookups for new objects. | ||||
* | | Move the hidden :id field logic to where it belongs to | Carlos Antonio da Silva | 2013-01-06 | 1 | -6/+9 |
| | | | | | | | | | | | | | | | | When dealing with nested forms, Rails automatically generates a hidden field with the id value of the current object being generated by fields_for. This logic was inside the method that's available from the template object, but we just need it when really dealing with nested attributes, so moving the code to here makes more sense. | ||||
* | | Eliminate the usage of parent_builder option from form_for | Carlos Antonio da Silva | 2013-01-06 | 1 | -19/+15 |
| | | | | | | | | Just use it internally from fields_for until we come up with a better solution. | ||||
* | | Do not call fields_for from form_for, to avoid instantiating two builders | Carlos Antonio da Silva | 2013-01-06 | 1 | -5/+5 |
| | | |||||
* | | Merge pull request #8705 from amparo-luna/change_update_attributes_to_update | Guillermo Iguaran | 2013-01-03 | 1 | -4/+4 |
|\ \ | | | | | | | Rename update_attributes method to update | ||||
| * | | Change docs to use update instead of update_attributes | Amparo Luna + Guillermo Iguaran | 2013-01-03 | 1 | -4/+4 |
| |/ | |||||
* | | take into account multipart when calculating tag attributes | Aaron Patterson | 2013-01-02 | 1 | -4/+7 |
| | | |||||
* | | ask the form builder for form tag attributes | Aaron Patterson | 2013-01-02 | 1 | -2/+3 |
| | | |||||
* | | do not mutate the options hash (when we do not need to) | Aaron Patterson | 2013-01-02 | 1 | -2/+2 |
|/ | |||||
* | repeating documentation for form helpers | Thiago Pinto | 2012-12-09 | 1 | -18/+406 |
| | |||||
* | API reader should look elsewhere for helper instructions | Thiago Pinto | 2012-12-06 | 1 | -0/+24 |
| | |||||
* | complementary options guidelines for f.file_field and file_field_tag | Thiago Pinto | 2012-12-06 | 1 | -0/+6 |
| | |||||
* | adding example for f.file_input | Thiago Pinto | 2012-12-06 | 1 | -0/+3 |
| | |||||
* | adding example for f.file_input | Thiago Pinto | 2012-12-06 | 1 | -0/+3 |
| | |||||
* | correct bad jquery syntax | Thiago Pinto | 2012-12-06 | 1 | -2/+2 |
| | |||||
* | copy edits [ci skip] | Vijay Dev | 2012-12-04 | 1 | -2/+1 |
| | |||||
* | Cleans and removes 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+0 |
| | |||||
* | Fixes wrong typo on FormHelper [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | Cleans documentation from Helpers [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+3 |
| | |||||
* | document :hidden_field_id option for fields_for | Yves Senn | 2012-11-24 | 1 | -0/+5 |
| | |||||
* | Make caller attribute in deprecation methods optional | Alexey Gaziev | 2012-10-30 | 1 | -2/+1 |
| | |||||
* | Provide a call stack for deprecation warnings where needed. | Nikita Afanasenko | 2012-10-29 | 1 | -2/+2 |
| | | | | It's sometimes hard to quickly find where deprecated call was performed, especially in case of migrating between Rails versions. So this is an attempt to improve the call stack part of the warning message by providing caller explicitly. | ||||
* | More Ruby 1.9 hash syntax. | Rafael Mendonça França | 2012-10-06 | 1 | -52/+47 |
| | |||||
* | We don't need to check blank? here. | Rafael Mendonça França | 2012-10-06 | 1 | -1/+1 |
| | | | | | | | | Also the blank? check introduced a bug. $ rails generate model Foo blank:boolean form_for(Foo.new(:blank => true)) => ArgumentError, "First argument in form cannot contain nil or be empty" | ||||
* | copy edits 137e5d9 | Xavier Noria | 2012-09-18 | 1 | -1/+1 |
| | |||||
* | Add extra documentation for password_field | Arek W | 2012-09-18 | 1 | -1/+1 |
| | | | | I think this should be explicit as the password fields behaviour is inconsistent with other fields in this regard. It had me scratching my head until I dug through the source code. | ||||
* | Add ActionView::ModelNaming | Piotr Sarnacki | 2012-08-28 | 1 | -3/+3 |
| | | | | | | | It's just a duplicate of ActionController::ModelNaming. These are just a simple helpers for decoupling Active Model, so it does not make sense to extract it to Active Support, but the point is to decouple also Action View and Action Pack | ||||
* | Fix documentation mistakes | Janko Marohnić | 2012-08-21 | 1 | -3/+3 |
| | |||||
* | check for nil or empty record in form_for | schneems | 2012-08-10 | 1 | -0/+1 |
| | | | | if nil or an empty array is passed into form_for you get a horrible error message, this one is much more indicative of what the programmer needs to know to fix the problem. | ||||
* | Allow data attributes to be set as a first-level option for form_for, so you ↵ | David Heinemeier Hansson | 2012-08-08 | 1 | -0/+19 |
| | | | | can write `form_for @record, data: { behavior: 'autosave' }` instead of `form_for @record, html: { data: { behavior: 'autosave' } }` *DHH* | ||||
* | load active_support/deprecation in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | load active_support/core_ext/class/attribute in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | Remove ActiveModel dependency from ActionPack | Guillermo Iguaran | 2012-06-30 | 1 | -13/+8 |
| | | | | | | | | ActiveModel is used in ActionPack for ActiveModel::Naming for a few, mostly optional aspects of ActionPack related to automatically converting an ActiveModel compliant object into a key for params and routing. It uses only three methods of ActiveModel (ActiveModel::Naming.route_key, ActiveModel::Naming.singular_route_key and ActiveModel::Naming.param_key). | ||||
* | fixes a few mistakes in api docs [ci skip] | Vijay Dev | 2012-06-22 | 1 | -1/+1 |
| | |||||
* | Add rdoc to phone_field | Gaurish Sharma | 2012-06-16 | 1 | -0/+1 |
| | | | [API docs](http://api.rubyonrails.org/classes/ActionView/Helpers/FormHelper.html#method-i-phone_field) for `phone_field` provide no information on what the method actually does. no description nor access to source code. so added this | ||||
* | accept a block in button helper. | Yuki Nishijima | 2012-06-05 | 1 | -2/+13 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-05-23 | 1 | -1/+1 |
|\ | |||||
| * | Revert "Remove blank trailing comments" | Vijay Dev | 2012-05-23 | 1 | -0/+11 |
| | | | | | | | | | | | | | | | | | | | | This reverts commit fa6d921e11363e9b8c4bc10f7aed0b9faffdc33a. Reason: Not a fan of such massive changes. We usually close such changes if made to Rails master as a pull request. Following the same principle here and reverting. [ci skip] | ||||
| * | Fix incorrect example for fields_for - without the '=' it will not output ↵ | David Morton | 2012-05-22 | 1 | -1/+1 |
| | | | | | | | | anything. |