Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix bug #2579. | rails-noob | 2011-09-06 | 1 | -1/+3 |
| | | | | Avoids double slash at start of paths when mounting an engine at the root. | ||||
* | minor details revised in a gsub | Xavier Noria | 2011-08-18 | 1 | -1/+1 |
| | | | | | | | | | Regexps have a construct to express alternation of characters, which is character classes. In addition to being the most specific idiom to write this pattern, it reads better without the backslashes. Also, it is better not to use a capture if none is needed. As a side-effect of these changes, the gsub is marginally faster, but speed is not the point of this commit. | ||||
* | Properly escape glob characters. | Aaron Patterson | 2011-08-16 | 3 | -3/+27 |
| | |||||
* | Tags with invalid names should also be stripped in order to prevent | Aaron Patterson | 2011-08-16 | 2 | -1/+8 |
| | | | | XSS attacks. Thanks Sascha Depold for the report. | ||||
* | Use lazy load hooks to set parameter wrapping configuration. This means that ↵ | Jon Leighton | 2011-08-16 | 2 | -9/+0 |
| | | | | it doesn't force Action Controller / Active Record to load, but it doesn't fail if they have already loaded. Thanks @josevalim for the hint. | ||||
* | Fix assets tests in railties | Jon Leighton | 2011-08-16 | 1 | -1/+3 |
| | |||||
* | Don't refer to ActionController::Base in the wrap_parameters initializer - ↵ | Jon Leighton | 2011-08-16 | 2 | -0/+9 |
| | | | | use config object instead. Cuts about 15% off the load time. (#734) | ||||
* | rake assets:precompile executes in production environment as default if ↵ | Santiago Pastorino | 2011-08-15 | 1 | -4/+4 |
| | | | | RAILS_ENV was not provided | ||||
* | Unused variable removed. | Arun Agrawal | 2011-08-15 | 1 | -4/+4 |
| | |||||
* | Ensure changing RAILS_GROUPS will load the proper dependencies. | José Valim | 2011-08-14 | 1 | -13/+16 |
| | |||||
* | Methods like status and location are interfering with redirect_to [Closes #2511] | Franck Verrot | 2011-08-14 | 2 | -3/+8 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Xavier Noria | 2011-08-13 | 3 | -21/+105 |
|\ | | | | | | | | | | | | | | | | | | | | | Conflicts: RELEASING_RAILS.rdoc actionpack/lib/sprockets/railtie.rb actionpack/test/template/sprockets_helper_test.rb activerecord/test/cases/calculations_test.rb railties/guides/source/3_1_release_notes.textile railties/guides/source/active_resource_basics.textile railties/guides/source/command_line.textile | ||||
| * | Typo fix | Oge Nnadi | 2011-08-13 | 1 | -1/+1 |
| | | |||||
| * | update abstract_controller callbacks to document meta-programmed filters | geemus | 2011-08-13 | 1 | -18/+102 |
| | | |||||
| * | Rubygems => RubyGems | Sukeerthi Adiga | 2011-08-13 | 1 | -1/+1 |
| | | |||||
| * | rephrase "like to be" | Waynn Lue | 2011-08-13 | 1 | -1/+1 |
| | | |||||
| * | ActionView::Helpers::TextHelper#simple_format should not change the text in ↵ | Casebook Developer | 2011-08-13 | 2 | -3/+3 |
| | | | | | | | | place. Now it duplicates it. | ||||
| * | fix stringify_keys destructive behavior for most FormTagHelper functions | Waynn Lue | 2011-08-13 | 2 | -4/+28 |
| | | | | | | | | add four new tests to verify that the other three methods that called stringify_keys! are fixed. verified that the tests break in master without the code patch. Closes #2355 | ||||
| * | Fix the issue where default_url_options is being cached on test cases. ↵ | thoefer | 2011-08-13 | 3 | -0/+35 |
| | | | | | | | | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Rename new method to_path to to_partial_path to avoid conflicts with ↵ | José Valim | 2011-08-13 | 4 | -10/+10 |
| | | | | | | | | File#to_path and similar. | ||||
| * | Removing extra requires from the test. Already loaded in abstract_unit. | Arun Agrawal | 2011-08-13 | 1 | -1/+0 |
| | | |||||
| * | remove extra require for 'active_support/dependencies' as it is required in ↵ | Vishnu Atrai | 2011-08-13 | 1 | -1/+0 |
| | | | | | | | | abstract_unit.rb | ||||
| * | remove redundant calls to stringify_keys | Waynn Lue | 2011-08-13 | 1 | -2/+2 |
| | | |||||
| * | Instead of removing the instance variable just set it to nil, resolves the ↵ | Brian Cardarella | 2011-08-13 | 1 | -2/+2 |
| | | | | | | | | warnings because of a missing instance variable | ||||
| * | We don't need to require erb here. | Franck Verrot | 2011-08-13 | 1 | -1/+0 |
| | | |||||
| * | Make use of the inherited initializer. | thedarkone | 2011-08-13 | 1 | -3/+2 |
| | | |||||
| * | There is no need to be destructive with the passed-in options. | thedarkone | 2011-08-13 | 2 | -6/+15 |
| | | | | | | | | This fixes a bug that is caused by Resource/SingletonResource mangling resource options when using inline "multi"-resource declarations. | ||||
| * | Move the cache to a nested hash which performs better than a hash with array ↵ | José Valim | 2011-08-13 | 1 | -11/+17 |
| | | | | | | | | as keys. | ||||
| * | Rename class method to_path to _to_path and make it explicit that it is an ↵ | José Valim | 2011-08-13 | 1 | -2/+2 |
| | | | | | | | | internal method. | ||||
| * | Let ActiveModel instances define partial paths. | Grant Hutchins & Peter Jaros | 2011-08-13 | 4 | -23/+69 |
| | | | | | | | | | | | | Deprecate ActiveModel::Name#partial_path. Now you should call #to_path directly on ActiveModel instances. | ||||
| * | Make Rails.application.assets available in initializers | Joshua Peek | 2011-08-13 | 1 | -44/+36 |
| | | |||||
| * | contrib app minor tweak | Xavier Noria | 2011-08-13 | 1 | -2/+2 |
| | | |||||
| * | Improve performance and memory usage for options_for_select with Ruby 1.8.7 | Christos Zisopoulos | 2011-08-13 | 1 | -2/+2 |
| | | |||||
| * | simplify conditionals by assuming hash values will never be `false` | Aaron Patterson | 2011-08-13 | 1 | -10/+2 |
| | | |||||
| * | use regular ruby rather than clever ruby | Aaron Patterson | 2011-08-13 | 1 | -3/+3 |
| | | |||||
| * | fixing wildcard path matching when wildcard is inside parenthesis | Aaron Patterson | 2011-08-13 | 2 | -1/+8 |
| | | |||||
| * | Remove blank line | Santiago Pastorino | 2011-08-13 | 1 | -1/+0 |
| | | |||||
| * | refactored 'assert_redirected_to': local call to validate_request! will be ↵ | thoefer | 2011-08-13 | 1 | -5/+4 |
| | | | | | | | | called in assert_response already. changed names of local variables in order to recognize the semantics a bit easier. | ||||
| * | added test case for fix to issue #2094 | Dan Gebhardt | 2011-08-13 | 1 | -0/+17 |
| | | |||||
| * | fixed problem in which options[:html][:remote] would be overridden in ↵ | Dan Gebhardt | 2011-08-13 | 1 | -1/+1 |
| | | | | | | | | form_for() - fixes #2094 | ||||
| * | use sprocket's append_path and assert_match | Damien Mathieu | 2011-08-13 | 1 | -26/+26 |
| | | |||||
| * | use_sprockets is not used anymore | Santiago Pastorino | 2011-08-13 | 1 | -5/+1 |
| | | |||||
| * | Bump sprockets up | Santiago Pastorino | 2011-08-13 | 1 | -1/+1 |
| | | |||||
| * | Bump rack up. Closes #2107 | Santiago Pastorino | 2011-08-13 | 1 | -1/+1 |
| | | |||||
| * | Allow a route to have :format => true | Erik Michaels-Ober | 2011-08-13 | 2 | -0/+9 |
| | | | | | | | | | | | | When format is true, it is mandatory (as opposed to :format => false). This is currently not possible with resource routes, which automatically make format optional by default. | ||||
| * | Remove unused use_sprockets config | Santiago Pastorino | 2011-08-13 | 2 | -8/+1 |
| | | |||||
| * | The trailing '/' isn't being picked up by Github anyway, and the link works ↵ | Rashmi Yadav | 2011-08-13 | 1 | -1/+1 |
| | | | | | | | | as is. | ||||
| * | document meta method | Vijay Dev | 2011-07-29 | 1 | -3/+3 |
| | | |||||
* | | We need [] method here. previously it's an array. | Arun Agrawal | 2011-08-13 | 1 | -0/+4 |
| | | |||||
* | | moving test_generate to an integration test with one assert per test | Aaron Patterson | 2011-08-12 | 2 | -108/+183 |
| | |