aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Cover one more case in auth_token and remote formsPiotr Sarnacki2012-03-281-1/+1
| | | | | | | | | | | | | | | | | | | | If embedding auth_token in remote forms is off and we pass a value for auth_token it should respect it.
* | | | config.action_view.embed_authenticity_token_in_remote_forms is true by defaultPiotr Sarnacki2012-03-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed default value for `config.action_view.embed_authenticity_token_in_remote_forms` to `false`. This change breaks remote forms that need to work also without javascript, so if you need such behavior, you can either set it to `true` or explicitly pass `:authenticity_token => true` in form options
* | | | Added config.action_view.embed_authenticity_token_in_remote_formsPiotr Sarnacki2012-03-281-8/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is a regression introduced in 16ee611fa, which breaks remote forms that should also work without javascript. This commit introduces config option that allows to configure this behavior defaulting to the old behavior (ie. include authenticity token in remote forms by default) Conflicts: actionpack/CHANGELOG.md
* | | | Merge pull request #5621 from rafaelfranca/fix-2492-masterJeremy Kemper2012-03-271-3/+3
|\ \ \ \ | | | | | | | | | | Fix label_tag to merge the options hash with the object hash
| * | | | Check if the options hash already exists and merge it with the anotherRafael Mendonça França2012-03-272-12/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hash. Closes #2492 and #5615
| * | | | Fixing issue #2492 for master branch. ActionView::Base.field_error_proc ↵Andrey Samsonov2012-03-271-2/+9
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | doesn't call for label. objectify_options method should be applied to the proper options arg. See explanation and example of the bug - https://github.com/kryzhovnik/rails_field_error_proc_bug_example
* / / / Don't break Haml with textarea newline fix.James Coleman2012-03-272-2/+6
|/ / / | | | | | | | | | | | | | | | See issue #393, issue #4000, issue #5190, and issue #5191. Adds a newline after the textarea opening tag based on @codykrieger's original patch so that we don't cause regressions in Haml-using apps. The regression caused textarea tags to add newlines to the field unintentionally (each update/save added an extra newline.) Also fix 6 more tests that didn't yet have the newline expectation.
* | | change number_with_precision to not return negative zerosTim McEwan2012-03-271-0/+1
| | |
* | | Merge pull request #5561 from carlosantoniodasilva/form-builder-block-argAaron Patterson2012-03-261-3/+9
|\ \ \ | | | | | | | | Properly deprecate the block argument in AV FormBuilder
| * | | Properly deprecate the block argument in AV FormBuilderCarlos Antonio da Silva2012-03-231-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 56089ca986c767763f29159c8de0aa1ebabfd4d2 introduced a backward incompatible change by changing the method signature of the FormBuilder. This brings back the same method signature warning in case someone gives a block to the builder. More info: https://github.com/rails/rails/commit/56089ca986c767763f29159c8de0aa1ebabfd4d2#commitcomment-1116166
* | | | apply form_for namespace option to date_selectSergey Nartimov2012-03-261-1/+4
| | | |
* | | | Merge pull request #5183 from lest/patch-1José Valim2012-03-261-2/+3
|\ \ \ \ | | | | | | | | | | use content_tag in button_to helper
| * | | | use content_tag in button_to helperSergey Nartimov2012-03-141-2/+3
| | | | |
* | | | | Remove code duplicationRafael Mendonça França2012-03-191-11/+11
| |/ / / |/| | |
* | | | Fixes issue #5222: DateTimeSelector builds invalid dates on leap day when ↵Jey Balachandran2012-03-181-6/+6
| | | | | | | | | | | | | | | | discarding both day and month.
* | | | time_tag support for blocksAntonio Tapiador del Dujo2012-03-181-2/+7
| | | |
* | | | Allow you to force the authenticity_token to be rendered even on remote ↵David Heinemeier Hansson2012-03-141-5/+9
| | | | | | | | | | | | | | | | forms if you pass true
* | | | Do not include the authenticity token in forms where remote: true as ajax ↵David Heinemeier Hansson2012-03-141-2/+9
|/ / / | | | | | | | | | forms use the meta-tag value
* | | Merge pull request #572 from sikachu/remove_activeresourceJosé Valim2012-03-131-1/+1
|\ \ \ | | | | | | | | Remove Active Resource source files from the repository.
| * | | Remove Active Resource source files from the repositoryPrem Sichanugrist2012-03-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dear Active Resource, It's not that I hate you or anything, but you didn't get much attention lately. There're so many alternatives out there, and I think people have made their choice to use them than you. I think it's time for you to have a big rest, peacefully in this Git repository. I will miss you, @sikachu.
* | | | add include_hidden option to checkbox tagSergey Nartimov2012-03-131-2/+8
|/ / /
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-131-66/+138
|\ \ \
| * | | Revised explanation of fields_for usage.Mark Thomson2012-03-111-10/+39
| | | | | | | | | | | | | | | | Minor tweak to previous comments on form_for and form_helper.rb preamble
| * | | Added clarification to description of how initial form values are derived.Mark Thomson2012-03-101-1/+2
| | | |
| * | | Revised explanation of form_for usageMark Thomson2012-03-101-53/+84
| | | |
| * | | Additional preamble comments in form_helper.rb.Mark Thomson2012-03-081-7/+18
| | | |
| * | | Small correction to the description of the role of the form builder.Mark Thomson2012-03-081-1/+1
| | | | | | | | | | | | | | | | (endpoints are generated by form_for without reference to the form builder)
* | | | add 'include_hidden' option to select tag, closes #5402Vasiliy Ermolovich2012-03-132-1/+4
| | | |
* | | | Merge pull request #5380 from benmmurphy/escape_unicode_paragraph_separatorAaron Patterson2012-03-121-1/+3
|\ \ \ \ | | | | | | | | | | ensure u2029 is escaped in escape_javascript helper
| * | | | ensure u2029 is escaped in escape_javascript helperbenmmurphy2012-03-111-1/+3
| |/ / /
* | | | Fix unintended removal of 'cols' from a text_area example.Philip Arndt2012-03-121-1/+1
| | | |
* | | | Fixes #5324 by removing default size options from input:text and default ↵Philip Arndt2012-03-104-22/+17
|/ / / | | | | | | | | | cols and rows options from textarea.
* | | simplify namespace assignment in fields_forSergey Nartimov2012-03-021-1/+1
| | |
* | | Merge pull request #5242 from rails/opt_routesJosé Valim2012-03-021-9/+14
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimize routes generation in simple cases. If you pass to the route helper the same amount of arguments as the required segments, route generation will be optimized as a string interpolation. After this commit, `post_path(post)` is about 6.5 times faster, `post_url(post)` is about 5 times.
| * | | Optimize url helpers.Sergey Nartimov + José Valim2012-03-021-9/+14
| | | |
* | | | Allow fieldsets to be created without a block.Ben Pickles2012-03-011-2/+1
|/ / /
* | | Merge branch 'master-security'Aaron Patterson2012-03-012-4/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master-security: Ensure [] respects the status of the buffer. delete vulnerable AS::SafeBuffer#[] use AS::SafeBuffer#clone_empty for flushing the output_buffer add AS::SafeBuffer#clone_empty fix output safety issue with select options Conflicts: actionpack/lib/action_view/helpers/tags/base.rb
| * | | use AS::SafeBuffer#clone_empty for flushing the output_bufferAkira Matsuda2012-02-201-1/+1
| | | |
| * | | fix output safety issue with select optionsSergey Nartimov2012-02-201-3/+3
| | | |
* | | | Change asset_host config language to be more helpful and accurateGabe Hollombe2012-02-281-2/+3
| | | |
* | | | Add a new line after the textarea opening tag.Rafael Mendonça França2012-02-271-1/+1
| | | | | | | | | | | | | | | | Closes #393
* | | | Fix collection_check_boxes and collection_radio_buttons when using localRafael Mendonça França2012-02-261-0/+1
| | | | | | | | | | | | | | | | variables in the form builder
* | | | Add documentation to object method of CollectionHelpers::BuilderRafael Mendonça França2012-02-261-6/+6
| |/ / |/| |
* | | use content_tag instead stringsVasiliy Ermolovich2012-02-251-3/+4
| | |
* | | Merge pull request #5117 from nashby/form-option-refactorJosé Valim2012-02-251-17/+11
|\ \ \ | | | | | | | | FormOptionsHelper refactor
| * | | refactor option_text_and_value and option_value_selected? methodsVasiliy Ermolovich2012-02-211-10/+3
| | | |
| * | | refactor time_zone_options_for_selectVasiliy Ermolovich2012-02-211-7/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | BTW, select works quite faster then find_all: require 'benchmark' n = [1]*100_000_000 Benchmark.bm do |x| x.report { n.select { |a| a > 1 } } x.report { n.find_all { |a| a > 1 } } end user system total real 7.590000 0.010000 7.600000 ( 7.927171) 9.650000 0.010000 9.660000 ( 9.634406)
* | | consistently mention first patch, then putXavier Noria2012-02-241-1/+1
| | | | | | | | | | | | | | | | | | There was a mix, sometimes patch first, sometimes put first. Use always patch first, since this is going to be the primary verb for updates.
* | | uses PATCH for the forms of persisted records, and routes PATCH and PUT to ↵Xavier Noria2012-02-241-1/+1
| | | | | | | | | | | | the update action of resources
* | | correct fetching :name option in form fieldsVasiliy Ermolovich2012-02-231-3/+3
| | |