aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Reindent and remove wrong line left in merge by mistake.Emilio Tagua2011-02-181-4/+3
| | |
* | | Use to_a instead :load in test, since :load changed.Emilio Tagua2011-02-181-1/+1
| | |
* | | No need to test against target anymore.Emilio Tagua2011-02-181-8/+8
| | |
* | | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-1847-791/+865
|\| | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/association.rb activerecord/lib/active_record/fixtures.rb
| * | Split AssociationProxy into an Association class (and subclasses) which ↵Jon Leighton2011-02-1826-502/+392
| | | | | | | | | | | | manages the association, and a CollectionProxy class which is *only* a proxy. Singular associations no longer have a proxy. See CHANGELOG for more.
| * | Add :nodoc: to ThroughAssociation and HasOneAssociationJon Leighton2011-02-182-2/+2
| | |
| * | Allow building and then later saving has_many :through records, such that ↵Jon Leighton2011-02-187-12/+124
| | | | | | | | | | | | the join record is automatically saved too. This requires the :inverse_of option to be set on the source association in the join model. See the CHANGELOG for details. [#4329 state:resolved]
| * | Ensure that association_ids uses the correct attribute where the association ↵Jon Leighton2011-02-184-2/+20
| | | | | | | | | | | | is a has_many :through with a :primary_key option on the source reflection. [#6376 state:resolved]
| * | Remove unused line in test setup.Ben Orenstein2011-02-171-1/+0
| | | | | | | | | | | | | | | | | | [#6442 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | explicitly allowing lolqueriesAaron Patterson2011-02-162-13/+11
| | |
| * | use the arel table rather than generating stringsAaron Patterson2011-02-161-1/+1
| | |
| * | no need for Array.wrapAaron Patterson2011-02-161-1/+1
| | |
| * | fixing whitespace problemsAaron Patterson2011-02-161-3/+3
| | |
| * | removed an unnecessary second query when passing an ActiveRecord::Relation ↵Steven Fenigstein2011-02-162-2/+32
| | | | | | | | | | | | to a where clause. And added ability to use subselects in where clauses.
| * | Fixed issue #6363, avoid to pluralized already pluralized names and ↵Timothy N. Tsvetkov2011-02-162-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | singularize a single in generators, for example stadia is a valid plural for stadium. But calling pluralize for stadia will return stadias which sematically is not corrent in this case [#6363 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | handle double pluralization for irregular pluralsDan Pickett2011-02-163-0/+12
| | | | | | | | | | | | | | | | | | [#6363] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | Remove unused code.Ben Orenstein2011-02-161-3/+0
| | | | | | | | | | | | | | | | | | [#6437 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | fixture methods can be overridden and super() is usefulAaron Patterson2011-02-152-14/+32
| | |
| * | fixing variable names to reflect realityAaron Patterson2011-02-151-11/+11
| | |
| * | this is in the AR namespace, so AR must existAaron Patterson2011-02-151-1/+1
| | |
| * | GcTime incorrectly checks GC.respond_to?(:total_time), it should check ↵Joel Nimety2011-02-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | GC::Profiler.respond_to?(:total_time) [#6435 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | Optimization for postgres adapter for add_column method. Set default and not ↵Timothy N. Tsvetkov2011-02-151-6/+10
| | | | | | | | | | | | null in one query
| * | fixing variable names, removing FixtureFile instances from test ivarsAaron Patterson2011-02-152-21/+14
| | |
| * | removing unused variableAaron Patterson2011-02-151-3/+1
| | |
| * | ActiveResource validation tests did not test ActiveModel validations. Adjust ↵Ken Mazaika2011-02-152-13/+12
| | | | | | | | | | | | | | | | | | the test to be done the Rails3 way. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * | fixed broken link to postgresql docsSergii Boiko2011-02-151-1/+1
| | | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | | Fix expected queries in relation tests.Emilio Tagua2011-02-151-2/+2
| | |
* | | Don't shadow outer local variable.Emilio Tagua2011-02-151-2/+2
| | |
* | | Run tests without IdentityMap when IM=false is given.Emilio Tagua2011-02-156-15/+23
| | |
* | | Remove identity map from benchmark script.Emilio Tagua2011-02-151-7/+0
| | |
* | | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-15430-9198/+12029
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/examples/performance.rb activerecord/lib/active_record/association_preload.rb activerecord/lib/active_record/associations.rb activerecord/lib/active_record/associations/association_proxy.rb activerecord/lib/active_record/autosave_association.rb activerecord/lib/active_record/base.rb activerecord/lib/active_record/nested_attributes.rb activerecord/test/cases/relations_test.rb
| * | bad tests are badAaron Patterson2011-02-141-18/+0
| | |
| * | remove accidental raise!Aaron Patterson2011-02-141-1/+0
| | |
| * | removing irrelevant testAaron Patterson2011-02-142-18/+1
| | |
| * | Merge branch 'fixtures'Aaron Patterson2011-02-141-28/+28
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fixtures: remove unused methods extract database activity out of Fixtures instances extract rows that should be inserted to a method fixtures will return a list of tables that may be effected, delete existing fixtures will delete those tables
| | * | remove unused methodsAaron Patterson2011-02-142-40/+0
| | | |
| | * | extract database activity out of Fixtures instancesAaron Patterson2011-02-141-7/+23
| | | |
| | * | extract rows that should be inserted to a methodAaron Patterson2011-02-141-12/+12
| | | |
| | * | fixtures will return a list of tables that may be effected, delete existing ↵Aaron Patterson2011-02-142-6/+30
| | | | | | | | | | | | | | | | fixtures will delete those tables
| * | | Merge remote branch 'jonleighton/association_fixes'Aaron Patterson2011-02-1423-117/+171
| |\ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * jonleighton/association_fixes: Add a transaction wrapper in add_to_target. This means that #build will now also use a transaction. IMO this is reasonable given that the before_add and after_add callbacks might do anything, and this great consistency allows us to abstract out the duplicate code from #build and #create. Inline ensure_owner_is_persisted! as it is only called from one place @target should always be an array Rename add_record_to_target_with_callbacks to add_to_target Don't pass the block through build_record Move create and create! next to build Get rid of create_record as it is not only used in one place Get rid of AssociationCollection#save_record Fix test/cases/connection_pool_test.rb for sqlite3 in-memory db Add interpolation of association conditions back in, in the form of proc { ... } rather than instance_eval-ing strings
| | * | Add a transaction wrapper in add_to_target. This means that #build will now ↵Jon Leighton2011-02-141-27/+27
| | | | | | | | | | | | | | | | also use a transaction. IMO this is reasonable given that the before_add and after_add callbacks might do anything, and this great consistency allows us to abstract out the duplicate code from #build and #create.
| | * | Inline ensure_owner_is_persisted! as it is only called from one placeJon Leighton2011-02-141-8/+4
| | | |
| | * | @target should always be an arrayJon Leighton2011-02-141-1/+2
| | | |
| | * | Rename add_record_to_target_with_callbacks to add_to_targetJon Leighton2011-02-142-5/+5
| | | |
| | * | Don't pass the block through build_recordJon Leighton2011-02-141-10/+8
| | | |
| | * | Move create and create! next to buildJon Leighton2011-02-141-21/+21
| | | |
| | * | Get rid of create_record as it is not only used in one placeJon Leighton2011-02-141-8/+7
| | | |
| | * | Get rid of AssociationCollection#save_recordJon Leighton2011-02-145-26/+26
| | | |
| | * | Fix test/cases/connection_pool_test.rb for sqlite3 in-memory dbJon Leighton2011-02-141-0/+10
| | | |
| | * | Add interpolation of association conditions back in, in the form of proc { ↵Jon Leighton2011-02-1418-47/+97
| | | | | | | | | | | | | | | | ... } rather than instance_eval-ing strings