aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | Simplify path traversal logicJosé Valim2011-12-091-7/+7
| |
* | load the encoding converter to work around [ruby-core:41556] when switching ↵Aaron Patterson2011-12-083-115/+1
| | | | | | | | encodings
* | Fix another regression related to the layout optimization.José Valim2011-12-082-1/+27
| |
* | Move symbolize keys to the inner options as we can assume url_options will ↵José Valim2011-12-082-2/+2
| | | | | | | | be properly symbolized.
* | Fix a regression and also fix broken test.José Valim2011-12-082-9/+14
| |
* | Improve cache on route_key lookup.José Valim2011-12-081-4/+6
| |
* | Revert accidental change from cattr_accessor to class_attribute.José Valim2011-12-081-1/+1
| |
* | Merge pull request #3906 from kennyj/fix_warnings_for_params_wrapper_testJosé Valim2011-12-081-1/+1
|\ \ | | | | | | Fix warning for params_wrapper_test.rb
| * | Fix warning for params_wrapper_test.kennyj2011-12-081-1/+1
| | |
* | | Just use the proc if there is a chance of layout lookup.José Valim2011-12-081-5/+6
| | |
* | | Optimize layout lookup to avoid double calls.José Valim2011-12-084-33/+41
| | |
* | | Some small optimizations and improvements to benchmark code.José Valim2011-12-084-17/+15
| | |
* | | Add performance scripts from wycats/rails-simple-benches to actionpack.José Valim2011-12-089-0/+216
|/ /
* | Merge pull request #3900 from jfturcot/accessible_wrap_paramsJosé Valim2011-12-083-2/+41
|\ \ | | | | | | ParamsWrapper only wrap the accessible attributes when they were set
| * | Add information to the changelog about the changes to ↵Jean-Francois Turcot2011-12-081-0/+5
| | | | | | | | | | | | ActionController::ParamsWrapper
| * | ParamsWrapper only wrap the accessible attributes when they were setJean-Francois Turcot2011-12-072-2/+36
| | |
* | | Remove dead broken code from AD::RequestJosé Valim2011-12-081-8/+0
| | |
* | | Merge pull request #3888 from kennyj/should_use_default_charsetJosé Valim2011-12-081-0/+1
|\ \ \ | |/ / |/| | Assign config.encoding to AD::Response.default_charset at the initialization time.
| * | Assign config.encoding to AD::Response.default_charset at the initialization ↵kennyj2011-12-071-0/+1
| | | | | | | | | | | | time.
* | | Remove #:nodoc: from partial renderer.José Valim2011-12-071-1/+1
| | |
* | | Use freezed string constant. Some string literals is used repeatedly.kennyj2011-12-083-24/+40
| | |
* | | Merge pull request #3884 from sikachu/master-fix_layoutJosé Valim2011-12-076-40/+125
|\ \ \ | | | | | | | | Allow layout fallback when using `layout` method
| * | | Allow layout fallback when using `layout` methodPrem Sichanugrist2011-12-063-26/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails will now use your default layout (such as "layouts/application") when you specify a layout with `:only` and `:except` condition, and those conditions fail. For example, consider this snippet: class CarsController layout 'single_car', :only => :show end Rails will use 'layouts/single_car' when a request comes in `:show` action, and use 'layouts/application' (or 'layouts/cars', if exists) when a request comes in for any other actions.
| * | | Fix bug in assert_template when using only `:layout` optionPrem Sichanugrist2011-12-063-14/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently if you're do this: assert_template :layout => "foo" Regardless of what layout you were using, the test will always pass. This was broken since the introduction of :layout option in [d9375f3f]. We have a lot of test cases in actionpack/test/controller/layout_test.rb that use this feature. This will make sure that those test cases are not true negative.
* | | | Merge pull request #3878 from kennyj/should_use_default_charsetJosé Valim2011-12-072-1/+12
|\ \ \ \ | |/ / / |/| / / | |/ / Use default charset when we read content type without charset.
| * | Use default charset when we read content type without charset.kennyj2011-12-072-1/+12
| | |
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-12-061-4/+4
|\ \ \ | |/ / |/| |
| * | Use do end for multiline block.Uģis Ozols2011-12-051-2/+2
| | |
| * | .rhtml and .rxml was removed in Rails 3.Uģis Ozols2011-12-051-2/+2
| | |
* | | Sync AP CHANGELOG with 3-1-stableSantiago Pastorino2011-12-061-0/+8
| | |
* | | Merge pull request #3428 from adrianpike/asset_path_conflictsSantiago Pastorino2011-12-062-8/+66
|\ \ \ | | | | | | | | Issue #3427 - asset_path_conflicts
| * | | Let's do the same trick for asset_path that we do for ↵Adrian Pike2011-12-052-8/+66
| | | | | | | | | | | | | | | | | | | | | | | | [image,javascript,stylesheet]_path to avoid namespace conflicts on named routes. Closes #3427
* | | | Allowing string as url argument for expire_actionThomas von Deyen2011-12-062-3/+22
|/ / /
* | | Merge pull request #3861 from andyjeffries/masterJosé Valim2011-12-052-3/+3
|\ \ \ | | | | | | | | Named Routes shouldn't override existing ones (currently route recognition goes with the earliest match, named routes use the latest match)
| * | | Named Routes shouldn't override existing ones (currently route recognition ↵Andy Jeffries2011-12-052-3/+3
| | | | | | | | | | | | | | | | goes with the earliest match, named routes use the latest match)
* | | | use classify in ParamsWrapper to derive model name from controller namelest2011-12-052-1/+36
|/ / /
* | | Minor enhancement by not unnecessarely escaping forward slashing within a ↵Overbryd2011-12-051-1/+2
| | | | | | | | | | | | curly regexp and by mentoining the protocol relative scheme in the internal comment
* | | Fix for redirect_to to respect urls with a network path reference like ↵Overbryd2011-12-052-2/+12
| | | | | | | | | | | | "//asset.host.com/resources/1235" see issue #3856
* | | form_for with +:as+ option uses "action_as" as css class and idVasiliy Ermolovich2011-12-043-13/+25
|/ /
* | Fix failing cascade exception.José Valim2011-12-033-2/+12
| |
* | Try to play nice with plugins doing monkey patches.José Valim2011-12-021-0/+5
| |
* | Merge branch 'exceptions' with the following features:José Valim2011-12-0111-230/+359
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | * A Railtie API for registering new exceptions and their respective status code (check Active Record railtie for an example) * Extraction of ShowExceptions middleware logging and debugging features into a middleware called DebugExceptions Conflicts: actionpack/CHANGELOG.md
| * | Remove unnecessary test setup.José Valim2011-12-011-6/+1
| | |
| * | Split and improve show and debug exceptions middlewares.José Valim2011-12-013-87/+141
| | |
| * | Split ShowExceptions responsibilities in two middlewares.José Valim2011-12-017-93/+123
| | |
| * | Add a deprecation to old show exceptions API (even though it was not public).José Valim2011-12-011-0/+14
| | |
| * | Add an ExceptionWrapper that wraps an exception and provide convenience helpers.José Valim2011-12-015-77/+104
| | |
| * | Allow rescue responses to be configured through a railtie.José Valim2011-12-013-9/+18
| | |
* | | Revert "Added ActiveRecord::Base#last_modified to work with the new ↵David Heinemeier Hansson2011-12-012-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | fresh_when/stale? conditional get methods from Action Pack" Needless indirection with no added value. This reverts commit 535853e83b9092078035a5abb2aa242fba815c05.
* | | Added ActiveRecord::Base#last_modified to work with the new ↵David Heinemeier Hansson2011-12-012-4/+4
| | | | | | | | | | | | fresh_when/stale? conditional get methods from Action Pack