Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Reuse variable to avoid symbol usage | Carlos Antonio da Silva | 2013-08-31 | 1 | -2/+2 |
| | |||||
* | Merge pull request #11912 from jonkessler/respond-to-signature-fixes | Rafael Mendonça França | 2013-08-31 | 1 | -1/+1 |
|\ | | | | | Update Rails::Railtie::Configuration and ActionDispatch::Response#respond_to? to accept include_private argument | ||||
| * | update Rails::Railtie::Configuration and ↵ | Jon Kessler | 2013-08-16 | 1 | -1/+1 |
| | | | | | | | | ActionDispatch::Response#respond_to? to accept include_private argument | ||||
* | | Stop messing up with instance variables, use protected as it was meant for | Carlos Antonio da Silva | 2013-08-30 | 1 | -2/+7 |
| | | |||||
* | | Port all remaining self.protected_instance_variables to class methods | Łukasz Strzałkowski | 2013-08-29 | 3 | -16/+16 |
| | | |||||
* | | Fix formatting of error message | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+1 |
| | | | | | | | | Make it oneliner | ||||
* | | Change description of BasicRendering#render | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 |
| | | | | | | | | | | | | We're not dealing with templates here [ci skip] | ||||
* | | Simplify if statements | Łukasz Strzałkowski | 2013-08-25 | 1 | -6/+4 |
| | | |||||
* | | Describe BasicRendering shortly | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+4 |
| | | | | | | | | [ci skip] | ||||
* | | Extend basic rendering, test it in railties | Łukasz Strzałkowski | 2013-08-25 | 1 | -3/+16 |
| | | |||||
* | | Load HTML in ActionView not ActionPack | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 |
| | | | | | | | | HTML Scanner is part of ActionView and it should be loaded along with it | ||||
* | | 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 | ||||
* | | 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 |
| | | |||||
* | | 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 | 2 | -3/+0 |
| | | |||||
* | | 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 | 3 | -1/+56 |
| | | | | | | | | 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 | 2 | -0/+2 |
| | | |||||
* | | 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 | 2 | -97/+0 |
| | | |||||
* | | Display exceptions in text format for xhr request | Kir Shatrov | 2013-08-22 | 13 | -13/+84 |
| | | |||||
* | | 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 | 1 | -0/+74 |
|\ \ | | | | | | | SSE class for ActionController::Live | ||||
| * | | Creating an SSE class to be used with ActionController::Live. | wangjohn | 2013-07-30 | 1 | -0/+74 |
| | | | |||||
* | | | Fail informatively in #respond_with when no appropriate #api_behavior ↵ | Ben Woosley | 2013-08-17 | 2 | -0/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2013-08-17 | 2 | -14/+12 |
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | 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 | 2 | -14/+12 |
| |/ | | | | | | | | | | | | | 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 | 1 | -1/+1 |
| | | | | | | | | | | | | :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 | 1 | -4/+4 |
| | | | | | | | | | | #assert_dom_equal and #assert_dom_not_equal both take a "failure" message argument, but this argument was not utilized. | ||||
* | | fix a gramatical mistake | Rajeev N Bharshetty | 2013-08-02 | 1 | -1/+1 |
| | | |||||
* | | Improve documentation for controller how to get included helpers [ci skip] | Paul Nikitochkin | 2013-07-30 | 1 | -1/+1 |
|/ | | | | Closes #11671 |