Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix trouble using :subdomain in development environment when using ↵ | Bradford Folkens | 2011-11-08 | 1 | -0/+11 |
| | | | | | | | | | | | | | | | | numeric addresses. See-also pull request #3561 from 3-1-stable Otherwise the following occurs: TypeError: can't convert nil into String /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/http/url.rb:75:in host_or_subdomain_and_domain' /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/http/url.rb:37:in url_for' /Users/bfolkens/dev/bfolkens-rails-core/actionpack/lib/action_dispatch/routing/url_for.rb:147:in test_subdomain_may_be_accepted_with_numeric_host' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/testing/setup_and_teardown.rb:67:in run' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/callbacks.rb:426:in send' /Users/bfolkens/dev/bfolkens-rails-core/activesupport/lib/active_support/callbacks.rb:81:in run' | ||||
* | Ensure that the format isn't applied twice to the cache key, else it becomes ↵ | Christopher Meiklejohn | 2011-10-31 | 1 | -0/+13 |
| | | | | impossible to target with expire_action. | ||||
* | Responders now return 204 No Content for API requests without a response ↵ | José Valim | 2011-10-26 | 1 | -10/+10 |
| | | | | body (as in the new scaffold) | ||||
* | ActionPack test fix for RBX | Arun Agrawal | 2011-10-24 | 1 | -1/+3 |
| | |||||
* | / is allowed in URI fragments | Jeremy Kemper | 2011-10-13 | 1 | -2/+2 |
| | |||||
* | Merge pull request #2219 from kommen/fix_fragment_caching_squashed | Santiago Pastorino | 2011-10-13 | 1 | -0/+49 |
| | | | | Fix fragment caching (squashed commits) | ||||
* | Support symbol and string actions in AC#respond_to options | Alexey Vakhov | 2011-10-11 | 1 | -1/+1 |
| | |||||
* | JSON responder should return errors with :error root | Denis Odorcic | 2011-10-10 | 1 | -0/+26 |
| | |||||
* | accept optional :host parameter to force_ssl | Ben Scheirman | 2011-10-08 | 1 | -1/+21 |
| | |||||
* | :subdomain can now be specified with a value of false in url_for, allowing ↵ | Kamil Sobieraj | 2011-10-04 | 1 | -0/+14 |
| | | | | for subdomain(s) removal from the host during link generation. Closes #2025 | ||||
* | normalize arg for AC::TestCase tests class method | Alexey Vakhov | 2011-10-03 | 1 | -0/+16 |
| | |||||
* | TestCase should respect the view_assigns API instead of pulling variables on ↵ | José Valim | 2011-10-02 | 1 | -0/+20 |
| | | | | its own. | ||||
* | Revert "Make process reuse the env var passed as argument" | Santiago Pastorino | 2011-09-24 | 1 | -6/+5 |
| | | | | This reverts commit 0e4748cd415660eb91e63d50aa15cdd027c612dd. | ||||
* | Deprecate passing the template handler in the template name. | José Valim | 2011-09-22 | 4 | -24/+20 |
| | | | | | | For example, calling hello.erb is now deprecated. Since Rails 3.0 passing the handler had no effect whatsover. This commit simply deprecates such cases so we can clean up the code in later releases. | ||||
* | Get rid of update_details in favor of passing details to find_template. | José Valim | 2011-09-22 | 3 | -9/+11 |
| | |||||
* | all routes can be stored in the Journey Routes object | Aaron Patterson | 2011-09-12 | 1 | -1/+1 |
| | |||||
* | Use ensure instead of rescue | Mike Dillon | 2011-09-10 | 1 | -1/+1 |
| | |||||
* | Add test for warning and CHANGELOG entry | Mike Dillon | 2011-09-10 | 1 | -0/+16 |
| | |||||
* | Pull up a method we only use once. | Aaron Patterson | 2011-09-08 | 1 | -5/+1 |
| | |||||
* | Conditions must never be equal | Aaron Patterson | 2011-09-08 | 1 | -6/+1 |
| | |||||
* | Merge pull request #2499 from akaspick/assert_select_email_fix | Jon Leighton | 2011-09-07 | 1 | -0/+19 |
|\ | | | | | Fix assert_select_email to work on non-multipart emails as well as converting the Mail::Body to a string to prevent errors. | ||||
| * | fix assert_select_email to work on non-multipart emails as well as ↵ | Andrew Kaspick | 2011-08-11 | 1 | -0/+19 |
| | | | | | | | | converting the Mail::Body to a string to prevent errors. | ||||
* | | Eliminate newlines in basic auth. fixes #2882 | Aaron Patterson | 2011-09-06 | 1 | -0/+8 |
| | | |||||
* | | * is not allowed in windows file names. Closes #2574 #2847 | Santiago Pastorino | 2011-09-04 | 1 | -1/+1 |
| | | |||||
* | | Add failing test case for #2654 | Andrew White | 2011-08-23 | 1 | -0/+17 |
| | | |||||
* | | Don't modify params in place - fixes #2624 | Andrew White | 2011-08-23 | 1 | -0/+6 |
| | | |||||
* | | Properly escape glob characters. | Aaron Patterson | 2011-08-16 | 1 | -0/+14 |
| | | |||||
* | | Use lazy load hooks to set parameter wrapping configuration. This means that ↵ | Jon Leighton | 2011-08-16 | 1 | -7/+0 |
| | | | | | | | | it doesn't force Action Controller / Active Record to load, but it doesn't fail if they have already loaded. Thanks @josevalim for the hint. | ||||
* | | Don't refer to ActionController::Base in the wrap_parameters initializer - ↵ | Jon Leighton | 2011-08-16 | 1 | -0/+7 |
| | | | | | | | | use config object instead. Cuts about 15% off the load time. (#734) | ||||
* | | Methods like status and location are interfering with redirect_to [Closes #2511] | Franck Verrot | 2011-08-14 | 1 | -0/+5 |
| | | |||||
* | | moving test_generate to an integration test with one assert per test | Aaron Patterson | 2011-08-12 | 2 | -108/+183 |
|/ | |||||
* | Fix the issue where default_url_options is being cached on test cases. ↵ | thoefer | 2011-08-01 | 1 | -0/+29 |
| | | | | | | Closes #1872. Closes #2031. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Merge pull request #2379 from arunagw/extra_require_removal | Santiago Pastorino | 2011-07-31 | 1 | -1/+0 |
|\ | | | | | Removing extra requires from the test. Already loaded in abstract_unit. | ||||
| * | Removing extra requires from the test. Already loaded in abstract_unit. | Arun Agrawal | 2011-07-31 | 1 | -1/+0 |
| | | |||||
* | | remove extra require for 'active_support/dependencies' as it is required in ↵ | Vishnu Atrai | 2011-07-31 | 1 | -1/+0 |
|/ | | | | abstract_unit.rb | ||||
* | There is no need to be destructive with the passed-in options. | thedarkone | 2011-07-28 | 1 | -0/+9 |
| | | | | This fixes a bug that is caused by Resource/SingletonResource mangling resource options when using inline "multi"-resource declarations. | ||||
* | - added ActionView::PartialRenderer#merge_path_into_partial(path, partial) | Gaston Ramos | 2011-07-24 | 1 | -1/+1 |
| | | | | fix issues/1951 | ||||
* | - added test case for issue: | Gaston Ramos | 2011-07-24 | 1 | -0/+14 |
| | | | | | https://github.com/rails/rails/issues/1951 Namespaced model partial_path is wrong in namespaced controllers | ||||
* | Ensure that status codes are logged properly | Jesse Storimer | 2011-07-18 | 1 | -0/+19 |
| | | | | | | Needed to move AC::Metal::Instrumentation before AM::Metal::Rescue so that status codes rendered from rescue_from blocks are logged properly. | ||||
* | Remove stream at the class level. | José Valim | 2011-07-06 | 1 | -1/+1 |
| | | | | | | This is because only template rendering works with streaming. Setting it at the class level was also changing the behavior of JSON and XML responses, closes #1337. | ||||
* | provide a more explicit message when using url_for with nil | Damien Mathieu | 2011-07-02 | 1 | -0/+7 |
| | | | This fixes the problem of having a non-explicit message when the :location option is not provided in respond_with. | ||||
* | Make sure respond_with with :js tries to render a template in all cases | José Valim | 2011-06-30 | 1 | -6/+15 |
| | |||||
* | Make send_file guess content-type from file extension, if type wasn't ↵ | Esad Hajdarevic | 2011-06-28 | 1 | -0/+19 |
| | | | | supplied (Issue #1847). Update tests & documentation. | ||||
* | Get the fixture_path from self.class instead of ActiveSupport::TestCase. | David Chelimsky | 2011-06-25 | 1 | -0/+7 |
| | | | | | | This allows test classes that are not subclasses of ActiveSupport::TestCase (like those in rspec-rails) to interact with with this variable without having to reference ActiveSupport::TestCase. | ||||
* | Tests only after filters in cache sweepers | Jeroen Jacobs | 2011-06-22 | 1 | -0/+5 |
| | |||||
* | remove meaningless assert true | Neeraj Singh | 2011-06-11 | 1 | -1/+0 |
| | |||||
* | expected message should come first | Neeraj Singh | 2011-06-11 | 1 | -2/+2 |
| | |||||
* | try not to use assert_blank when nil will would | Neeraj Singh | 2011-06-11 | 1 | -2/+2 |
| | | | | | | be considered as failure Test for specific value to the extent possible | ||||
* | add missing ensure | Neeraj Singh | 2011-06-10 | 1 | -0/+1 |
| | |||||
* | replaced and with && as per coding guidelines | Neeraj Singh | 2011-06-10 | 1 | -1/+1 |
| |