aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
* Fix bug #2579.rails-noob2011-09-061-1/+3
| | | | Avoids double slash at start of paths when mounting an engine at the root.
* minor details revised in a gsubXavier Noria2011-08-181-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 Patterson2011-08-163-3/+27
|
* Tags with invalid names should also be stripped in order to preventAaron Patterson2011-08-162-1/+8
| | | | XSS attacks. Thanks Sascha Depold for the report.
* Use lazy load hooks to set parameter wrapping configuration. This means that ↵Jon Leighton2011-08-162-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 railtiesJon Leighton2011-08-161-1/+3
|
* Don't refer to ActionController::Base in the wrap_parameters initializer - ↵Jon Leighton2011-08-162-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 Pastorino2011-08-151-4/+4
| | | | RAILS_ENV was not provided
* Unused variable removed.Arun Agrawal2011-08-151-4/+4
|
* Ensure changing RAILS_GROUPS will load the proper dependencies.José Valim2011-08-141-13/+16
|
* Methods like status and location are interfering with redirect_to [Closes #2511]Franck Verrot2011-08-142-3/+8
|
* Merge branch 'master' of github.com:lifo/docrailsXavier Noria2011-08-133-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 fixOge Nnadi2011-08-131-1/+1
| |
| * update abstract_controller callbacks to document meta-programmed filtersgeemus2011-08-131-18/+102
| |
| * Rubygems => RubyGemsSukeerthi Adiga2011-08-131-1/+1
| |
| * rephrase "like to be"Waynn Lue2011-08-131-1/+1
| |
| * ActionView::Helpers::TextHelper#simple_format should not change the text in ↵Casebook Developer2011-08-132-3/+3
| | | | | | | | place. Now it duplicates it.
| * fix stringify_keys destructive behavior for most FormTagHelper functionsWaynn Lue2011-08-132-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. ↵thoefer2011-08-133-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é Valim2011-08-134-10/+10
| | | | | | | | File#to_path and similar.
| * Removing extra requires from the test. Already loaded in abstract_unit.Arun Agrawal2011-08-131-1/+0
| |
| * remove extra require for 'active_support/dependencies' as it is required in ↵Vishnu Atrai2011-08-131-1/+0
| | | | | | | | abstract_unit.rb
| * remove redundant calls to stringify_keysWaynn Lue2011-08-131-2/+2
| |
| * Instead of removing the instance variable just set it to nil, resolves the ↵Brian Cardarella2011-08-131-2/+2
| | | | | | | | warnings because of a missing instance variable
| * We don't need to require erb here.Franck Verrot2011-08-131-1/+0
| |
| * Make use of the inherited initializer.thedarkone2011-08-131-3/+2
| |
| * There is no need to be destructive with the passed-in options.thedarkone2011-08-132-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é Valim2011-08-131-11/+17
| | | | | | | | as keys.
| * Rename class method to_path to _to_path and make it explicit that it is an ↵José Valim2011-08-131-2/+2
| | | | | | | | internal method.
| * Let ActiveModel instances define partial paths.Grant Hutchins & Peter Jaros2011-08-134-23/+69
| | | | | | | | | | | | Deprecate ActiveModel::Name#partial_path. Now you should call #to_path directly on ActiveModel instances.
| * Make Rails.application.assets available in initializersJoshua Peek2011-08-131-44/+36
| |
| * contrib app minor tweakXavier Noria2011-08-131-2/+2
| |
| * Improve performance and memory usage for options_for_select with Ruby 1.8.7Christos Zisopoulos2011-08-131-2/+2
| |
| * simplify conditionals by assuming hash values will never be `false`Aaron Patterson2011-08-131-10/+2
| |
| * use regular ruby rather than clever rubyAaron Patterson2011-08-131-3/+3
| |
| * fixing wildcard path matching when wildcard is inside parenthesisAaron Patterson2011-08-132-1/+8
| |
| * Remove blank lineSantiago Pastorino2011-08-131-1/+0
| |
| * refactored 'assert_redirected_to': local call to validate_request! will be ↵thoefer2011-08-131-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 #2094Dan Gebhardt2011-08-131-0/+17
| |
| * fixed problem in which options[:html][:remote] would be overridden in ↵Dan Gebhardt2011-08-131-1/+1
| | | | | | | | form_for() - fixes #2094
| * use sprocket's append_path and assert_matchDamien Mathieu2011-08-131-26/+26
| |
| * use_sprockets is not used anymoreSantiago Pastorino2011-08-131-5/+1
| |
| * Bump sprockets upSantiago Pastorino2011-08-131-1/+1
| |
| * Bump rack up. Closes #2107Santiago Pastorino2011-08-131-1/+1
| |
| * Allow a route to have :format => trueErik Michaels-Ober2011-08-132-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 configSantiago Pastorino2011-08-132-8/+1
| |
| * The trailing '/' isn't being picked up by Github anyway, and the link works ↵Rashmi Yadav2011-08-131-1/+1
| | | | | | | | as is.
| * document meta methodVijay Dev2011-07-291-3/+3
| |
* | We need [] method here. previously it's an array.Arun Agrawal2011-08-131-0/+4
| |
* | moving test_generate to an integration test with one assert per testAaron Patterson2011-08-122-108/+183
| |