Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #3906 from kennyj/fix_warnings_for_params_wrapper_test | José Valim | 2011-12-08 | 1 | -1/+1 |
|\ | | | | | Fix warning for params_wrapper_test.rb | ||||
| * | Fix warning for params_wrapper_test. | kennyj | 2011-12-08 | 1 | -1/+1 |
| | | |||||
* | | Just use the proc if there is a chance of layout lookup. | José Valim | 2011-12-08 | 1 | -5/+6 |
| | | |||||
* | | Optimize layout lookup to avoid double calls. | José Valim | 2011-12-08 | 4 | -33/+41 |
| | | |||||
* | | Some small optimizations and improvements to benchmark code. | José Valim | 2011-12-08 | 4 | -17/+15 |
| | | |||||
* | | Add performance scripts from wycats/rails-simple-benches to actionpack. | José Valim | 2011-12-08 | 9 | -0/+216 |
|/ | |||||
* | Merge pull request #3900 from jfturcot/accessible_wrap_params | José Valim | 2011-12-08 | 3 | -2/+41 |
|\ | | | | | ParamsWrapper only wrap the accessible attributes when they were set | ||||
| * | Add information to the changelog about the changes to ↵ | Jean-Francois Turcot | 2011-12-08 | 1 | -0/+5 |
| | | | | | | | | ActionController::ParamsWrapper | ||||
| * | ParamsWrapper only wrap the accessible attributes when they were set | Jean-Francois Turcot | 2011-12-07 | 2 | -2/+36 |
| | | |||||
* | | Remove dead broken code from AD::Request | José Valim | 2011-12-08 | 1 | -8/+0 |
| | | |||||
* | | Merge pull request #3888 from kennyj/should_use_default_charset | José Valim | 2011-12-08 | 1 | -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 ↵ | kennyj | 2011-12-07 | 1 | -0/+1 |
| | | | | | | | | time. | ||||
* | | Remove #:nodoc: from partial renderer. | José Valim | 2011-12-07 | 1 | -1/+1 |
| | | |||||
* | | Use freezed string constant. Some string literals is used repeatedly. | kennyj | 2011-12-08 | 3 | -24/+40 |
| | | |||||
* | | Merge pull request #3884 from sikachu/master-fix_layout | José Valim | 2011-12-07 | 6 | -40/+125 |
|\ \ | | | | | | | Allow layout fallback when using `layout` method | ||||
| * | | Allow layout fallback when using `layout` method | Prem Sichanugrist | 2011-12-06 | 3 | -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` option | Prem Sichanugrist | 2011-12-06 | 3 | -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_charset | José Valim | 2011-12-07 | 2 | -1/+12 |
|\ \ \ | |/ / |/| / | |/ | Use default charset when we read content type without charset. | ||||
| * | Use default charset when we read content type without charset. | kennyj | 2011-12-07 | 2 | -1/+12 |
| | | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-12-06 | 1 | -4/+4 |
|\ \ | |/ |/| | |||||
| * | Use do end for multiline block. | Uģis Ozols | 2011-12-05 | 1 | -2/+2 |
| | | |||||
| * | .rhtml and .rxml was removed in Rails 3. | Uģis Ozols | 2011-12-05 | 1 | -2/+2 |
| | | |||||
* | | Sync AP CHANGELOG with 3-1-stable | Santiago Pastorino | 2011-12-06 | 1 | -0/+8 |
| | | |||||
* | | Merge pull request #3428 from adrianpike/asset_path_conflicts | Santiago Pastorino | 2011-12-06 | 2 | -8/+66 |
|\ \ | | | | | | | Issue #3427 - asset_path_conflicts | ||||
| * | | Let's do the same trick for asset_path that we do for ↵ | Adrian Pike | 2011-12-05 | 2 | -8/+66 |
| | | | | | | | | | | | | | | | | | | [image,javascript,stylesheet]_path to avoid namespace conflicts on named routes. Closes #3427 | ||||
* | | | Allowing string as url argument for expire_action | Thomas von Deyen | 2011-12-06 | 2 | -3/+22 |
|/ / | |||||
* | | Merge pull request #3861 from andyjeffries/master | José Valim | 2011-12-05 | 2 | -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 Jeffries | 2011-12-05 | 2 | -3/+3 |
| | | | | | | | | | | | | goes with the earliest match, named routes use the latest match) | ||||
* | | | use classify in ParamsWrapper to derive model name from controller name | lest | 2011-12-05 | 2 | -1/+36 |
|/ / | |||||
* | | Minor enhancement by not unnecessarely escaping forward slashing within a ↵ | Overbryd | 2011-12-05 | 1 | -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 ↵ | Overbryd | 2011-12-05 | 2 | -2/+12 |
| | | | | | | | | "//asset.host.com/resources/1235" see issue #3856 | ||||
* | | form_for with +:as+ option uses "action_as" as css class and id | Vasiliy Ermolovich | 2011-12-04 | 3 | -13/+25 |
|/ | |||||
* | Fix failing cascade exception. | José Valim | 2011-12-03 | 3 | -2/+12 |
| | |||||
* | Try to play nice with plugins doing monkey patches. | José Valim | 2011-12-02 | 1 | -0/+5 |
| | |||||
* | Merge branch 'exceptions' with the following features: | José Valim | 2011-12-01 | 11 | -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é Valim | 2011-12-01 | 1 | -6/+1 |
| | | |||||
| * | Split and improve show and debug exceptions middlewares. | José Valim | 2011-12-01 | 3 | -87/+141 |
| | | |||||
| * | Split ShowExceptions responsibilities in two middlewares. | José Valim | 2011-12-01 | 7 | -93/+123 |
| | | |||||
| * | Add a deprecation to old show exceptions API (even though it was not public). | José Valim | 2011-12-01 | 1 | -0/+14 |
| | | |||||
| * | Add an ExceptionWrapper that wraps an exception and provide convenience helpers. | José Valim | 2011-12-01 | 5 | -77/+104 |
| | | |||||
| * | Allow rescue responses to be configured through a railtie. | José Valim | 2011-12-01 | 3 | -9/+18 |
| | | |||||
* | | Revert "Added ActiveRecord::Base#last_modified to work with the new ↵ | David Heinemeier Hansson | 2011-12-01 | 2 | -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 Hansson | 2011-12-01 | 2 | -4/+4 |
| | | | | | | | | fresh_when/stale? conditional get methods from Action Pack | ||||
* | | Allow fresh_when/stale? to take a record instead of an options hash [DHH] | David Heinemeier Hansson | 2011-12-01 | 3 | -4/+97 |
|/ | |||||
* | Update CHANGELOG. | José Valim | 2011-12-01 | 1 | -1/+3 |
| | |||||
* | Leave default_asset_host_protocol unset | Jonathan del Strother | 2011-12-01 | 1 | -2/+0 |
| | | | When default_asset_host_protocol is left as nil, it will use absolute protocols when a request is present, and relative protocols otherwise (eg in asset generation) | ||||
* | minor typo fix | Vijay Dev | 2011-12-01 | 1 | -1/+1 |
| | |||||
* | fix exception page when template contains utf-8 and parameters contain utf-8 | lest | 2011-11-30 | 1 | -1/+5 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-11-30 | 2 | -7/+7 |
|\ | |||||
| * | Fixing incorrect documentation | Aviv Ben-Yosef | 2011-11-30 | 1 | -2/+2 |
| | | | | | | `path_names` can only be used for affecting `new` and `edit` |