Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | Merge pull request #10500 from kennyj/fix_10450 | Rafael Mendonça França | 2013-09-22 | 1 | -3/+3 | |
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed a bug in when using has_many association with :inverse_of option and UUID primary key. Conflicts: activerecord/CHANGELOG.md | |||||
| | * | | | | | Fixed a bug in when using has_many association with :inverse_of option and ↵ | kennyj | 2013-05-08 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | UUID primary key. | |||||
| * | | | | | | Add back options argument in the ActiveRecord::Base.initialize method | Rafael Mendonça França | 2013-09-21 | 1 | -2/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will make easier to hook protected_attributes gem in our code without making that gem fragile to change in Rails code base. Closes #12243 | |||||
* | | | | | | | remove initialize method | Aaron Patterson | 2013-09-25 | 1 | -4/+0 | |
| | | | | | | | ||||||
* | | | | | | | extract association resetting to a method | Aaron Patterson | 2013-09-25 | 1 | -9/+16 | |
| | | | | | | | ||||||
* | | | | | | | hash insertion order doesn't matter anymore, so only loop over the | Aaron Patterson | 2013-09-25 | 1 | -4/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | owners once | |||||
* | | | | | | | always populate the preloaded records instance variable so we can remove | Aaron Patterson | 2013-09-25 | 2 | -10/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | the @associated_records_by_owner ivar | |||||
* | | | | | | | keep preloaded records in a list rather than extract from a hash | Aaron Patterson | 2013-09-25 | 3 | -16/+21 | |
| | | | | | | | ||||||
* | | | | | | | push slice loading to it's own method so we can remove the type casting | Aaron Patterson | 2013-09-24 | 2 | -24/+20 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | code | |||||
* | | | | | | | guarantee that `klass` is not nil inside the preloader objects | Aaron Patterson | 2013-09-24 | 2 | -5/+12 | |
| | | | | | | | ||||||
* | | | | | | | eliminate unused ivar | Aaron Patterson | 2013-09-24 | 1 | -1/+0 | |
| | | | | | | | ||||||
* | | | | | | | all records have a preloaded, so eliminate that conditional | Aaron Patterson | 2013-09-24 | 1 | -11/+7 | |
| | | | | | | | ||||||
* | | | | | | | eliminate the `loaded?` conditional | Aaron Patterson | 2013-09-24 | 3 | -12/+7 | |
| | | | | | | | ||||||
* | | | | | | | push preloaded test up to the factory method so we can eliminate | Aaron Patterson | 2013-09-24 | 3 | -8/+21 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | conditionals from the individual preloaded classes | |||||
* | | | | | | | we can't sort by lhs since the middle records have difference classes | Aaron Patterson | 2013-09-23 | 1 | -19/+19 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | and possibly different rules for finding those objects | |||||
* | | | | | | | pass the preloader down so we only have to construct one | Aaron Patterson | 2013-09-23 | 7 | -21/+21 | |
| | | | | | | | ||||||
* | | | | | | | reuse the same preloader object for each association | Aaron Patterson | 2013-09-23 | 1 | -2/+2 | |
| | | | | | | | ||||||
* | | | | | | | remove state from the preloader | Aaron Patterson | 2013-09-23 | 3 | -28/+18 | |
| | | | | | | | ||||||
* | | | | | | | pass the scope all the way down to the constructors | Aaron Patterson | 2013-09-23 | 1 | -10/+10 | |
| | | | | | | | ||||||
* | | | | | | | only do the should_reset test once | Aaron Patterson | 2013-09-23 | 1 | -6/+6 | |
| | | | | | | | ||||||
* | | | | | | | fix variable names and speed up relation ordering | Aaron Patterson | 2013-09-23 | 2 | -16/+12 | |
| | | | | | | | ||||||
* | | | | | | | hm:t preloading will respect order set on the RHS association | Aaron Patterson | 2013-09-23 | 5 | -8/+53 | |
| | | | | | | | ||||||
* | | | | | | | cache associated target records hash | Aaron Patterson | 2013-09-20 | 1 | -17/+11 | |
| | | | | | | | ||||||
* | | | | | | | combine methods so we can reuse preloaders | Aaron Patterson | 2013-09-20 | 1 | -19/+18 | |
| | | | | | | | ||||||
* | | | | | | | return a list rather than hash | Aaron Patterson | 2013-09-20 | 1 | -7/+13 | |
| | | | | | | | ||||||
* | | | | | | | preserve order on the RHS query | Aaron Patterson | 2013-09-20 | 1 | -0/+8 | |
| | | | | | | | ||||||
* | | | | | | | split up construction and mutation | Aaron Patterson | 2013-09-20 | 1 | -1/+4 | |
| | | | | | | | ||||||
* | | | | | | | fix method name | Aaron Patterson | 2013-09-20 | 1 | -3/+5 | |
| | | | | | | | ||||||
* | | | | | | | access preloaders independently of model mutations | Aaron Patterson | 2013-09-20 | 1 | -12/+12 | |
| | | | | | | | ||||||
* | | | | | | | push running the preloaders up | Aaron Patterson | 2013-09-20 | 1 | -19/+7 | |
| | | | | | | | ||||||
* | | | | | | | push `run` up to preload | Aaron Patterson | 2013-09-20 | 3 | -14/+41 | |
| | | | | | | | ||||||
* | | | | | | | the hash should only ever be length one, so decompose it | Aaron Patterson | 2013-09-20 | 1 | -4/+5 | |
| | | | | | | | ||||||
* | | | | | | | stop depending on preloader interals so we can create fewer preloader | Aaron Patterson | 2013-09-20 | 1 | -19/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | objects | |||||
* | | | | | | | lhs preload is always a single preload, so just preload one | Aaron Patterson | 2013-09-20 | 1 | -3/+3 | |
| | | | | | | | ||||||
* | | | | | | | just read the attribute rather than `send`ing | Aaron Patterson | 2013-09-20 | 1 | -1/+1 | |
| | | | | | | | ||||||
* | | | | | | | extract exception raising, clean up group_by statement | Aaron Patterson | 2013-09-20 | 1 | -6/+7 | |
| | | | | | | | ||||||
* | | | | | | | generate fewer objects when grouping | Aaron Patterson | 2013-09-20 | 1 | -5/+7 | |
| | | | | | | | ||||||
* | | | | | | | do not access internal data structures | Aaron Patterson | 2013-09-20 | 1 | -1/+1 | |
| | | | | | | | ||||||
* | | | | | | | exposing target records on the preloader | Aaron Patterson | 2013-09-20 | 2 | -16/+29 | |
|/ / / / / / | ||||||
* | | | | | | support objects with blank string primary keys | Aaron Patterson | 2013-09-18 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | ActiveRecord::Base#<=> has been removed. Primary keys may not be in order, | Aaron Patterson | 2013-09-18 | 1 | -9/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | or even be numbers, so sorting by id doesn't make sense. Please use `sort_by` and specify the attribute you wish to sort with. For example, change: Post.all.to_a.sort to: Post.all.to_a.sort_by(&:id) | |||||
* | | | | | | do what the superclass does in the case that objects do not match | Aaron Patterson | 2013-09-18 | 1 | -0/+2 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #12011 from jetthoughts/11963_fix_join_with_association_scope | Rafael Mendonça França | 2013-09-16 | 1 | -6/+5 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Collapse where constraints to the Arel::Nodes::And node Conflicts: activerecord/CHANGELOG.md | |||||
| * | | | | | | Collapse where constraints to one where constraint | Paul Nikitochkin | 2013-09-13 | 1 | -6/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to remove duplication with joining arel where constraints with `AND`, all constraints on `build_arel` are collapsed into one head node: `Arel::Nodes::And` Closes: #11963 | |||||
* | | | | | | | Removes unused code related to DatabaseTasks. | kennyj | 2013-09-16 | 1 | -4/+0 | |
| | | | | | | | ||||||
* | | | | | | | Merge pull request #12129 from Empact/deprecate-array-bang-delegation | Rafael Mendonça França | 2013-09-12 | 1 | -3/+14 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deprecate the delegation of Array bang methods in ActiveRecord::Delegation Conflicts: activerecord/CHANGELOG.md activerecord/test/cases/relation_test.rb | |||||
| * | | | | | | | Deprecate the delegation of Array bang methods in ActiveRecord::Delegation | Ben Woosley | 2013-09-04 | 1 | -3/+14 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The primary means of returning results for Array bang methods is to modify the array in-place. When you call these methods on a relation, that array is created, modified, and then thrown away. Only the secondary return value is exposed to the caller. Removing this delegation is a straight-forward way to reduce user error by forcing callers to first explicitly call #to_a in order to expose the array to be acted on by the bang method. | |||||
* | | | | | | | | Merge pull request #12137 from lann/fix_association_first_last | Rafael Mendonça França | 2013-09-12 | 1 | -3/+1 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make CollectionAssociation first/last with integer fetch with query Conflicts: activerecord/CHANGELOG.md | |||||
| * | | | | | | | | Make CollectionAssociation first/last with integer fetch with query | Lann Martin | 2013-09-09 | 1 | -3/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When first or last is called with an integer on an unloaded association, the entire collection is loaded. This differs surprisingly from the behavior of Relation#first/last, which translate the call into a limit query. For large collections this can make a big difference in performance. Change CollectionAssociation#fetch_first_or_last_using_find? to make this kind of call delegate to Relation. | |||||
* | | | | | | | | | Merge pull request #11698 from dmathieu/sqlite_tasks_without_rails | Rafael Mendonça França | 2013-09-12 | 3 | -2/+4 | |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let the SQLite rake tasks run without rails |