aboutsummaryrefslogtreecommitdiffstats
path: root/actionview/lib
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge pull request #19844 from stevenspiel/link_to_if_block_helper_additionYves Senn2015-05-011-1/+1
|\ \ \ | | | | | | | | | | | | Update url_helper.rb
| * | | Update url_helper.rbSteven Spiel2015-04-211-1/+1
| |/ / | | | | | | add block to link_to_if when condition is true
* | | Make ActionMailer #cache helper a no-op, not an exceptionJavan Makhmali2015-04-281-1/+1
| | |
* | | Correct translate helper docs [ci skip]Derek Prior2015-04-241-23/+36
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The documentation previously stated that calling `translate(".foo")` was equivalent to calling `I18n.translate("people.index.foo")` which is incorrect due to the various other functions of the `translate` view helper. This has been fixed. Additionally, a note about forcing the view helper to re-raise exceptions was added to the section detailing the handling of missing translations. Other cleanup includes: * Consistent formatting of code * Stop indenting bulleted list as a code sample * Tighten some of the language * Wrap at 80 characters.
* | fix typo asser url [ci skip]Mehmet Emin İNAÇ2015-04-161-6/+6
| |
* | Merge pull request #19774 from davydovanton/doc-update-asser-urlAbdelkader Boudih2015-04-151-0/+30
|\ \ | | | | | | [Skip ci] Add information about :host options
| * | [Skip ci] Add information about :host optionsAnton Davydov2015-04-161-0/+30
| | | | | | | | | | | | | | | | | | Added informations about :host options for each url helper (#javascript_url, #video_url, #stylesheet_url, etc) based on #asser_url method.
* | | Merge pull request #19736 from kmcphillips/masterRafael Mendonça França2015-04-142-1/+4
|\ \ \ | |_|/ |/| | Set default form builder for a controller
| * | Override default form builder for a controllerKevin McPhillips2015-04-132-1/+4
| | |
* | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2015-04-141-1/+1
|\ \ \ | |/ / |/| |
| * | [ci skip] Add `,`yui-knk2015-04-081-1/+1
| | |
* | | Merge pull request #19566 from aditya-kapoor/remove-dup-docRichard Schneeman2015-04-111-22/+10
|\ \ \ | | | | | | | | [ci skip] remove duplicate doc for current_page?
| * | | [ci skip] remove duplicate doc for current_page?Aditya Kapoor2015-04-091-22/+10
| | | |
* | | | Merge pull request #17512 from ↵Zachary Scott2015-04-101-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | JackDanger/danger/use-default-view-pattern-in-docs [docs] Using the real resolver pattern in docs
| * | | | Using the real resolver pattern in docsJack Danger Canty2014-11-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If someone copies the docs into their app they'll find it simply doesn't work because the locale pattern doesn't have the same logic. This makes the doc examples work exactly as written. [ci skip]
* | | | | Merge pull request #19685 from vngrs/actionview_parent_layout_bugRafael Mendonça França2015-04-081-8/+17
|\ \ \ \ \ | |_|_|_|/ |/| | | | fix for actionview parent layout bug
| * | | | fix for actionview parent layout bugMehmet Emin İNAÇ2015-04-081-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes issue #19626 Don't need to check layout conditions if there is no condition test for parent layout bug fix
* | | | | Merge pull request #19661 from Azdaroth/fields-for-child-index-lambdaRafael Mendonça França2015-04-061-1/+5
|\ \ \ \ \ | |/ / / / |/| | | | Accept lambda as child_index option in #fields_for method
| * | | | Accept lambda as child_index option in #fields_for methodKarol Galanciak2015-04-051-1/+5
| | | | |
* | | | | head no_content when there is no template or action performedStephen Bussey2015-04-051-1/+1
|/ / / /
* | | | Merge pull request #19644 from y-yagi/auto_discovery_link_tag_docYves Senn2015-04-051-1/+1
|\ \ \ \ | | | | | | | | | | correct output of auto_discovery_link_tag [ci skip]
| * | | | correct output of auto_discovery_link_tag [ci skip]yuuji.yaginuma2015-04-041-1/+1
| | | | |
* | | | | Allow an array to be a default translation value.Adam Prescott2015-04-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4.2.1 introduced a change to the way `translate`/`t` works with an option of `default: [[]]`. In 4.2.0, this would give a default value of `[]`, but in 4.2.1, it leads to a missing translation. `default: [[]]` is again allowed for cases where a default of `[]` is needed. This addresses GitHub issue 19640.
* | | | | [skip ci] Update information about #link_to attributesAnton Davydov2015-04-041-4/+3
|/ / / /
* | | | Freeze static arguments for gsubbrainopia2015-04-021-1/+1
| | | |
* | | | Prefer string patterns for gsubbrainopia2015-04-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/ruby/ruby/pull/579 - there is a new optimization since ruby 2.2 Previously regexp patterns were faster (since a string was converted to regexp underneath anyway). But now string patterns are faster and better reflect the purpose. Benchmark.ips do |bm| bm.report('regexp') { 'this is ::a random string'.gsub(/::/, '/') } bm.report('string') { 'this is ::a random string'.gsub('::', '/') } bm.compare! end # string: 753724.4 i/s # regexp: 501443.1 i/s - 1.50x slower
* | | | [skip ci] Add information about #link_to target optionAnton Davydov2015-04-011-0/+6
| |/ / |/| |
* | | Merge pull request #19535 from JuanitoFatas/doc/av-commentYves Senn2015-03-271-1/+1
|\ \ \ | | | | | | | | [ci skip] Unify the code comment in ActionView::Renderer#render_template...
| * | | [ci skip] Unify the code comment in ActionView::Renderer#render_template.Juanito Fatas2015-03-271-1/+1
| | |/ | |/|
* / | Add missing :html option in determine_template error message.Juanito Fatas2015-03-271-1/+1
|/ /
* | Merge branch 'master' of github.com:rails/docrailsVijay Dev2015-03-263-10/+11
|\ \ | | | | | | | | | | | | Conflicts: guides/source/4_0_release_notes.md
| * | [ci skip] Fix `&>` with `%>`yui-knk2015-03-251-9/+9
| | |
| * | [ci skip] Add a line between code example and docsyui-knk2015-03-211-0/+1
| | |
| * | [ci skip] Add space after erb block.yui-knk2015-03-121-1/+1
| | |
* | | Simplify setting button form optionsCarlos Antonio da Silva2015-03-221-2/+3
| | | | | | | | | | | | No need to merge hashes when simply setting options does the job.
* | | Rework form helper example to use Person.newCarlos Antonio da Silva2015-03-211-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous change 92a3c8dcdf174dab5e8759031b6bbe6cd891fe3c improved this example, but calling .create + #valid? ends up running validations twice, and we don't want to showcase that. Lets use the normal use case of building a new object and calling #save on it, which is what a basic scaffold with Active Record does. [ci skip]
* | | [ci skip] Check a result of `valid?` instead of `create`yui-knk2015-03-211-1/+2
| | |
* | | Merge pull request #19421 from jcoyne/translate_defaults_with_nilRafael Mendonça França2015-03-201-1/+1
| | | | | | | | | | | | Strip nils out of default translations. Fixes #19419
* | | Remove additional handling of boolean attributes on button to helperCarlos Antonio da Silva2015-03-181-30/+0
| | | | | | | | | | | | | | | | | | | | | | | | This logic was just doing duplicated work, since the button_to helper relies on tag/content_tag to generate the button html, which already handles all boolean attributes it knows about. The code dates back to 2005: 43c470fae468ef63e0d5c3dc1e202925685fd47b.
* | | [ci skip] correct output of selectKuldeep Aggarwal2015-03-131-15/+15
| | |
* | | [CI SKIP] Correct output of options_for_selectAkshay Vishnoi2015-03-131-5/+5
| | |
* | | Let strip_tags leave HTML escaping to Rails.Kasper Timm Hansen2015-03-101-1/+1
| | | | | | | | | | | | Prevents double escaping errors, such as "&" becoming "&".
* | | don't need to rescue Exception in this caseAaron Patterson2015-03-051-1/+1
| | |
* | | [ci skip] Delete excess white in documentationAnton Davydov2015-03-051-1/+1
| | |
* | | [ci skip] Add information about sanitize option in TextHelper#highlightAnton Davydov2015-03-051-1/+6
| | |
* | | Merge pull request #19144 from y-yagi/fix_streaming_bufferRafael Mendonça França2015-03-041-1/+3
|\ \ \ | | | | | | | | fix ActionView::Template::Error when using Streaming with capture.
| * | | fix ActionView::Template::Error when using Streaming with capture.yuuji.yaginuma2015-03-011-1/+3
| |/ / | | | | | | | | | can't acquire a encoding from StreamingBuffer. fixes #12001
* | | Fix spelling [ci skip]Yu Haidong2015-03-041-8/+8
| | |
* | | move _generate_paths_by_default to where it is usedAaron Patterson2015-03-011-0/+6
|/ / | | | | | | | | _generate_paths_by_default wasn't used in AD::Routing::UrlFor, so we should be able to move it where it is used in AV::Routing
* | Fix regression when passing a value different of String.Ulisses Almeida2015-02-261-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous version of rails(4.2.0) you can pass objects to the default option of translation helper. For example: ```ruby t('foo', default: 1) ``` But on rails 4.2.1 version this kind of use stopped to work, because started only to accept String types. Now with this fix we can use orther value types on this helper again.