Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 |
| | |||||
* | remove unused variable | Aaron Patterson | 2013-09-18 | 1 | -1/+0 |
| | |||||
* | support objects with blank string primary keys | Aaron Patterson | 2013-09-18 | 2 | -1/+7 |
| | |||||
* | ActiveRecord::Base#<=> has been removed. Primary keys may not be in order, | Aaron Patterson | 2013-09-18 | 3 | -16/+10 |
| | | | | | | | | | | | 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 | 2 | -0/+10 |
| | |||||
* | Merge pull request #12011 from jetthoughts/11963_fix_join_with_association_scope | Rafael Mendonça França | 2013-09-16 | 4 | -6/+25 |
|\ | | | | | | | | | | | | | 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 | 4 | -6/+25 |
| | | | | | | | | | | | | | | 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 | 5 | -147/+265 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | 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 | 3 | -3/+117 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
| * | | Pull the RelationMutationTests from cases/relation_test to ↵ | Ben Woosley | 2013-09-03 | 2 | -135/+139 |
| | | | | | | | | | | | | cases/relation/mutation_test. | ||||
* | | | Merge pull request #12137 from lann/fix_association_first_last | Rafael Mendonça França | 2013-09-12 | 3 | -6/+11 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 3 | -6/+11 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | | Add CHANGELOG entry for #11698 | Rafael Mendonça França | 2013-09-12 | 1 | -0/+4 |
| | | | | | | | | | | | | | | | | [ci skip] | ||||
* | | | | 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 | ||||
| * | | | | let the sqlite task run without rails | Damien Mathieu | 2013-08-07 | 3 | -2/+4 |
| | | | | | |||||
* | | | | | Merge pull request #12156 from rywall/from-copy-binds | Rafael Mendonça França | 2013-09-12 | 3 | -0/+12 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow Relation#from to accept other relations with bind values. Conflicts: activerecord/CHANGELOG.md | ||||
| * | | | | | Allow Relation#from to accept other relations with bind values. | Ryan Wallace | 2013-09-08 | 3 | -0/+12 |
| | | | | | | |||||
* | | | | | | Remove unnecessary loop | Vipul A M | 2013-09-13 | 1 | -5/+2 |
| | | | | | | |||||
* | | | | | | Disable prepared statements in the unprepared_statement block | Rafael Mendonça França | 2013-09-12 | 1 | -2/+3 |
| | | | | | | |||||
* | | | | | | Fix typo on instance variable get call | Carlos Antonio da Silva | 2013-09-12 | 1 | -1/+1 |
| | | | | | | |||||
* | | | | | | these are not real developer objects, so counting them doesn't make | Aaron Patterson | 2013-09-11 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | sense. Let's load the object to ensure it's an array and count the array. | ||||
* | | | | | | Remove conditional adding a new method | Rafael Mendonça França | 2013-09-11 | 2 | -6/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At the point we want to add the reflection we already know the reflection is of the AggregateReflection type so we can call a specific method | ||||
* | | | | | | Fix inverted conditional | Rafael Mendonça França | 2013-09-11 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | Oops, I should had run the tests | ||||
* | | | | | | Remove invalid comment | Rafael Mendonça França | 2013-09-11 | 1 | -4/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | This is not valid anymore after 08477a651648ba4417ded128aa37b9ae0dcbc9ce | ||||
* | | | | | | Check if the SQL is not a prepared statement | Rafael Mendonça França | 2013-09-11 | 10 | -7/+29 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the adapter is with prepared statement disabled and the binds array is not empty the connection adapter will try to set the binds values and will fail. Now we are checking if the adapter has the prepared statement disabled. Fixes #12023 | ||||
* | | | | | | Merge pull request #12188 from SamSaffron/master | Aaron Patterson | 2013-09-11 | 4 | -12/+16 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Perf: avoid dupes add fallback logic for coders | ||||
| * | | | | | | Perf: avoid dupes add fallback logic for coders | Sam | 2013-09-11 | 4 | -12/+16 |
| | | | | | | | |||||
* | | | | | | | Merge pull request #12135 from dylanahsmith/avoid_empty_transaction | Rafael Mendonça França | 2013-09-11 | 3 | -1/+10 |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid empty transaction from setting has_one association on new record. Conflicts: activerecord/CHANGELOG.md | ||||
| * | | | | | | | Avoid empty transaction from setting has_one association on new record. | Dylan Thacker-Smith | 2013-09-11 | 3 | -1/+9 |
| | | | | | | | | |||||
* | | | | | | | | Whitespaces | Rafael Mendonça França | 2013-09-11 | 1 | -1/+2 |
| | | | | | | | | |||||
* | | | | | | | | Merge pull request #12185 from SamSaffron/join_dep | Rafael Mendonça França | 2013-09-11 | 1 | -1/+13 |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Reduce allocations when extracting AR models | ||||
| * | | | | | | | | Reduce allocations when extracting AR models | Sam | 2013-09-11 | 1 | -1/+13 |
| | | | | | | | | | |||||
* | | | | | | | | | Merge pull request #12194 from thedarkone/readonly-merger-fix | Rafael Mendonça França | 2013-09-11 | 3 | -1/+17 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Relation#merge should not lose readonly(false) flag. | ||||
| * | | | | | | | | | Relation#merge should not lose readonly(false) flag. | thedarkone | 2013-09-11 | 3 | -1/+17 |
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original code ignores the `false` value because `false.blank? # => true`. | ||||
* | / / / / / / / | fix deleting join models with no pk | Aaron Patterson | 2013-09-11 | 2 | -6/+62 |
| |/ / / / / / / |/| | | | | | | | |||||
* | | | | | | | | add a comment for sanity of other people to come | Aaron Patterson | 2013-09-10 | 1 | -0/+8 |
| | | | | | | | | |||||
* | | | | | | | | ask the association for records rather than calling `send` | Aaron Patterson | 2013-09-10 | 1 | -1/+2 |
| | | | | | | | | |||||
* | | | | | | | | More unused associations in AR test models | Akira Matsuda | 2013-09-10 | 8 | -13/+0 |
| | | | | | | | | |||||
* | | | | | | | | :scissors: [ci skip] | Carlos Antonio da Silva | 2013-09-10 | 1 | -1/+1 |
| | | | | | | | | |||||
* | | | | | | | | change function def self.table_name to self.table_name | Rajarshi Das | 2013-09-10 | 2 | -5/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | change def self.primary_key to self.primary_key change def self.primary_key to self.primary_key | ||||
* | | | | | | | | Clean up unused associations in AR test model | Akira Matsuda | 2013-09-10 | 1 | -3/+0 |
|/ / / / / / / | |||||
* | | | | | | | let the class cache object clean up user input | Aaron Patterson | 2013-09-09 | 1 | -8/+20 |
| | | | | | | |