aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Adding test coverage and better logging to Rails::TemplateRunner [#1618 ↵Aaron Quint2008-12-272-66/+243
| | | | | | | | | | | | | | | | | | state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | | Fix ActionPack build on Windows: we really should not test anything ↵Yaroslav Markin2008-12-271-10/+12
| | | | | | | | | | | | | | | | | | regarding symlinks on Windows. Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | | Refactor ActiveRecord::Base#new_record? [#1647 state:committed]Yaroslav Markin2008-12-271-1/+1
| | | | | | | | | | | | Signed-off-by: David Heinemeier Hansson <david@loudthinking.com>
* | | ActiveRecord::Base#new_record? now returns false for existing records (was ↵Yaroslav Markin2008-12-273-2/+9
| | | | | | | | | | | | | | | | | | nil) [#1219 state:committed] Signed-off-by: David Heinemeier Hansson <david@loudthinking.com>
* | | More optimizations on respond_to after a profile and benching:Yehuda Katz2008-12-271-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
* | Merge commit 'fred/pullable'Pratik Naik2008-12-2611-27/+101
|\ \
| * | Preload uses exclusive scope [#643 state:resolved]Frederick Cheung2008-12-265-16/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | With self referential associations, the scope for the the top level should not affect fetching of associations, for example when doing Person.male.find :all, :include => :friends we should load all of the friends for each male, not just the male friends.
| * | Association preloading no longer stops if it hits a nil object [#1630 ↵Pivotal Labs2008-12-262-2/+10
| | | | | | | | | | | | | | | | | | state:resolved] Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>
| * | MaKe Hash#slice! return removed values, akin to Array [#971 state:resolved]trans2008-12-262-4/+25
| | | | | | | | | | | | Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>
| * | Fix assert_select_rjs not checking id for inserts [#540 state:resolved]Frederick Cheung2008-12-262-0/+9
| | |
| * | Fix randomly failing cookie store testsFrederick Cheung2008-12-261-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_toYehuda Katz2008-12-261-4/+23
| | |
* | | Merge docrailsPratik Naik2008-12-263-84/+334
| | |
* | | Don't recurse when ActionController#render is called without any argumentsPratik Naik2008-12-261-1/+1
| | |
* | | Make ActionController#render(symbol) behave same as ↵Pratik Naik2008-12-263-6/+16
| | | | | | | | | | | | ActionController#render(string) [#1435]
* | | Make ActionController#render(string) work as a shortcut for render :action ↵Pratik Naik2008-12-253-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 Naik2008-12-253-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 '/'
* | | Make ActionController#render(string) work as a shortcut for render :file => ↵Pratik Naik2008-12-253-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 methodPratik Naik2008-12-251-8/+14
| | |
* | | Make render_test.rb run in isolationPratik Naik2008-12-251-0/+2
| | |
* | | Undry ActionController::TestCase#<HTTP_METHODS> for better documentationPratik Naik2008-12-252-14/+26
|/ /
* | Move request parsing related code to ActionController::RequestParserPratik Naik2008-12-257-460/+512
| |
* | ActionController::VerbPiggybacking middlewareJoshua Peek2008-12-237-38/+49
| |
* | Remove duplicate attr_reader :envPratik Naik2008-12-231-4/+2
| |
* | Unify ActionController::AbstractRequest and ActionController::RequestPratik Naik2008-12-232-110/+81
| |
* | Use Rack::MockRequest for TestRequestPratik Naik2008-12-233-20/+9
| |
* | Rename RackRequest to RequestPratik Naik2008-12-236-11/+11
| |
* | Remove rack_process.rbPratik Naik2008-12-233-74/+73
| |
* | Remove deprecated relative_url_rootPratik Naik2008-12-231-7/+0
| |
* | Defining a new method is atomic, no mutex needed.Joshua Peek2008-12-221-8/+2
| |
* | Move default middleware stack to middlewares.rbJoshua Peek2008-12-222-17/+21
| |
* | ActiveRecord::QueryCache middlewareJoshua Peek2008-12-225-34/+28
| |
* | Ensure of Model#create support custom updated_at and updated_on attributes ↵Luis Hurtado2008-12-222-2/+12
| | | | | | | | | | | | [#1612 state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | Use I18n for date/time select helpers prompt text [#561 state:resolved]Sam Oliver2008-12-223-6/+20
| | | | | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | Add :allow_nil option to delegate [#1127 state:resolved]Sergio Gil2008-12-213-1/+56
| | | | | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | Ensure Model#last doesn't affects order for another finders inside the same ↵Daniel Luz2008-12-212-3/+26
| | | | | | | | | | | | scope [#1499 state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | Added prompt options to date helpers [#561 state:resolved]Sam Oliver2008-12-213-0/+402
| | | | | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* | Ensure the template format is always passed to the template finder. Now we ↵Joshua Peek2008-12-219-108/+102
| | | | | | | | can cleanup some nasty stuff.
* | Merge commit 'fred/more_pullable'Pratik Naik2008-12-213-4/+38
|\ \
| * | Allow use of symbols for :type option of ↵=?utf-8?q?Adam=20Cig=C3=A1nek?=2008-12-212-3/+37
| | | | | | | | | | | | | | | | | | ActionController::Streaming#send_file/#send_data [#1232 state:resolved] Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>
| * | Fix script/dbconsole not handling numeric password [#1395 state:resolved]pfagiani2008-12-211-1/+1
| | | | | | | | | | | | Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>
* | | Correct description of tmp:create task [#1607 state:resolved] [Roderick van ↵Pratik Naik2008-12-211-2/+2
| | | | | | | | | | | | Domburg]
* | | Fix configure_dependency_for_has_many not quoting conditions properly [#1461 ↵Frederick Cheung2008-12-213-3/+17
| | | | | | | | | | | | state:resolved]
* | | Use explicit order to stop test failing randomlyFrederick Cheung2008-12-211-2/+2
| | |
* | | Fix failing cookie store testFrederick Cheung2008-12-211-3/+3
| | |
* | | Remove observe_field :on option as prototype no longer supports it [#1088 ↵Frederick Cheung2008-12-211-11/+0
| | | | | | | | | | | | state:resolved]
* | | Remove dead commented out code [#1467 state:resolved]Yaroslav Markin2008-12-211-1/+0
|/ / | | | | | | Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>
* / Switch to Rack::Response#set_cookie instead of using CGI::Cookie to build ↵Joshua Peek2008-12-206-107/+66
|/ | | | cookie headers
* Fix Mime::Type#=~ not using Regexp.quoteFrederick Cheung2008-12-212-1/+2
| | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* Fix has many through not quoting table names [#1163 state:resolved]Karthik Krishnan2008-12-203-4/+8
| | | | Signed-off-by: Frederick Cheung <frederick.cheung@gmail.com>