Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | ||||
| * | | | | 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 | 1 | -0/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 1 | -0/+1 |
| | | | | | | |||||
* | | | | | | 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 |
| | | | | | | |||||
* | | | | | | 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 | 8 | -7/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 3 | -11/+15 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Perf: avoid dupes add fallback logic for coders | ||||
| * | | | | | | Perf: avoid dupes add fallback logic for coders | Sam | 2013-09-11 | 3 | -11/+15 |
| | | | | | | | |||||
* | | | | | | | Merge pull request #12135 from dylanahsmith/avoid_empty_transaction | Rafael Mendonça França | 2013-09-11 | 1 | -1/+3 |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | 1 | -1/+2 |
| | | | | | | | | |||||
* | | | | | | | | 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 | 1 | -1/+5 |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Relation#merge should not lose readonly(false) flag. | ||||
| * | | | | | | | | | Relation#merge should not lose readonly(false) flag. | thedarkone | 2013-09-11 | 1 | -1/+5 |
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The original code ignores the `false` value because `false.blank? # => true`. | ||||
* | / / / / / / / | fix deleting join models with no pk | Aaron Patterson | 2013-09-11 | 1 | -1/+15 |
| |/ / / / / / / |/| | | | | | | | |||||
* | | | | | | | | ask the association for records rather than calling `send` | Aaron Patterson | 2013-09-10 | 1 | -1/+2 |
|/ / / / / / / | |||||
* | | | | | | | let the class cache object clean up user input | Aaron Patterson | 2013-09-09 | 1 | -8/+20 |
| | | | | | | | |||||
* | | | | | | | Fix fixtures regression that required table names to map to classes only, ↵ | Jeremy Kemper | 2013-09-08 | 1 | -1/+1 |
| |/ / / / / |/| | | | | | | | | | | | | | | | | | not class names | ||||
* | | | | | | hm:t join tables may not have a primary key | Aaron Patterson | 2013-09-06 | 1 | -1/+5 |
| | | | | | | |||||
* | | | | | | use polymorphic proxies to remove duplicate code | Aaron Patterson | 2013-09-06 | 1 | -24/+43 |
| | | | | | | |||||
* | | | | | | reduce duplication somewhat | Aaron Patterson | 2013-09-06 | 1 | -14/+14 |
| | | | | | | |||||
* | | | | | | make the duplicate code more similar | Aaron Patterson | 2013-09-06 | 1 | -4/+10 |
| | | | | | | |||||
* | | | | | | auto link hm:t fixture join tables like habtm | Aaron Patterson | 2013-09-06 | 1 | -0/+16 |
| |_|_|_|/ |/| | | | | |||||
* | | | | | don't depend on object state when reading fixture files | Aaron Patterson | 2013-09-06 | 1 | -8/+7 |
| | | | | | |||||
* | | | | | ensure @model_class is initialized | Aaron Patterson | 2013-09-06 | 1 | -0/+1 |
| | | | | | |||||
* | | | | | no need to dup | Aaron Patterson | 2013-09-05 | 1 | -1/+1 |
| | | | | | |||||
* | | | | | push logic outside the FixtureSet constructor | Aaron Patterson | 2013-09-05 | 1 | -3/+3 |
| | | | | | |||||
* | | | | | only construct the FixtureSet with AR models | Aaron Patterson | 2013-09-05 | 1 | -2/+11 |
| | | | | | |||||
* | | | | | push constant lookup in the a cache object | Aaron Patterson | 2013-09-05 | 1 | -2/+21 |
| |_|/ / |/| | | | |||||
* | | | | Merge pull request #11958 from jetthoughts/extract_pre_process_orders_args | Yves Senn | 2013-09-04 | 1 | -13/+16 |
|\ \ \ \ | |_|_|/ |/| | | | Re-use order arguments pre-processing for reorder | ||||
| * | | | Extracted from `order` processing of arguments, and use it for `reorder` to ↵ | Paul Nikitochkin | 2013-09-04 | 1 | -13/+16 |
| | | | | | | | | | | | | | | | | be consistent. |