aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | Merge pull request #29182 from kamipo/remove_unused_left_joins_values_generationMatthew Draper2017-05-242-2/+2
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove unused `left_joins_values` generation
| | * | | | | | | | Remove unused `left_joins_values` generationRyuta Kamizono2017-05-232-2/+2
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | This was added at #22125 but `left_joins_values` is never used.
| * | | | | | | | Merge pull request #29183 from kamipo/refactor_making_join_constraintsMatthew Draper2017-05-241-23/+6
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Refactor making join constraints
| | * | | | | | | | Refactor making join constraintsRyuta Kamizono2017-05-231-23/+6
| | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only difference between `make_inner_joins` and `make_left_outer_joins` is the `join_type`.
| * / / / / / / / More friendly exception in nested attributesKir Shatrov2017-05-222-2/+2
| |/ / / / / / /
| * | | | | | | Merge pull request #29156 from kamipo/make_valid_directions_to_setAndrew White2017-05-201-3/+5
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Make `VALID_DIRECTIONS` to `Set`
| | * | | | | | | Make `VALID_DIRECTIONS` to `Set`Ryuta Kamizono2017-05-201-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ```ruby require "benchmark/ips" require "set" array = [:asc, :desc, :ASC, :DESC, "asc", "desc", "ASC", "DESC"] set = array.to_set item = "DESC" Benchmark.ips do |x| x.report "array" do array.include?(item) end x.report "set" do set.include?(item) end end ``` ``` % ruby array_vs_set.rb Warming up -------------------------------------- array 188.441k i/100ms set 229.531k i/100ms Calculating ------------------------------------- array 3.508M (± 9.0%) i/s - 17.525M in 5.043058s set 5.134M (± 7.6%) i/s - 25.707M in 5.038921s ```
| * | | | | | | | Fix `warning: assigned but unused variable - timestamp`Ryuta Kamizono2017-05-201-1/+1
| |/ / / / / / /
| * | / / / / / Add cache_key_with_version and use it in ActiveSupport::Cache.expand_cache_keyDavid Heinemeier Hansson2017-05-193-24/+45
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This retains the existing behavior of ActiveSupport::Cache.expand_cache_key (as used by etaging) where the cache key includes the version.
| * | | | | | Cleanup CHANGELOGs [ci skip]Ryuta Kamizono2017-05-191-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix indentation. * Add backticks.
| * | | | | | Use recyclable cache keys (#29092)David Heinemeier Hansson2017-05-184-11/+117
| | | | | | |
| * | | | | | Merge pull request #29118 from kamipo/remove_respond_to_reset_scopeRafael França2017-05-171-1/+1
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove unneeded `association.respond_to?(:reset_scope)`
| | * | | | | | Remove unneeded `association.respond_to?(:reset_scope)`Ryuta Kamizono2017-05-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since 86390c3 all associations have `reset_scope` so `respond_to?` is unneeded.
| * | | | | | | Set non 0 value to logger not to be affected by Ruby versionsYasuo Honda2017-05-171-23/+23
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Addresses #29021
| * | | | | | Fix association documentationRafael Mendonça França2017-05-161-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The object inside the scope is of the class that define the association not the associated class. [ci skip]
| * | | | | | Changelog entry [ci-skip]Guillermo Iguaran2017-05-151-0/+4
| | | | | | |
| * | | | | | Improvements for SQLite rake task.Guillermo Iguaran2017-05-151-7/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Use NOT IN in SQL query * Quote table names propertly * Use array form of command invocation
| * | | | | | Update SchemaDumper.ignore_tables docsGuillermo Iguaran2017-05-151-2/+2
| | | | | | |
| * | | | | | Respect 'ignore_tables' in SQLite structure dumpGuillermo Iguaran2017-05-152-1/+30
| | | | | | |
| * | | | | | Respect 'ignore_tables' in MySQL structure dumpGuillermo Iguaran2017-05-152-0/+22
| | | | | | |
| * | | | | | Respect `ignore_tables` in Postgres structure dumpRusty Geldmacher2017-05-152-0/+14
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | When using `sql` as the schema format, or even just doing `rake db:structure:dump`, it would be good to respect the list of ignored tables that has been configured.
| * | | | | Merge pull request #28919 from ↵Kasper Timm Hansen2017-05-121-1/+1
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | meinac/fix_ambigious_exception_message_of_select_query_method Fix ambigious error message of select query method [ci skip]
| | * | | | | Fix ambigious error message of select query methodMehmet Emin INAC2017-05-081-1/+1
| | | | | | |
| * | | | | | Merge pull request #29043 from kamipo/dont_eager_loading_if_unneeded_for_existsAaron Patterson2017-05-113-4/+6
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | Don't eager loading if unneeded for `FinderMethods#exists?`
| | * | | | | | Don't eager loading if unneeded for `FinderMethods#exists?`Ryuta Kamizono2017-05-113-4/+6
| | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | Fixes #29025.
| * / | | | | Defer loading each DB Tasks class from AR DatabaseTasksAkira Matsuda2017-05-111-6/+6
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because we don't need to load tasks for DBs that we don't use for the current app. Also, these Tasks classes load AR::Base in their class level, and so immediately kick :active_record on_load hooks. This used to happen when we were loading tasks, not when we run a task.
| * | | / / Should escape meta characters in regexpRyuta Kamizono2017-05-0710-26/+26
| | |_|/ / | |/| | |
| * | | | Fix `warning: ambiguous first argument`Ryuta Kamizono2017-05-061-2/+2
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` % ARCONN=sqlite3 be ruby -w -Itest test/cases/quoting_test.rb test/cases/quoting_test.rb:92: warning: ambiguous first argument; put parentheses or a space even after `/' operator test/cases/quoting_test.rb:96: warning: ambiguous first argument; put parentheses or a space even after `/' operator Using sqlite3 Run options: --seed 9495 ..................................... Finished in 0.046403s, 797.3622 runs/s, 1120.6172 assertions/s. 37 runs, 52 assertions, 0 failures, 0 errors, 0 skips ```
| * | | Merge pull request #28989 from matthewd/quoted_id-deprecationMatthew Draper2017-05-062-2/+22
| |\ \ \ | | | | | | | | | | Clarify deprecation message for #quoted_id
| | * | | Clarify deprecation message for #quoted_idMatthew Draper2017-05-052-2/+22
| | | | | | | | | | | | | | | | | | | | | | | | | In this case, it's the method definition that's more at fault, rather than the current caller.
| * | | | Remove unused `DelegationTest#call_method`Ryuta Kamizono2017-05-061-27/+7
| | | | | | | | | | | | | | | | | | | | `DelegationTest#call_method` is no longer used since 9d79334a.
| * | | | Don't pass `arel.engine` to `Arel::SelectManager.new`Ryuta Kamizono2017-05-053-9/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The argument of `Arel::SelectManager.new` is `table`, not `engine`. https://github.com/rails/arel/blob/v8.0.0/lib/arel/select_manager.rb#L10
| * | | | Remove useless `target_records_from_association`Ryuta Kamizono2017-05-041-11/+3
| |/ / / | | | | | | | | | | | | Since through association is always loaded by `preloader.preload`.
| * | | Should test against `Relation#bound_attributes`Ryuta Kamizono2017-05-041-8/+12
| | | | | | | | | | | | | | | | Since legacy `Relation#bind_values` was removed in b06f64c.
| * | | Merge pull request #28974 from kamipo/extract_bind_param_and_bind_attributeRafael França2017-05-0311-51/+38
| |\ \ \ | | | | | | | | | | Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`
| | * | | Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`Ryuta Kamizono2017-05-0411-51/+38
| | | | | | | | | | | | | | | | | | | | These are used in tests from anywhere.
| * | | | Remove duplicated `test_merging_reorders_bind_params`Ryuta Kamizono2017-05-041-9/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | The same test exists in `test/cases/relation/merging_test.rb`. https://github.com/rails/rails/blob/v5.1.0/activerecord/test/cases/relation/merging_test.rb#L94-L101
| * | | Add type caster to `RuntimeReflection#alias_name`Jon Moss2017-05-025-3/+13
| | | | | | | | | | | | | | | | | | | | Since we have been using this `Arel::Table` since 111ccc832bc977b15af12c14e7ca078dad2d4373, in order to properly handle queries, it's important that we properly type cast arguments.
| * | | Merge pull request #28938 from kamipo/deprecate_supports_statement_cacheRafael França2017-05-016-22/+12
| |\ \ \ | | | | | | | | | | Deprecate `supports_statement_cache?`
| | * | | Deprecate `supports_statement_cache?`Ryuta Kamizono2017-05-016-22/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `supports_statement_cache?` was introduced in 3.1.0.beta1 (104d0b2) for bind parameter substitution, but it is no longer used in 3.1.0.rc1 (73ff679). Originally it should respect `prepared_statements` rather than `supports_statement_cache?` (fd39847). One more thing, named `supports_statement_cache?` is pretty misreading. We have `StatementCache` and `StatementPool`. However, `supports_statement_cache?` doesn't mean `StatementCache`, but `StatementPool` unlike its name. https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/statement_cache.rb https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/connection_adapters/statement_pool.rb
| * | | | Merge pull request #28930 from kamipo/use_flat_mapRafael França2017-05-011-1/+1
| |\ \ \ \ | | |/ / / | |/| | | Use `flat_map` rather than `map(&:...).flatten`
| | * | | Use `flat_map` rather than `map(&:...).flatten`Ryuta Kamizono2017-04-291-1/+1
| | | | |
| * | | | Cleanup CHANGELOGs [ci skip]Ryuta Kamizono2017-04-301-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | * Remove trailing spaces. * Add backticks around method and command. * Fix indentation.
| * | | Stop creating duplicate Struct instancesJon Moss2017-04-281-23/+15
| | | | | | | | | | | | | | | | | | | | | | | | Just use one `Event` class. Reduces duplication, makes the tests easier to read. It might seem like each tests needs a different kind of Struct, since we make a new one for each test case.
| * | | Also raise error when VERSION is nilRafael Mendonça França2017-04-272-3/+4
| | | | | | | | | | | | | | | | Fix #28905
| * | | Evaluate belongs_to :default option against the owner, not the associationGeorge Claghorn2017-04-272-1/+19
| |/ /
| * | Restore `fixtures :author_addresses`Ryuta Kamizono2017-04-2721-26/+26
| | | | | | | | | | | | | | | This change reverted in eac6f369 but it is needed for data integrity. See #25328.
* | | Explicitly create necessary data for testyuuji.yaginuma2017-05-131-0/+4
| | | | | | | | | | | | | | | | | | `DefaultScopingWithThreadTest` expects that there are two or more of `developers` data, but have not created data in the test. Therefore, tests may fail depending on execution order.
* | | Run ReloadModelTest in a different proccessRafael Mendonça França2017-04-271-1/+3
| | | | | | | | | | | | | | | This will make sure it doesn't change the state of the current proccess when removing the owners constant.
* | | Only clean the connection of the current connection poolRafael Mendonça França2017-04-261-1/+1
| | | | | | | | | | | | This will avoid us to close the connection of the saved connection pool.
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-24 18:39:34 +0000