Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Removed deprecated APIs in text and number helpers [#5156 state:resolved] | Łukasz Strzałkowski | 2010-07-26 | 2 | -64/+5 |
| | | | | | | They're deprecated since 2008. It's time to get rid of them. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge remote branch 'docrails/master' | Xavier Noria | 2010-07-26 | 1 | -1/+1 |
|\ | |||||
| * | update remote_function docs referencing link_to_remote | Andrew Kaspick | 2010-07-22 | 1 | -1/+1 |
| | | |||||
* | | Change returning with tap | Santiago Pastorino | 2010-07-25 | 1 | -2/+1 |
| | | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | | Fixed output_buffer encoding problem [#5179] | Sebastian A. Espindola | 2010-07-25 | 1 | -1/+1 |
| | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | | options could be of any kind of Hash (Hash, HashWithIndifferentAccess or ↵ | Santiago Pastorino | 2010-07-21 | 1 | -1/+1 |
|/ | | | | OrderedHash) this way we keep the properties of the options passed as an argument | ||||
* | doc: form_for does return output rather than merely evaluate its block | Wincent Colaiuta | 2010-07-22 | 1 | -4/+0 |
| | |||||
* | remote_function patch with more detailed test | Andrew Kaspick | 2010-07-21 | 1 | -1/+1 |
| | | | | Signed-off-by: wycats <wycats@gmail.com> | ||||
* | Use capture instead of yield in link_to_unless. | José Valim | 2010-07-21 | 1 | -1/+1 |
| | | | | [#5162 state:resolved] | ||||
* | No need to delegate. Simply include the whole RecordIdentifier module. | José Valim | 2010-07-21 | 3 | -25/+4 |
| | |||||
* | Moved a few methods from RecordIdentifier to ActiveModel::Naming | Piotr Sarnacki | 2010-07-21 | 1 | -5/+5 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Moved PolymorphicRoutes to ActionDispatch::Routing | Piotr Sarnacki | 2010-07-20 | 2 | -2/+2 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | select :include_blank or :prompt should return escaped string [#5099 ↵ | Ivan Torres | 2010-07-18 | 1 | -3/+3 |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | fixing the ternary operation where the logic is very confusing. | Neeraj Singh | 2010-07-18 | 1 | -1/+2 |
| | | | | | | [#5136 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge remote branch 'docrails/master' | Xavier Noria | 2010-07-13 | 2 | -15/+15 |
|\ | |||||
| * | Fixed many references to the old config/environment.rb and Rails::Initializer | Benjamin Quorning | 2010-07-13 | 1 | -14/+14 |
| | | |||||
| * | Removed deprecated |map| block argument from routing docs since it is no ↵ | Steven Hancock | 2010-07-11 | 1 | -1/+1 |
| | | | | | | | | | | | | longer generated in edge Rails config/routes.rb Didn't touch plugins guide since I'm not too clear on how routes work in plugins. | ||||
* | | remove size attribute from tag generated by file_field helper [#5084 ↵ | Miles Egan | 2010-07-13 | 1 | -1/+1 |
|/ | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge remote branch 'docrails/master' | Xavier Noria | 2010-07-07 | 1 | -1/+11 |
|\ | |||||
| * | Added documentation for usage of associative resources with form_for | Rizwan Reza | 2010-07-01 | 1 | -1/+11 |
| | | |||||
* | | Unify routes naming by renaming router to routes | Piotr Sarnacki | 2010-07-02 | 3 | -7/+7 |
|/ | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | s/escape_once/html_escape/, since html safety is the contract that now says ↵ | Xavier Noria | 2010-06-30 | 3 | -16/+19 |
| | | | | whether something has to be escaped | ||||
* | url_for no longer escapes HTML, the :escape option is also gone | Xavier Noria | 2010-06-30 | 3 | -17/+8 |
| | | | | Rationale: url_for is just a path/URL generator, it is the responsability of the caller to escape conveniently HTML needs it, JavaScript needs different escaping, a text mail needs no escaping at all, etc. | ||||
* | Revert "Revert "Name compiled render methods". This caused several failures ↵ | Jeremy Kemper | 2010-06-29 | 1 | -8/+11 |
| | | | | | | | | on AP test suite for 1.9.2." This reverts commit 9013227e00895ac95f29077229ec2fb156f450b0. minitest was 'running' compiled templates beginning with test! | ||||
* | Rename _snowman_ to _snowman to be in sync with _method and _csrf_token. | José Valim | 2010-06-29 | 1 | -1/+1 |
| | |||||
* | Revert "Name compiled render methods". This caused several failures on AP ↵ | José Valim | 2010-06-29 | 1 | -11/+8 |
| | | | | | | test suite for 1.9.2. This reverts commit 2c4f8aae5ca8489bb3e47712f50b3f57015811d9. | ||||
* | Slice new buffer from beginning of old buffer to skip encoding special case | Jeremy Kemper | 2010-06-28 | 1 | -3/+1 |
| | |||||
* | Name compiled render methods | Jeremy Kemper | 2010-06-28 | 1 | -8/+11 |
| | |||||
* | returning -> tap | Jeremy Kemper | 2010-06-28 | 1 | -5/+4 |
| | |||||
* | Restores the escaping of urls generated from hashes. [#4765 state:resolved] | Andrew White | 2010-06-28 | 1 | -1/+1 |
| | | | | | | | | | | | | | | HTML specifications recommend the escaping of urls in web pages, which url_for does by default for string urls and consquently urls generated by path helpers as these return strings. Hashes passed to url_for are not escaped by default and this commit reverses this default so that they are escaped. Undoes the changes of this commit: http://github.com/rails/rails/commit/1b3195b63ca44f0a70b61b75fcf4991cb2fbb944 Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Convert instance_variables to symbols before excluding internal vars [#4965 ↵ | rohit | 2010-06-28 | 1 | -1/+1 |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Small typo | wycats | 2010-06-27 | 1 | -1/+1 |
| | |||||
* | Fix several known web encoding issues: | wycats | 2010-06-27 | 1 | -3/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | * Specify accept-charset on all forms. All recent browsers, as well as IE5+, will use the encoding specified for form parameters * Unfortunately, IE5+ will not look at accept-charset unless at least one character in the form's values is not in the page's charset. Since the user can override the default charset (which Rails sets to UTF-8), we provide a hidden input containing a unicode character, forcing IE to look at the accept-charset. * Now that the vast majority of web input is UTF-8, we set the inbound parameters to UTF-8. This will eliminate many cases of incompatible encodings between ASCII-8BIT and UTF-8. * You can safely ignore params[:_snowman_] TODO: * Validate inbound text to confirm it is UTF-8 * Combine the whole_form implementations in form_helper_test and form_tag_helper_test | ||||
* | AV::logger returns AC::logger if it's defined, workaround meanwhile AV ↵ | Santiago Pastorino | 2010-06-28 | 1 | -2/+3 |
| | | | | | | doesn't have it's own logger Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Do not wrap hidden fields with error proc [#4962 state:resolved] | Carlos Antonio da Silva | 2010-06-25 | 2 | -3/+16 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Move Rails::LogSubscriber to ActiveSupport::LogSubscriber, allowing ↵ | José Valim | 2010-06-24 | 4 | -30/+28 |
| | | | | frameworks like ActiveRecord and ActiveResource to log outsude Rails::Application [#4816 state:resolved] | ||||
* | Avoid using Pathname on Resolver and AS::Dependencies. | José Valim | 2010-06-24 | 1 | -1/+1 |
| | |||||
* | Expose view via the view() method in AV::TestCase::Behavior | David Chelimsky | 2010-06-23 | 1 | -5/+8 |
| | | | | | | | | | | - was exposed as _view, which suggested it was private - left _view as an alias of view as not to break any extensions that are relying on _view [#4932 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | In ActionView::TestCase::Behavior, assign variables right before | David Chelimsky | 2010-06-23 | 2 | -2/+7 |
| | | | | | | | | | | | rendering the view. - Previously, _assigns were locked down the first time _view was referenced. [#4931 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | make text_field and hidden_field omit the value attribute if the developer ↵ | Jeff Dean | 2010-06-23 | 1 | -4/+4 |
| | | | | | | explicitly passes in :value => nil [#4839 state:resolved] Signed-off-by: Michael Koziarski <michael@koziarski.com> | ||||
* | Tidy up asset tag a bit and make railties tests green again. | José Valim | 2010-06-23 | 2 | -27/+22 |
| | |||||
* | refactored the javascript asset tag helpers and moved the default scripts ↵ | Josh Kalderimis | 2010-06-23 | 2 | -14/+14 |
| | | | | | | setup within the railtie Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | update the list of instance variables that are excluded from assigns in view ↵ | David Chelimsky | 2010-06-22 | 1 | -9/+10 |
| | | | | | | | | tests [#4930 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | adds minus to value part of id [#4862 state:resolved] | RainerBlessing | 2010-06-22 | 1 | -1/+1 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Add support for specifying locals in view tests with assert template [#4927 ↵ | David Chelimsky | 2010-06-22 | 1 | -0/+15 |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | removed 'unless const_defined?' code smell | Josh Kalderimis | 2010-06-21 | 5 | -24/+20 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge remote branch 'rails/master' | Xavier Noria | 2010-06-20 | 6 | -19/+26 |
|\ | | | | | | | | | Conflicts: actionpack/lib/abstract_controller/base.rb | ||||
| * | Fixed date_select date_separator when discard_month and/or discard_day are ↵ | Thibaud Guillaume-Gentil | 2010-06-20 | 1 | -2/+4 |
| | | | | | | | | | | | | true [#4856 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Make sure a namespaced <%= render form %> still renders the _form partial ↵ | Jan De Poorter | 2010-06-20 | 1 | -1/+1 |
| | | | | | | | | | | | | [#4784 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Support render_template in view tests. Useful for specifying which | David Chelimsky | 2010-06-20 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | partials are rendered under different conditions. [#4903 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> |