aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers/form_helper.rb
Commit message (Collapse)AuthorAgeFilesLines
...
* Remove InstanceTagMethods module and define the methods inside the ↵Santiago Pastorino2011-01-091-7/+2
| | | | InstanceTag class
* Remove nodoc from FormBuilder because the methods inside are public API methodsRyan Bigg2010-12-131-1/+1
|
* class inheritable attributes is used no more! all internal use of class ↵Josh Kalderimis2010-11-201-2/+2
| | | | | | inheritable has been changed to class_attribute. class inheritable attributes has been deprecated. Signed-off-by: José Valim <jose.valim@gmail.com>
* Revert "Remove deprecated form_for with strings or symbols"José Valim2010-11-181-3/+9
| | | | | | | | | | | | | | | | This code was not deprecated. What was deprecated is the following: form_for(:foo, @foo) Which now should be rewritten as: form_for(@foo, :as => :foo) The following format is valid: form_for(:foo) This reverts commit be797750e6ce866ea08307f63bf35304a965c8d4.
* Remove deprecated form_for with strings or symbolsLeonardo Capillera2010-11-181-9/+3
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Add f.submit to forms just to show the most common use caseSantiago Pastorino2010-11-141-0/+12
|
* Merge branch 'spastorino-master'Santiago Pastorino2010-11-141-2/+2
|\
| * This doesn't make sense hereSantiago Pastorino2010-11-141-2/+2
| |
* | Add empty line between previous method and RDoc of submit to allow RDoc ↵Santiago Pastorino2010-11-141-0/+1
|/ | | | appear on api.rubyonrails.org
* fields_for should treat ActiveRecord::Relation as an arrayNeeraj Singh2010-11-111-2/+2
| | | | | | [#5795 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Just initialize options with an empty hashSantiago Pastorino2010-11-031-2/+1
|
* Call html_escape in ERB::Util module and don't mix it in in the helpersSantiago Pastorino2010-10-181-2/+3
|
* Revert "Make InstanceTagMethods#value_before_type_cast raise if the model ↵Santiago Pastorino2010-10-101-8/+3
| | | | | | | | don't respond to attr_before_type_cast or attr method" And "Makes form_helper use overriden model accessors" This reverts commit 3ba8e3100548f10fce0c9784981a4589531476dd and fb0bd8c1092db51888ec4bb72af6c595e13c31fa.
* Fixing search_field to remove object attribute from options hash [#5730 ↵Aditya Sanghi2010-10-021-1/+1
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* tag value can be false, so nil? check is necessaryAaron Patterson2010-09-291-1/+1
|
* no need to check for nil?Neeraj Singh2010-09-301-2/+2
|
* Avoid (@_var ||= nil) pattern by using initialize methods and ensuring ↵José Valim2010-09-291-14/+17
| | | | everyone calls super as expected.
* Ask if the instance variable is defined before asking for it, avoid *many* ↵Emilio Tagua2010-09-281-1/+5
| | | | warnings.
* Define @emitted_hidden_id if it doesn't exists and reuse it if it does.Emilio Tagua2010-09-271-1/+1
|
* We don't need to check if the format exists to generate the url, it already ↵Carlos Antonio da Silva2010-09-261-8/+7
| | | | does it internally
* More cleanup on form helpersCarlos Antonio da Silva2010-09-261-11/+12
|
* Remove last tests with deprecated form_for and cleanup form_for helperCarlos Antonio da Silva2010-09-261-4/+2
| | | | | | | This cleans up the last bits of deprecation stuff from form_for helper. However there is still a bug when using :as => foo[], with index. The classes and ids are being generated using [], such as foo[]_edit. This bug already existed but it was not detected before.
* Refactor form_for helperCarlos Antonio da Silva2010-09-261-21/+11
|
* resolves rdoc conflictXavier Noria2010-09-231-4/+6
|\
| * Document form_for behaviour when using file_field inside the blockJoost Baaij2010-09-211-1/+3
| |
| * Typosrspeicher2010-09-151-3/+3
| |
* | file_field propagates up multipart property even inside of fields_forSantiago Pastorino2010-09-201-8/+13
| |
* | removing some warningsAaron Patterson2010-09-181-2/+2
| |
* | freeze is not necessaryAaron Patterson2010-09-181-3/+3
| |
* | argument error is done for usAaron Patterson2010-09-181-2/+0
| |
* | file_field makes the enclosing form multipartSantiago Pastorino2010-09-181-19/+36
|/
* Change singular to param_key in form_helper, param_key is now used for ↵Piotr Sarnacki2010-09-111-2/+2
| | | | | | generating field names Signed-off-by: Mikel Lindsaar <raasdnil@gmail.com>
* Simplify form_for by removing *args and extract_options! (at asakusa.rb)wycats2010-09-091-15/+16
|
* Remove namespace for isolated namespaced models in formsPiotr Sarnacki2010-09-031-5/+5
|
* Added :format option for form_for helper and spec for this [#5226 ↵Alex Chrome2010-09-011-1/+9
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Speed up form_for by specializing extract_options! (at asakusa.rb)wycats2010-08-311-6/+8
|
* password_field renders with nil value by defaultSantiago Pastorino2010-08-281-5/+5
| | | | | | | | This makes the use of passwords secure by default if you want to render the value of the password_field you have to do for instance f.password_field(:password, :value => @user.password) # => <input type=password id=user_password name=user[password] value=#{@user.password} />
* type="password" for password_fieldsSantiago Pastorino2010-08-281-4/+4
|
* Make InstanceTagMethods#value_before_type_cast raise if the model don't ↵Santiago Pastorino2010-08-271-3/+8
| | | | | | respond to attr_before_type_cast or attr method [#3374] [#5471 state:committed]
* Fix label form helper to use I18n and html options, without the need of ↵Carlos Antonio da Silva2010-08-011-3/+4
| | | | | | | | | | | 'nil' text param: Before: f.label :title, nil, :class => 'title' After : f.label :title, :class => 'title' [#5267 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Makes form_helper use overriden model accessorsSantiago Pastorino2010-08-011-3/+3
| | | | [#3374 state:committed]
* doc: form_for does return output rather than merely evaluate its blockWincent Colaiuta2010-07-221-4/+0
|
* Moved a few methods from RecordIdentifier to ActiveModel::NamingPiotr Sarnacki2010-07-211-5/+5
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* remove size attribute from tag generated by file_field helper [#5084 ↵Miles Egan2010-07-131-1/+1
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Added documentation for usage of associative resources with form_forRizwan Reza2010-07-011-1/+11
|
* Do not wrap hidden fields with error proc [#4962 state:resolved]Carlos Antonio da Silva2010-06-251-1/+0
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* make text_field and hidden_field omit the value attribute if the developer ↵Jeff Dean2010-06-231-4/+4
| | | | | | explicitly passes in :value => nil [#4839 state:resolved] Signed-off-by: Michael Koziarski <michael@koziarski.com>
* adds minus to value part of id [#4862 state:resolved]RainerBlessing2010-06-221-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* removed 'unless const_defined?' code smellJosh Kalderimis2010-06-211-9/+9
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'rails/master'Xavier Noria2010-06-201-1/+1
|\ | | | | | | | | Conflicts: actionpack/lib/abstract_controller/base.rb