aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_view/helpers
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #1778 from spohlenz/hash-modelsJosé Valim2011-06-221-1/+1
|\ | | | | Fix nested fields_for when Hash-based model is passed.
| * Test for extractable_options? within nested fields_for.Sam Pohlenz2011-06-201-1/+1
| | | | | | | | | | This fixes an error when a record object that is a subclass of Hash is passed to fields_for, which is incorrectly interpreted as field options.
* | Rename AV::Helpers::AssetPaths to AV::AssetPaths to solve autoload mess.José Valim2011-06-212-81/+4
| |
* | Only mark the sliced buffer as safe, if it was safe to begin with.Christopher Meiklejohn2011-06-201-1/+2
| |
* | Streamline fragment_for SafeBuffer logic to make it more concise.Christopher Meiklejohn2011-06-201-6/+3
| |
* | Do not change a frozen text passed to simple_format text helperTadas Tamosauskas2011-06-201-0/+1
|/
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-182-12/+16
|\
| * Typo.R.T. Lechow2011-06-161-2/+2
| |
| * form => form_forVijay Dev2011-06-151-1/+1
| |
| * Clarify importance of *_attributes= writer methods for nested fieldsChristine Yen2011-06-151-8/+12
| |
| * cycle: make an odd number be marked as odd.Robert Massaioli2011-06-131-1/+1
| | | | | | | | Just a minor issue that was annoying me so I thought that I would jump in and fix it.
* | Select tag helpers: remove some code dupsBogdan Gusiev2011-06-171-25/+10
| |
* | Use html_safeSantiago Pastorino2011-06-161-2/+2
| |
* | Remove usage of memoizable from ActionPack.José Valim2011-06-161-7/+6
| |
* | simplify to only one conditionDamien Mathieu2011-06-151-5/+1
| | | | | | | | Signed-off-by: Andrew White <andyw@pixeltrix.co.uk>
* | Make MissingTranslation exception handler respect :rescue_formatAndrew White2011-06-151-1/+5
| |
* | Merge pull request #1552 from bogdan/selectPiotr Sarnacki2011-06-111-6/+37
|\ \ | | | | | | Fixing select[multiple] html specification problem.
| * | Fixing select[multiple] html specification problem.Bogdan Gusiev2011-06-081-6/+37
| | | | | | | | | | | | Generating hidden input with same name before each multiple select
* | | No need for a configuration option here.José Valim2011-06-111-10/+2
| | |
* | | Make utf8_enforcer_tag an overrideable methodDavid Lee2011-06-111-3/+7
| | |
* | | Make utf8 enforcer param customizeableDavid Lee2011-06-112-4/+11
| | |
* | | There are no snowmen hereDavid Lee2011-06-111-3/+3
| |/ |/|
* | Ensure number helpers can handle HTML safe strings - closes #1597.Andrew White2011-06-101-1/+1
| |
* | Make sure `escape_javascript` return `SafeBuffer` if the incoming argument ↵Prem Sichanugrist2011-06-091-1/+2
| | | | | | | | is already html_safe
* | Make escape_javascript happy to handle SafeBuffersPaul Gallagher2011-06-081-1/+1
|/ | | | | * see GH#1553 * allow for the fact that gsub on SafeBuffer does not pass match variables $1, $2 etc to a block
* Merge pull request #1547 from sikachu/safebufferJosé Valim2011-06-072-5/+9
|\ | | | | Fix ActionPack tests on `master`
| * Add proper fix to `mail_to` helper.Prem Sichanugrist2011-06-081-2/+2
| | | | | | | | | | * Fix the problem on manipulating on the `ActiveSupport::SafeBuffer` * Make sure that we run `escape_javascript` on the `String`, to avoid unexpected behavior.
| * Adapt [823aa223efbac6ad4d31ea33402892267bb77cb4] to make sure we perform ↵Prem Sichanugrist2011-06-081-3/+7
| | | | | | | | | | | | cloning before manipulation only on `OutputBuffer`. After the fragment rendering, `Builder` returns the `String` object instead of `ActionView::OutputBuffer`. Somehow the same procedure which was in [823aa223efbac6ad4d31ea33402892267bb77cb4] does not play nice with the String, and result in the fragment got lost.
* | Merge pull request #1542 from cmeiklejohn/dont_gsub_on_safebufferJosé Valim2011-06-071-1/+2
|\ \ | |/ |/| Don't operate on the safebuffer, operate on a string and convert.
| * Don't operate on the safebuffer, operate on a string and convert.Christopher Meiklejohn2011-06-071-1/+2
| |
* | remove warning: assigned but unused variableSantiago Pastorino2011-06-081-1/+1
| |
* | Fragment caching needs to operate on the pure output, not theChristopher Meiklejohn2011-06-071-1/+3
|/ | | | safebuffer.
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-063-4/+4
|\ | | | | | | | | Conflicts: activerecord/RUNNING_UNIT_TESTS
| * Remove trailing white-spacesGuillermo Iguaran2011-06-051-1/+1
| |
| * The example in the API docs incorrectly indicated that form_for would use ↵Mark Roghelia2011-05-311-2/+2
| | | | | | | | the same default route for both new and previously persisted records. The default path given in the example would have generated a routing error for new records. form_for actually uses polymorphic_path to determine which route to use based on the state of the record. The example has been updated to reflect the method's actual behavior.
| * Added missing quote in grouped_options_for_select examplediego2011-05-261-1/+1
| |
* | Merge pull request #1448 from ernie/attr_internal_requireJosé Valim2011-06-011-0/+2
|\ \ | | | | | | Require attr_internal before using in ActionView::Helpers
| * | Require attr_internal before using in ActionView::HelpersErnie Miller2011-06-011-0/+2
| | |
* | | fixed asset_paths when called from sprockets for relative_url_rootjacott2011-05-282-11/+7
| |/ |/|
* | Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-05-258-22/+22
|\ \ | | | | | | | | | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/core_ext/kernel/requires.rb
| * | Remove extra white spaces on ActionPack docs.Sebastian Martinez2011-05-238-22/+22
| | |
* | | Redundant condition in compute_asset_host is redundant.Ernie Miller2011-05-241-3/+3
| |/ |/|
* | Remaining cleanup from changes to AssetTagHelpertomhuda2011-05-242-12/+4
| |
* | Move the last pockets of Sprockets stuff into the Sprockets directorywycats2011-05-231-14/+0
| |
* | Move javascript/css helpers into the Sprockets helper and just have them ↵wycats2011-05-231-43/+0
| | | | | | | | override the old behavior
* | Start moving some logic from being embedded in AV into the Rails Sprockets ↵wycats2011-05-232-62/+6
| | | | | | | | extensions
* | Merge branch '3-1-stable'Joshua Peek2011-05-221-15/+47
|\ \ | | | | | | | | | | | | | | | Conflicts: actionpack/actionpack.gemspec activemodel/activemodel.gemspec
| * | Add debug_asset supportJoshua Peek2011-05-221-15/+47
| | |
* | | Edited actionpack/lib/action_view/helpers/form_helper.rb via GitHubJosé Valim2011-05-221-0/+1
| | |
* | | Merge pull request #1189 from jmbejar/7c562d5e460d97b18e4f3367b3cfb13401732920José Valim2011-05-221-0/+19
|\ \ \ | |/ / |/| | Add method FormHelper#fields_for_with_index