Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | Move render_test to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -1298/+106 | |
| | | | | | | ||||||
* | | | | | | Fist stab on basic rendering | Łukasz Strzałkowski | 2013-08-25 | 3 | -0/+20 | |
| | | | | | | ||||||
* | | | | | | Do not include action_view/layouts | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | We don't need them. They'll be loaded automaticly if AV will be included | |||||
* | | | | | | Add #rendered_format method to controllers | Łukasz Strzałkowski | 2013-08-25 | 2 | -4/+9 | |
| | | | | | | ||||||
* | | | | | | Do not load AV inside AP | Łukasz Strzałkowski | 2013-08-25 | 2 | -9/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Move that part to AV railtie | |||||
* | | | | | | Remove dependency on AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -3/+3 | |
| | | | | | | ||||||
* | | | | | | Revert "Move setting content_type to AV" | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit f4d602aff6cec80304b131ecfcc2676d0304f0cc. | |||||
* | | | | | | Change documentation of metal anonymous class | Łukasz Strzałkowski | 2013-08-25 | 1 | -5/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make it clearer [ci skip] | |||||
* | | | | | | Move setting content_type to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 | |
| | | | | | | ||||||
* | | | | | | Improve AV::Rendering docs | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+3 | |
| | | | | | | ||||||
* | | | | | | Load AV::Layout to AM::Base in railties | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+3 | |
| | | | | | | ||||||
* | | | | | | Remove RenderERBUtils from AP (only AV use it) | Łukasz Strzałkowski | 2013-08-25 | 1 | -22/+0 | |
| | | | | | | ||||||
* | | | | | | Remove AV::TestCase from AP | Łukasz Strzałkowski | 2013-08-25 | 1 | -8/+0 | |
| | | | | | | ||||||
* | | | | | | Indent protected methods | Łukasz Strzałkowski | 2013-08-25 | 1 | -18/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
* | | | | | | Load AV::Layouts dynamicly via railties | Łukasz Strzałkowski | 2013-08-25 | 3 | -3/+2 | |
| | | | | | | ||||||
* | | | | | | Code formatting & typo fixes | Łukasz Strzałkowski | 2013-08-25 | 2 | -2/+2 | |
| | | | | | | ||||||
* | | | | | | Move anonymous class to the top, add documentation | Łukasz Strzałkowski | 2013-08-25 | 1 | -4/+9 | |
| | | | | | | ||||||
* | | | | | | Remove abstract_controller load hooks | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+0 | |
| | | | | | | ||||||
* | | | | | | Move protected_instance_variables & view_assigns to AbstractController | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+20 | |
| | | | | | | ||||||
* | | | | | | No need for .rb extension | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Create AbstractController::Rendering interface | Łukasz Strzałkowski | 2013-08-25 | 10 | -3/+65 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This interface should be use when implementing renderers. | |||||
* | | | | | | Revert "Rename abstract_controller/rendering. to errors.rb" | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6fe91ec5008838338e54ab8570f7c95ee0cdd8e8. | |||||
* | | | | | | Rename abstract_controller/rendering. to errors.rb | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Since all rendering stuff was extracted to AV, the only thing that left was single class with error so file name wasn't relevant anymore | |||||
* | | | | | | Fix AP test suite after moving stuff to AV | Łukasz Strzałkowski | 2013-08-25 | 7 | -6/+9 | |
| | | | | | | ||||||
* | | | | | | Hook up AV::Rendering on AV intialization | Łukasz Strzałkowski | 2013-08-25 | 2 | -2/+2 | |
| | | | | | | ||||||
* | | | | | | Move layouts to AV | Łukasz Strzałkowski | 2013-08-25 | 2 | -2/+1 | |
| | | | | | | ||||||
* | | | | | | Move rendering from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 5 | -613/+2 | |
| | | | | | | ||||||
* | | | | | | Move view_paths from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 5 | -273/+0 | |
| |_|_|/ / |/| | | | | ||||||
* | | | | | Display exceptions in text format for xhr request | Kir Shatrov | 2013-08-22 | 15 | -13/+129 | |
| | | | | | ||||||
* | | | | | Escape the message of an exception in debug_exceptions to avoid bad rendering | Adrien Siami | 2013-08-21 | 5 | -5/+5 | |
| | | | | | ||||||
* | | | | | Use each_with_object | Santiago Pastorino | 2013-08-19 | 1 | -4/+2 | |
| | | | | | ||||||
* | | | | | Organize normalize_encoding_params' conditionals a bit better and use a case ↵ | Santiago Pastorino | 2013-08-19 | 1 | -15/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | statement for it Refactor of the work done in #11891 | |||||
* | | | | | Merge pull request #11891 from robertomiranda/remove_upload_module | Santiago Pastorino | 2013-08-19 | 3 | -14/+2 | |
|\ \ \ \ \ | | | | | | | | | | | | | Normalize file parameters in same place as other parameters (ActionDispatch::Http::Parameters#normalize_encode_params) | |||||
| * | | | | | Normalize file parameters in same place as other parameters ↵ | robertomiranda | 2013-08-14 | 3 | -14/+2 | |
| |/ / / / | | | | | | | | | | | | | | | | (ActionDispatch::Http::Parameters#normalize_encode_params) | |||||
* | | | | | Merge pull request #11443 from wangjohn/sse_reloader_class | Santiago Pastorino | 2013-08-19 | 2 | -0/+162 | |
|\ \ \ \ \ | | | | | | | | | | | | | SSE class for ActionController::Live | |||||
| * | | | | | Creating an SSE class to be used with ActionController::Live. | wangjohn | 2013-07-30 | 2 | -0/+162 | |
| | | | | | | ||||||
* | | | | | | Fail informatively in #respond_with when no appropriate #api_behavior ↵ | Ben Woosley | 2013-08-17 | 3 | -0/+44 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | renderer is available. Currently if a user calls #respond_with(csvable), but has not csv renderer available, Responder will just run through the default render behavior twice, raising ActionView::MissingTemplate both times. This changes ActionController::Metal::Responder#api_behavior to check in advance whether there is a renderer available, and raise ActionController::MissingRenderer if not. | |||||
* | | | | | | Split the 1200+ line mime_responds_test into 3 more focused and manageable ↵ | Ben Woosley | 2013-08-17 | 3 | -587/+587 | |
| | | | | | | | | | | | | | | | | | | | | | | | | test files. | |||||
* | | | | | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2013-08-17 | 5 | -20/+13 | |
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionview/README.rdoc activerecord/lib/active_record/migration.rb guides/source/development_dependencies_install.md guides/source/getting_started.md | |||||
| * | | | | | Revert "Merge branch 'master' of github.com:rails/docrails" | Vijay Dev | 2013-08-17 | 5 | -20/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 70d6e16fbad75b89dd1798ed697e7732b8606fa3, reversing changes made to ea4db3bc078fb3093ecdddffdf4f2f4ff3e1e8f9. Seems to be a code merge done by mistake. | |||||
* | | | | | | Execute conditional procs on controller filters only for current action. | Nicholas Jakobsen | 2013-08-10 | 3 | -1/+17 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :only and :except options for controller filters are now added before :if and :unless. This prevents running :if and :unless procs when not on the specified. Closes #11786. | |||||
* | | | | | | Added data for permit array into scalar value | Rahul P. Chaudhari | 2013-08-05 | 1 | -0/+1 | |
| | | | | | | ||||||
* | | | | | | Pass assert_dom_equal message arg to underlying assertion | Ryan McGeary | 2013-08-04 | 2 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | #assert_dom_equal and #assert_dom_not_equal both take a "failure" message argument, but this argument was not utilized. | |||||
* | | | | | | Merge pull request #11721 from gaurish/join-timeout | Guillermo Iguaran | 2013-08-02 | 1 | -1/+1 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Add timeout to test_async_stream | |||||
| * | | | | | | Add timeout to test_async_stream | Gaurish Sharma | 2013-08-02 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without timeout: this test executed infinitely on JRuby Passes on MRI With Timeout: this test gracefully fails on JRuby Passes on MRI, tested on v2.0 & v1.9.3 | |||||
* | | | | | | | fix a gramatical mistake | Rajeev N Bharshetty | 2013-08-02 | 1 | -1/+1 | |
| | | | | | | | ||||||
* | | | | | | | Moved README into action_view [ci skip] | Arun Agrawal | 2013-07-31 | 1 | -5/+0 | |
|/ / / / / / | ||||||
* | / / / / | Improve documentation for controller how to get included helpers [ci skip] | Paul Nikitochkin | 2013-07-30 | 1 | -1/+1 | |
| |/ / / / |/| | | | | | | | | | | | | | | Closes #11671 | |||||
* | | | | | grab executable from rubygems | Arun Agrawal | 2013-07-26 | 1 | -2/+1 | |
|/ / / / | | | | | | | | | As done here d7fc97d3f90c0e30865d32ce202658f03248cacc | |||||
* | | | | Refactor handling of action normalization | Max Shytikov | 2013-07-26 | 1 | -11/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Reference: Bloody mess internals http://gusiev.com/slides/rails_contribution/static/#40 |