Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Get rid of extra_conditions param from configure_dependency_for_has_many. I ↵ | Jon Leighton | 2010-12-31 | 1 | -9/+3 |
| | | | | can't see a particularly plausible argument for this being used by plugins, and if they really want they can just redefine the callback or whatever. Note also that before my recent commit the extra_conditions param was completely ignored for :dependent => :destroy. | ||||
* | Refactor configure_dependency_for_has_many to use ↵ | Jon Leighton | 2010-12-31 | 1 | -38/+20 |
| | | | | AssociationCollection#delete_all. It was necessary to change test_before_destroy in lifecycle_test.rb so that it checks topic.replies.size *before* doing the destroy, as afterwards it will now (correctly) be 0. | ||||
* | Add a HasAssociation module for common code for has_* associations | Jon Leighton | 2010-12-26 | 1 | -0/+2 |
| | |||||
* | Raise an error for associations which try to go :through a polymorphic ↵ | Jon Leighton | 2010-12-23 | 1 | -1/+7 |
| | | | | association [#6212 state:resolved] | ||||
* | If a has_many goes :through a belongs_to, and the foreign key of the ↵ | Jon Leighton | 2010-12-23 | 1 | -1/+5 |
| | | | | belongs_to changes, then the has_many should be considered stale. | ||||
* | Improved strategy for updating a belongs_to association when the foreign key ↵ | Jon Leighton | 2010-12-23 | 1 | -42/+1 |
| | | | | changes. Rather than resetting each affected association when the foreign key changes, we should lazily check for 'staleness' (where fk does not match target id) when the association is accessed. | ||||
* | Revert "Optimize <association>_ids for hm:t with belongs_to source". The ↵ | Jon Leighton | 2010-12-23 | 1 | -8/+1 |
| | | | | | | | | | | optimisation has too many edge cases, such as when the reflection, source reflection, or through reflection has conditions, orders, etc. [#6153 state:resolved] This reverts commit 373b053dc8b99dac1abc3879a17a2bf8c30302b5. Conflicts: activerecord/lib/active_record/associations.rb | ||||
* | Don't allow a has_one association to go :through a collection association ↵ | Jon Leighton | 2010-12-23 | 1 | -0/+6 |
| | | | | [#2976 state:resolved] | ||||
* | removing method to prevent warnings | Aaron Patterson | 2010-12-08 | 1 | -0/+4 |
| | |||||
* | Setting the id of a belongs_to object updates all referenced objects [#2989 ↵ | Jeff Dean | 2010-12-08 | 1 | -0/+37 |
| | | | | state:resolved] | ||||
* | breaking classes up in to respective files | Aaron Patterson | 2010-11-23 | 1 | -574/+1 |
| | |||||
* | class inheritable attributes is used no more! all internal use of class ↵ | Josh Kalderimis | 2010-11-20 | 1 | -6/+7 |
| | | | | | | inheritable has been changed to class_attribute. class inheritable attributes has been deprecated. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | use persisted? instead of new_record? wherever possible | David Chelimsky | 2010-11-09 | 1 | -1/+1 |
| | | | | | | | | | | | - persisted? is the API defined in ActiveModel - makes it easier for extension libraries to conform to ActiveModel APIs without concern for whether the extended object is specifically ActiveRecord [#5927 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | reduce duplication in assiciations #construct() | Aaron Patterson | 2010-11-06 | 1 | -9/+3 |
| | |||||
* | make sure we keep parent records in order | Aaron Patterson | 2010-11-06 | 1 | -6/+6 |
| | |||||
* | only call to_s on the association variable once | Aaron Patterson | 2010-11-05 | 1 | -2/+5 |
| | |||||
* | simplify instantiate in the join parts object | Aaron Patterson | 2010-11-05 | 1 | -8/+7 |
| | |||||
* | index is not used, so removing it | Aaron Patterson | 2010-11-05 | 1 | -1/+1 |
| | |||||
* | reduce method calls to the join base object | Aaron Patterson | 2010-11-05 | 1 | -4/+6 |
| | |||||
* | refactoring find_join_association | Aaron Patterson | 2010-10-30 | 1 | -5/+6 |
| | |||||
* | Fix issues when including the same association multiple times and mixing ↵ | Ernie Miller | 2010-10-30 | 1 | -11/+37 |
| | | | | joins/includes together. | ||||
* | removing space errors | Aaron Patterson | 2010-10-22 | 1 | -45/+45 |
| | |||||
* | Delete unused methods in JoinAssociation | Jon Leighton | 2010-10-13 | 1 | -8/+0 |
| | |||||
* | Renaming and formatting changes in JoinDependency | Jon Leighton | 2010-10-13 | 1 | -47/+51 |
| | |||||
* | Refactor JoinDependency and friends so that a JoinAssociation can produce an ↵ | Jon Leighton | 2010-10-13 | 1 | -145/+265 |
| | | | | arbitrary number of joins, which will be needed in order to support nested through associations. | ||||
* | avoid a proc object | Aaron Patterson | 2010-09-22 | 1 | -1/+1 |
| | |||||
* | avoid creating a range object | Aaron Patterson | 2010-09-22 | 1 | -1/+1 |
| | |||||
* | drying up construct_association | Aaron Patterson | 2010-09-21 | 1 | -15/+15 |
| | |||||
* | dry up some conditionals | Aaron Patterson | 2010-09-21 | 1 | -3/+4 |
| | |||||
* | fixing case / when indentation | Aaron Patterson | 2010-09-21 | 1 | -19/+19 |
| | |||||
* | fix case / when indentation | Aaron Patterson | 2010-09-21 | 1 | -18/+18 |
| | |||||
* | avoid a hash lookup | Aaron Patterson | 2010-09-21 | 1 | -2/+2 |
| | |||||
* | refacoring duplicate code. <3 <3 <3 | Aaron Patterson | 2010-09-21 | 1 | -2/+3 |
| | |||||
* | this reject can be replaced with compact | Aaron Patterson | 2010-09-21 | 1 | -1/+1 |
| | |||||
* | removing unused code | Aaron Patterson | 2010-09-21 | 1 | -2/+1 |
| | |||||
* | remove some repeated code | Aaron Patterson | 2010-09-21 | 1 | -2/+2 |
| | |||||
* | break up giant method | Aaron Patterson | 2010-09-21 | 1 | -62/+72 |
| | |||||
* | removing lolinject | Aaron Patterson | 2010-09-21 | 1 | -1/+1 |
| | |||||
* | remove join table rows before removing owner row for habtm associations, ↵ | Hemant Kumar | 2010-09-21 | 1 | -1/+1 |
| | | | | fixes#5674 | ||||
* | providing arel with column information when possible [#5392 state:resolved] | Aaron Patterson | 2010-09-21 | 1 | -3/+10 |
| | |||||
* | Use new finders syntax in docs. | Emilio Tagua | 2010-09-01 | 1 | -1/+1 |
| | |||||
* | lifecycle should be two words, life cycle | Jaime Iniesta | 2010-08-26 | 1 | -1/+1 |
| | |||||
* | Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵ | Santiago Pastorino | 2010-08-14 | 1 | -172/+172 |
| | | | | 's/[ \t]*$//' -i {} \;) | ||||
* | converting to a symbol is not necessary | Aaron Patterson | 2010-08-13 | 1 | -1/+1 |
| | |||||
* | Removing most of the symbol to proc usage in Active Record | Prem Sichanugrist | 2010-08-13 | 1 | -6/+6 |
| | | | | This will hopefully make Active Record run a bit more faster. | ||||
* | be more precise re :validate and :autosave | Xavier Noria | 2010-08-12 | 1 | -4/+4 |
| | |||||
* | revises some autosave docs, style and content | Xavier Noria | 2010-08-12 | 1 | -2/+1 |
| | |||||
* | adding more documentation for autosave option | Neeraj Singh | 2010-08-09 | 1 | -1/+2 |
| | |||||
* | updating documentation for method configure_dependency_for_has_many | Neeraj Singh | 2010-08-06 | 1 | -8/+5 |
| | |||||
* | Merge remote branch 'docrails/master' into 3-0-stable | Xavier Noria | 2010-08-03 | 1 | -182/+269 |
|\ |