aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* Make the API for compression consistent between JS and CSS. By default, ↵wycats2011-06-161-2/+6
| | | | | | users just need to say whether they want compression or not, and a default will be chosen by a Railtie. In the case of CSS, this default is already chosen by the sass-rails gem. Users can still explicitly choose a compressor in their application.rb if they have a preference, but will usually want to let plugins choose defaults in their Railties.
* Use html_safeSantiago Pastorino2011-06-161-2/+2
|
* Remove usage of memoizable from ActionPack.José Valim2011-06-162-10/+9
|
* Also change the default route commentWen-Tien Chang2011-06-161-1/+1
|
* 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
|
* Fix appending sprockets pathsJoshua Peek2011-06-141-1/+6
|
* Removed warning while running test with ruby 1.9.2bagwanpankaj2011-06-141-1/+1
|
* all requests are utf-8. Don't use the external encoding.Damien Mathieu2011-06-141-3/+2
|
* encode the uploaded file's name in the default external encoding - Closes #869Damien Mathieu2011-06-141-1/+12
|
* 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
| |
* | Remove utf8_enforcer_param config optionDavid Lee2011-06-111-7/+0
| |
* | Make utf8_enforcer_tag an overrideable methodDavid Lee2011-06-111-3/+7
| |
* | Make utf8 enforcer param customizeableDavid Lee2011-06-113-4/+18
| |
* | There are no snowmen hereDavid Lee2011-06-111-3/+3
| |
* | Revert "Make sure that we don't perform in-place mutation on SafeBuffer string"Santiago Pastorino and José Ignacio Costa2011-06-101-3/+0
| | | | | | | | This reverts commit 115e80dccc65c3ed9a9750649d9ca4ea2a7e64f1.
* | Merge pull request #1642 from sikachu/master-render-inlineAaron Patterson2011-06-101-0/+3
|\ \ | | | | | | Render inline fix for master
| * | Make sure that we don't perform in-place mutation on SafeBuffer stringPrem Sichanugrist2011-06-101-0/+3
| | | | | | | | | | | | | | | | | | This will make sure `render :inline` is working. Closes #1633
* | | Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-111-3/+3
|\ \ \ | |/ / |/| |
| * | shallow_path example updatedganesh2011-06-101-3/+3
| | |
* | | Ensure number helpers can handle HTML safe strings - closes #1597.Andrew White2011-06-101-1/+1
| | |
* | | Merge pull request #1523 from raghunadhd/refactor_regexXavier Noria2011-06-101-4/+7
|\ \ \ | | | | | | | | rafactored the regex related code in the mapper class
| * | | removed the unnecessary back slashes from the regular expressionRaghunadh2011-06-101-2/+2
| | | |
| * | | rafactored the regex related code in the mapper class Raghunadh2011-06-071-4/+7
| | | |
* | | | Make sure `escape_javascript` return `SafeBuffer` if the incoming argument ↵Prem Sichanugrist2011-06-091-1/+2
| | | | | | | | | | | | | | | | is already html_safe
* | | | Indention style + whitespace for readabilityDavid Heinemeier Hansson2011-06-091-33/+38
| | | |
* | | | If you set a js_compressor, it means that you want shit compressed -- we ↵David Heinemeier Hansson2011-06-091-2/+2
| | | | | | | | | | | | | | | | dont need a second option to turn that on
* | | | 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-087-9/+4
| | |
* | | Fragment caching needs to operate on the pure output, not theChristopher Meiklejohn2011-06-071-1/+3
|/ / | | | | | | safebuffer.
* / Update asset helpers to use `config.assets.prefix` Joshua Peek2011-06-071-2/+2
|/ | | Fixes #1489
* Added missing assert to test checking form with namespaced isolated model ↵Piotr Sarnacki2011-06-071-1/+1
| | | | and fixed helpers to work correctly in such case
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-069-14/+13
|\ | | | | | | | | Conflicts: activerecord/RUNNING_UNIT_TESTS
| * Remove trailing white-spacesGuillermo Iguaran2011-06-055-6/+5
| |
| * some grammatical correctionsVijay Dev2011-06-041-3/+3
| |
| * Removing trailing white-spacesGuillermo Iguaran2011-05-311-2/+2
| |
| * 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
| |
* | Add backward compatibility for testing cookiesAndrew White2011-06-053-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit restores the ability to assign cookies for testing via @request.env['HTTP_COOKIE'] and @request.cookies, e.g: @request.env['HTTP_COOKIE'] = 'user_name=david' get :index assert_equal 'david', cookies[:user_name] and @request.cookies[:user_name] = 'david' get :index assert_equal 'david', cookies[:user_name] Assigning via cookies[] is the preferred method and will take precedence over the other two methods. This is so that cookies set in controller actions have precedence and are carried over between calls to get, post, etc.
* | Ensure cookie keys are stringsAndrew White2011-06-051-2/+3
| |
* | Allow multiple sources in Sprockets helpersGuillermo Iguaran2011-06-051-28/+34
| |
* | Add missing require for cookies middlewareAndrew White2011-06-041-0/+1
| |
* | Refactor ActionController::TestCase cookiesAndrew White2011-06-044-39/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Assigning cookies for test cases should now use cookies[], e.g: cookies[:email] = 'user@example.com' get :index assert_equal 'user@example.com', cookies[:email] To clear the cookies, use clear, e.g: cookies.clear get :index assert_nil cookies[:email] We now no longer write out HTTP_COOKIE and the cookie jar is persistent between requests so if you need to manipulate the environment for your test you need to do it before the cookie jar is created.