Commit message (Expand) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | | | | | | | | Remove unused delegation to `reflection.options` in `Preloader::Association` | Ryuta Kamizono | 2017-09-18 | 2 | -4/+1 | |
* | | | | | | | | | | | | | The name of the key on the associated record is abstracted as `reflection.joi... | Ryuta Kamizono | 2017-09-18 | 5 | -23/+14 | |
* | | | | | | | | | | | | | The name of the key on the owner is abstracted as `reflection.join_foreign_key` | Ryuta Kamizono | 2017-09-18 | 4 | -17/+5 | |
* | | | | | | | | | | | | | Extract `associate_records_to_owner` to refactor `Preloader::Association` | Ryuta Kamizono | 2017-09-18 | 3 | -20/+14 | |
* | | | | | | | | | | | | | Fix collided sequence name detection | Ryuta Kamizono | 2017-09-18 | 1 | -1/+8 | |
* | | | | | | | | | | | | | Remove the code that swapping `scope` and `options` | Ryuta Kamizono | 2017-09-18 | 2 | -8/+3 | |
* | | | | | | | | | | | | | `id` (primary key) is not only an integer [ci skip] | Ryuta Kamizono | 2017-09-18 | 1 | -1/+1 | |
* | | | | | | | | | | | | | Ensure returning affected objects for class level `update` and `destroy` | Ryuta Kamizono | 2017-09-18 | 1 | -2/+4 | |
* | | | | | | | | | | | | | Place class level `update`, `destroy`, and `delete` in `Persistence::ClassMet... | Ryuta Kamizono | 2017-09-18 | 3 | -95/+93 | |
* | | | | | | | | | | | | | Early return if `records.empty?` in `Preloader#preload` | Ryuta Kamizono | 2017-09-18 | 1 | -3/+3 | |
* | | | | | | | | | | | | | Don't pass `reflection_scope` to `preload_scope` if `reflection.scope` isn't ... | Ryuta Kamizono | 2017-09-18 | 1 | -1/+3 | |
* | | | | | | | | | | | | | Return `through_scope` only if the scope is not empty scope | Ryuta Kamizono | 2017-09-18 | 1 | -4/+2 | |
* | | | | | | | | | | | | | Remove useless condition in `reset_association` | Ryuta Kamizono | 2017-09-18 | 1 | -2/+1 | |
* | | | | | | | | | | | | | Fix docs describing rollback [ci skip] | dixpac | 2017-09-17 | 1 | -2/+2 | |
|/ / / / / / / / / / / / | ||||||
* | | | | | | | | | | | | Remove unused explicit delegation to `klass` in `relation` | Ryuta Kamizono | 2017-09-14 | 2 | -3/+2 | |
* | | | | | | | | | | | | Don't use `collection.table_name` in `collection_cache_key` | Ryuta Kamizono | 2017-09-14 | 1 | -1/+1 | |
* | | | | | | | | | | | | Don't use `quoted_table_name` in `limited_ids_for` | Ryuta Kamizono | 2017-09-14 | 2 | -2/+8 | |
* | | | | | | | | | | | | `quoted_table_name` doesn't respect table alias | Ryuta Kamizono | 2017-09-14 | 1 | -1/+1 | |
* | | | | | | | | | | | | Make `in_batches` queries to preparable | Ryuta Kamizono | 2017-09-14 | 1 | -3/+10 | |
* | | | | | | | | | | | | PERF: Incorrect memoization in `ActiveRecord::Associations::Preloader::Associ... | Guo Xiang Tan | 2017-09-11 | 1 | -1/+5 | |
* | | | | | | | | | | | | Merge pull request #30524 from tgxworld/recover_plucK_performance | Sean Griffin | 2017-09-07 | 1 | -14/+17 | |
|\ \ \ \ \ \ \ \ \ \ \ \ | ||||||
| * | | | | | | | | | | | | PERF: Recover `ActiveRecord::pluck` performance. | Guo Xiang Tan | 2017-09-06 | 1 | -14/+17 | |
* | | | | | | | | | | | | | Remove unnecessary `join_type` in `AssociationScope` | Ryuta Kamizono | 2017-09-08 | 1 | -5/+1 | |
* | | | | | | | | | | | | | Remove duplicated `klass` method in `AssociationReflection` | Ryuta Kamizono | 2017-09-08 | 1 | -16/+11 | |
* | | | | | | | | | | | | | Remove unused `primary_key_type` and `quoted_table_name` in `Reflection` | Ryuta Kamizono | 2017-09-08 | 1 | -8/+0 | |
* | | | | | | | | | | | | | Fix `quote_default_expression` for UUID with array default | Ryuta Kamizono | 2017-09-08 | 1 | -1/+1 | |
* | | | | | | | | | | | | | Don't pass `table` to `last_chain_scope` and `next_chain_scope` | Ryuta Kamizono | 2017-09-07 | 2 | -19/+17 | |
* | | | | | | | | | | | | | `RuntimeReflection` is not a subclass of `PolymorphicReflection` | Ryuta Kamizono | 2017-09-07 | 1 | -26/+6 | |
* | | | | | | | | | | | | | `has_many :through` with unscope should affect to through scope | Ryuta Kamizono | 2017-09-07 | 2 | -23/+14 | |
* | | | | | | | | | | | | | Don't pass unneeded `reflection` to `add_constraints` | Ryuta Kamizono | 2017-09-05 | 1 | -3/+3 | |
* | | | | | | | | | | | | | Assigning `values` is only necessary when `reflection_scope.where_clause` is ... | Ryuta Kamizono | 2017-09-05 | 1 | -1/+1 | |
* | | | | | | | | | | | | | Don't merge `reflection_scope` if `reflection.scope` isn't given | Ryuta Kamizono | 2017-09-05 | 1 | -1/+1 | |
* | | | | | | | | | | | | | Should quote composite primary key names | Ryuta Kamizono | 2017-09-04 | 1 | -1/+1 | |
|/ / / / / / / / / / / / | ||||||
* | | | / / / / / / / / | Scope in associations should treat nil as `all` | Ryuta Kamizono | 2017-09-04 | 5 | -8/+10 | |
| |_|_|/ / / / / / / / |/| | | | | | | | | | | ||||||
* | | | | | | | | | | | Fix preloading through association with custom scope | Ryuta Kamizono | 2017-09-02 | 1 | -1/+6 | |
* | | | | | | | | | | | `values[:includes]` in `reflection_scope` is not compatible with `through_scope` | Ryuta Kamizono | 2017-09-02 | 1 | -1/+6 | |
* | | | | | | | | | | | Don't call `scope.eager_loading?` when `reflection_scope.where_clause` is empty | Ryuta Kamizono | 2017-09-02 | 1 | -10/+10 | |
* | | | | | | | | | | | `add_reference` should respect column position for both reference id and type... | Ryuta Kamizono | 2017-09-01 | 1 | -1/+1 | |
* | | | | | | | | | | | Clarify intentions around method redefinitions | Matthew Draper | 2017-09-01 | 1 | -3/+2 | |
* | | | | | | | | | | | Merge pull request #30377 from keepcosmos/delegate-missing-methods | Matthew Draper | 2017-08-31 | 1 | -2/+2 | |
|\ \ \ \ \ \ \ \ \ \ \ | ||||||
| * | | | | | | | | | | | Delegate :rindex, :slice, :rotate to 'records' | keepcosmos | 2017-08-24 | 1 | -2/+2 | |
* | | | | | | | | | | | | Fix `can't modify frozen String` error in `DatabaseTasks` | yuuji.yaginuma | 2017-08-30 | 2 | -2/+2 | |
| |/ / / / / / / / / / |/| | | | | | | | | | | ||||||
* | | | | | | | | | | | Merge pull request #30445 from prathamesh-sonpatki/fix-30441 | Kasper Timm Hansen | 2017-08-28 | 1 | -0/+2 | |
|\ \ \ \ \ \ \ \ \ \ \ | ||||||
| * | | | | | | | | | | | Clarify that bulk option is supported only by MySQL [ci skip] | Prathamesh Sonpatki | 2017-08-29 | 1 | -0/+2 | |
* | | | | | | | | | | | | `@previous_reflection.options[:source_type]` in `PolymorphicReflection` is al... | Ryuta Kamizono | 2017-08-28 | 1 | -10/+2 | |
* | | | | | | | | | | | | Remove unused `RuntimeReflection#alias_candidate` | Ryuta Kamizono | 2017-08-28 | 1 | -4/+0 | |
* | | | | | | | | | | | | Should be appear deprecation message for every call (#29649) | Ryuta Kamizono | 2017-08-27 | 1 | -8/+0 | |
* | | | | | | | | | | | | Omit the default limit for float columns (#28041) | Ryuta Kamizono | 2017-08-27 | 1 | -1/+1 | |
* | | | | | | | | | | | | Prefer to place a table options before `force: :cascade` (#28005) | Ryuta Kamizono | 2017-08-27 | 1 | -2/+1 | |
|/ / / / / / / / / / / | ||||||
* | | | | | | | | | | | Use tt in doc for ActiveRecord [ci skip] | Yoshiyuki Hirano | 2017-08-27 | 6 | -20/+20 |