aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* | Load AV::Layouts dynamicly via railtiesŁukasz Strzałkowski2013-08-253-3/+2
| |
* | Code formatting & typo fixesŁukasz Strzałkowski2013-08-252-2/+2
| |
* | Move anonymous class to the top, add documentationŁukasz Strzałkowski2013-08-251-4/+9
| |
* | Remove abstract_controller load hooksŁukasz Strzałkowski2013-08-251-2/+0
| |
* | Move protected_instance_variables & view_assigns to AbstractControllerŁukasz Strzałkowski2013-08-251-1/+20
| |
* | No need for .rb extensionŁukasz Strzałkowski2013-08-251-1/+1
| |
* | Create AbstractController::Rendering interfaceŁukasz Strzałkowski2013-08-2510-3/+65
| | | | | | | | This interface should be use when implementing renderers.
* | Revert "Rename abstract_controller/rendering. to errors.rb"Łukasz Strzałkowski2013-08-252-1/+1
| | | | | | | | This reverts commit 6fe91ec5008838338e54ab8570f7c95ee0cdd8e8.
* | Rename abstract_controller/rendering. to errors.rbŁukasz Strzałkowski2013-08-252-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łkowski2013-08-257-6/+9
| |
* | Hook up AV::Rendering on AV intializationŁukasz Strzałkowski2013-08-252-2/+2
| |
* | Move layouts to AVŁukasz Strzałkowski2013-08-252-2/+1
| |
* | Move rendering from AP to AVŁukasz Strzałkowski2013-08-255-613/+2
| |
* | Move view_paths from AP to AVŁukasz Strzałkowski2013-08-255-273/+0
| |
* | Display exceptions in text format for xhr requestKir Shatrov2013-08-2215-13/+129
| |
* | Escape the message of an exception in debug_exceptions to avoid bad renderingAdrien Siami2013-08-215-5/+5
| |
* | Use each_with_objectSantiago Pastorino2013-08-191-4/+2
| |
* | Organize normalize_encoding_params' conditionals a bit better and use a case ↵Santiago Pastorino2013-08-191-15/+18
| | | | | | | | | | | | statement for it Refactor of the work done in #11891
* | Merge pull request #11891 from robertomiranda/remove_upload_moduleSantiago Pastorino2013-08-193-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 ↵robertomiranda2013-08-143-14/+2
| |/ | | | | | | (ActionDispatch::Http::Parameters#normalize_encode_params)
* | Merge pull request #11443 from wangjohn/sse_reloader_classSantiago Pastorino2013-08-192-0/+162
|\ \ | | | | | | SSE class for ActionController::Live
| * | Creating an SSE class to be used with ActionController::Live.wangjohn2013-07-302-0/+162
| | |
* | | Fail informatively in #respond_with when no appropriate #api_behavior ↵Ben Woosley2013-08-173-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 Woosley2013-08-173-587/+587
| | | | | | | | | | | | test files.
* | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2013-08-175-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 Dev2013-08-175-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 Jakobsen2013-08-103-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 valueRahul P. Chaudhari2013-08-051-0/+1
| | |
* | | Pass assert_dom_equal message arg to underlying assertionRyan McGeary2013-08-042-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-timeoutGuillermo Iguaran2013-08-021-1/+1
|\ \ \ | | | | | | | | Add timeout to test_async_stream
| * | | Add timeout to test_async_streamGaurish Sharma2013-08-021-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 mistakeRajeev N Bharshetty2013-08-021-1/+1
| | | |
* | | | Moved README into action_view [ci skip]Arun Agrawal2013-07-311-5/+0
|/ / /
* | / Improve documentation for controller how to get included helpers [ci skip]Paul Nikitochkin2013-07-301-1/+1
| |/ |/| | | | | Closes #11671
* | grab executable from rubygemsArun Agrawal2013-07-261-2/+1
|/ | | As done here d7fc97d3f90c0e30865d32ce202658f03248cacc
* Refactor handling of action normalizationMax Shytikov2013-07-261-11/+13
| | | | | | Reference: Bloody mess internals http://gusiev.com/slides/rails_contribution/static/#40
* Reset ActionView::Base.logger instead of AC::Base.loggerAkira Matsuda2013-07-261-1/+1
| | | | see: 9b0ac0bc74569db460f87ea6888b3847be0ff5be
* Fix order dependent testsAkira Matsuda2013-07-261-0/+4
| | | | Reset ActionView::Base.logger after tests
* Fix order dependent AP testAkira Matsuda2013-07-261-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 fixAnkit Gupta2013-07-251-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 hashAndrew White2013-07-253-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 Vasiliev2013-07-221-1/+1
|
* Clear named routes when routes.rb is reloadedAndrew White2013-07-212-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.
* Refactor to reduce number of loopsAndrew White2013-07-171-15/+25
| | | | | | Only build the missing_keys array once we have detected that there actually are missing keys by moving the check to be part of the block that performs the path substitution.
* Fix failing test missed for the past year :(Andrew White2013-07-172-3/+18
| | | | | | | | | When optimized path helpers were re-introduced in d7014bc the test added in a328f2f broke but no-one noticed because it wasn't being run by the test suite. Fix the test by checking for nil values or empty strings after the args have been parameterized.
* Added routing in test for actionpackArun Agrawal2013-07-171-2/+4
|
* Skip Rack applications and redirects when generating urlsAndrew White2013-07-166-13/+93
| | | | | | | | | | When generating an unnamed url (i.e. using `url_for` with an options hash) we should skip anything other than standard Rails routes otherwise it will match the first mounted application or redirect and generate a url with query parameters rather than raising an error if the options hash doesn't match any defined routes. Fixes #8018
* Removed unused file! Moved to actionviewArun Agrawal2013-07-131-91/+0
| | | | Removed unused file as this has been moved to actionview
* add test-case to verify `error.path` when a helper was not found.Yves Senn2013-07-121-0/+5
| | | | | This verifies the code added with: https://github.com/rails/rails/commit/99c9d18601539c7e7e87f26bb047add1f93072af
* move `MissingHelperError` out of the `ClassMethods` module.Yves Senn2013-07-123-16/+23
|