Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | Make Mime::TEXT default format in AbstractController | Łukasz Strzałkowski | 2013-09-03 | 2 | -4/+1 | |
| | | | | | ||||||
* | | | | | Move skeleton methods from AV to AbsC | Łukasz Strzałkowski | 2013-09-03 | 1 | -7/+19 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The methods: * #render_to_body * #render_to_string * #_normalize_render Haven't had anything specyfic to ActionView. This was common code which should belong to AbstractController | |||||
* | | | | | Return to using protected_instance_variables in AV | Łukasz Strzałkowski | 2013-09-02 | 1 | -5/+6 | |
| | | | | | ||||||
* | | | | | Revert "Port all remaining self.protected_instance_variables to class methods" | Łukasz Strzałkowski | 2013-09-02 | 3 | -16/+16 | |
| | | | | | | | | | | | | | | | | | | | | This reverts commit 7de994fa215e9f4c2856d85034bc4dd7b65d0c01. | |||||
* | | | | | 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. |