Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | ||||
* | Reset ActionView::Base.logger instead of AC::Base.logger | Akira Matsuda | 2013-07-26 | 1 | -1/+1 |
| | | | | see: 9b0ac0bc74569db460f87ea6888b3847be0ff5be | ||||
* | Fix order dependent tests | Akira Matsuda | 2013-07-26 | 1 | -0/+4 |
| | | | | Reset ActionView::Base.logger after tests | ||||
* | Fix order dependent AP test | Akira Matsuda | 2013-07-26 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | | | | | | | any_instance.stubs + unstub with Mocha doesn't restore the original status in the following case, so we need to undef Customer#to_json before every test require 'test/unit' require 'mocha/setup' module M def foo() :foo; end end class C include M undef_method :foo end C.any_instance.stubs(:foo).returns(:mocha) C.any_instance.unstub(:foo) | ||||
* | Typo fix | Ankit Gupta | 2013-07-25 | 1 | -1/+1 |
| | | | | | [skip ci] Fixing the typo which is formed a not required link. Check here http://api.rubyonrails.org/classes/ActionController/Base.html under paramters section keeping it under tt tag | ||||
* | Allow overriding of all headers from passed environment hash | Andrew White | 2013-07-25 | 3 | -5/+42 |
| | | | | | | | Allow REMOTE_ADDR, HTTP_HOST and HTTP_USER_AGENT to be overridden from the environment passed into `ActionDispatch::TestRequest.new`. Fixes #11590 | ||||
* | Removed random dot in code sample [ci skip] | Max Vasiliev | 2013-07-22 | 1 | -1/+1 |
| | |||||
* | Clear named routes when routes.rb is reloaded | Andrew White | 2013-07-21 | 2 | -0/+9 |
| | | | | | | | | Fix an issue where Journey was failing to clear the named routes hash when the routes were reloaded and since it doesn't overwrite existing routes then if a route changed but wasn't renamed it kept the old definition. This was being masked by the optimised url helpers so it only became apparent when passing an options hash to the url helper. |