Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Sync with rails/rails/master, merge two metaprogramming annotation efforts | Michael S. Klishin | 2008-12-28 | 8 | -54/+59 |
|\ | |||||
| * | Inline code comments for class_eval/module_eval [#1657 state:resolved] | Xavier Noria | 2008-12-28 | 5 | -46/+55 |
| | | | | | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
| * | Request#env['SERVER_NAME'] does not contain port number | Pratik Naik | 2008-12-28 | 3 | -8/+4 |
| | | |||||
| * | Fix ActionPack build on Windows: we really should not test anything ↵ | Yaroslav Markin | 2008-12-27 | 1 | -10/+12 |
| | | | | | | | | | | | | regarding symlinks on Windows. Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
| * | More optimizations on respond_to after a profile and benching: | Yehuda Katz | 2008-12-27 | 1 | -6/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | App with simple respond_to: def index respond_to do |format| format.html format.xml format.json end end On JRuby (after complete hotspot warmup) -- 8% improvement: 550 requests per second after this commit 510 requests per second with old method_missing technique On MRI (8% improvement): 430 requests per second after this commit 400 requests per second with old method_missing technique | ||||
* | | Annotated metaprogramming code across ActiveSupport | Michael S. Klishin | 2008-12-28 | 2 | -16/+14 |
| | | |||||
* | | Merge commit 'rails/master' | Yehuda Katz | 2008-12-26 | 4 | -9/+36 |
|\| | |||||
| * | Merge commit 'fred/pullable' | Pratik Naik | 2008-12-26 | 3 | -5/+13 |
| |\ | |||||
| | * | Fix assert_select_rjs not checking id for inserts [#540 state:resolved] | Frederick Cheung | 2008-12-26 | 2 | -0/+9 |
| | | | |||||
| | * | Fix randomly failing cookie store tests | Frederick Cheung | 2008-12-26 | 1 | -5/+4 |
| | | | | | | | | | | | | | | | Marshal.dump(Marshal.load(marshaled_hash)) is not guarenteed to be equal to marshaled_hash because of the lack of ordering of hash | ||||
| * | | Remove method missing use in respond_to | Yehuda Katz | 2008-12-26 | 1 | -4/+23 |
| | | | |||||
* | | | Merge commit 'rails/master' | Yehuda Katz | 2008-12-26 | 3 | -10/+68 |
|\| | | |||||
| * | | Don't recurse when ActionController#render is called without any arguments | Pratik Naik | 2008-12-26 | 1 | -1/+1 |
| | | | |||||
| * | | Make ActionController#render(symbol) behave same as ↵ | Pratik Naik | 2008-12-26 | 3 | -6/+16 |
| | | | | | | | | | | | | ActionController#render(string) [#1435] | ||||
| * | | Make ActionController#render(string) work as a shortcut for render :action ↵ | Pratik Naik | 2008-12-25 | 3 | -4/+28 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | => string. [#1435] Examples: # Instead of render(:action => 'other_action') render('other_action') Note : Argument must not have any '/' | ||||
| * | | Make ActionController#render(string) work as a shortcut for render :template ↵ | Pratik Naik | 2008-12-25 | 3 | -4/+28 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | => string. [#1435] Examples: # Instead of render(:template => 'controller/action') render('controller/action') Note : Argument must not begin with a '/', but have at least one '/' | ||||
* | | | Inline the only call site of Mime::Type#browser_generated? | Michael S. Klishin | 2008-12-26 | 1 | -5/+1 |
| | | | | | | | | | | | | Signed-off-by: Michael S. Klishin <michael@novemberain.com> | ||||
* | | | Initial work to merge several places with similar logic | Yehuda Katz | 2008-12-25 | 4 | -13/+23 |
|/ / | |||||
* | | Make ActionController#render(string) work as a shortcut for render :file => ↵ | Pratik Naik | 2008-12-25 | 3 | -9/+36 |
| | | | | | | | | | | | | | | | | | | | | string. [#1435] Examples: # Instead of render(:file => '/Users/lifo/home.html.erb') render('/Users/lifo/home.html.erb') Note : Filename must begin with a forward slash ('/') | ||||
* | | Move ActionController::Base#render arguments validation to a separate method | Pratik Naik | 2008-12-25 | 1 | -8/+14 |
| | | |||||
* | | Make render_test.rb run in isolation | Pratik Naik | 2008-12-25 | 1 | -0/+2 |
| | | |||||
* | | Undry ActionController::TestCase#<HTTP_METHODS> for better documentation | Pratik Naik | 2008-12-25 | 2 | -14/+26 |
|/ | |||||
* | Move request parsing related code to ActionController::RequestParser | Pratik Naik | 2008-12-25 | 7 | -460/+512 |
| | |||||
* | ActionController::VerbPiggybacking middleware | Joshua Peek | 2008-12-23 | 7 | -38/+49 |
| | |||||
* | Remove duplicate attr_reader :env | Pratik Naik | 2008-12-23 | 1 | -4/+2 |
| | |||||
* | Unify ActionController::AbstractRequest and ActionController::Request | Pratik Naik | 2008-12-23 | 2 | -110/+81 |
| | |||||
* | Use Rack::MockRequest for TestRequest | Pratik Naik | 2008-12-23 | 3 | -20/+9 |
| | |||||
* | Rename RackRequest to Request | Pratik Naik | 2008-12-23 | 6 | -11/+11 |
| | |||||
* | Remove rack_process.rb | Pratik Naik | 2008-12-23 | 3 | -74/+73 |
| | |||||
* | Remove deprecated relative_url_root | Pratik Naik | 2008-12-23 | 1 | -7/+0 |
| | |||||
* | Defining a new method is atomic, no mutex needed. | Joshua Peek | 2008-12-22 | 1 | -8/+2 |
| | |||||
* | Move default middleware stack to middlewares.rb | Joshua Peek | 2008-12-22 | 2 | -17/+21 |
| | |||||
* | ActiveRecord::QueryCache middleware | Joshua Peek | 2008-12-22 | 3 | -20/+2 |
| | |||||
* | Use I18n for date/time select helpers prompt text [#561 state:resolved] | Sam Oliver | 2008-12-22 | 3 | -6/+20 |
| | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
* | Added prompt options to date helpers [#561 state:resolved] | Sam Oliver | 2008-12-21 | 3 | -0/+402 |
| | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
* | Ensure the template format is always passed to the template finder. Now we ↵ | Joshua Peek | 2008-12-21 | 7 | -104/+93 |
| | | | | can cleanup some nasty stuff. | ||||
* | Merge commit 'fred/more_pullable' | Pratik Naik | 2008-12-21 | 2 | -3/+37 |
|\ | |||||
| * | Allow use of symbols for :type option of ↵ | =?utf-8?q?Adam=20Cig=C3=A1nek?= | 2008-12-21 | 2 | -3/+37 |
| | | | | | | | | | | | | ActionController::Streaming#send_file/#send_data [#1232 state:resolved] Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com> | ||||
* | | Fix failing cookie store test | Frederick Cheung | 2008-12-21 | 1 | -3/+3 |
| | | |||||
* | | Remove observe_field :on option as prototype no longer supports it [#1088 ↵ | Frederick Cheung | 2008-12-21 | 1 | -11/+0 |
|/ | | | | state:resolved] | ||||
* | Switch to Rack::Response#set_cookie instead of using CGI::Cookie to build ↵ | Joshua Peek | 2008-12-20 | 6 | -107/+66 |
| | | | | cookie headers | ||||
* | Fix Mime::Type#=~ not using Regexp.quote | Frederick Cheung | 2008-12-21 | 2 | -1/+2 |
| | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
* | Fix reset_session with lazy cookie stores [#1601 state:resolved] | Matt Bauer | 2008-12-20 | 4 | -13/+64 |
| | | | | Signed-off-by: Joshua Peek <josh@joshpeek.com> | ||||
* | Fix duplicate test name [#1058 state:resolved] [Dave Rothlisberger, Pratik Naik] | Pratik Naik | 2008-12-20 | 1 | -1/+1 |
| | |||||
* | Rename AbstractResponse to Response and inheirt from Rack::Response | Joshua Peek | 2008-12-19 | 10 | -137/+86 |
| | |||||
* | Process CGI 'cookie' header into 'Set-Cookie' for all responses. | Joshua Peek | 2008-12-19 | 4 | -44/+49 |
| | | | | This mostly affects response.headers['cookie'] for test requests. Use response.cookies instead. | ||||
* | Improve cookie test coverage | Joshua Peek | 2008-12-19 | 1 | -19/+24 |
| | |||||
* | Use status response accessor instead of the 'Status' header | Joshua Peek | 2008-12-19 | 13 | -27/+20 |
| | |||||
* | Merge docrails | Pratik Naik | 2008-12-19 | 2 | -6/+14 |
| | |||||
* | Revert "Auto-load template handlers based on unmatched extensions [#1540 ↵ | Pratik Naik | 2008-12-19 | 2 | -27/+8 |
| | | | | | | | | | | state:resolved]" This reverts commit e8c1915416579a3840573ca2c80822d96cb31823. Reasons : - ActionPack tests run very slow - Gem.searcher hanging for long time when extension is nil |