aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | Reduce Array assignment by not giving a name for unused `*args`Akira Matsuda2019-07-314-6/+6
| |
* | Reduce Array allocationsAkira Matsuda2019-07-311-1/+1
| |
* | Reduce some more Hash#fetch + default object allocationsAkira Matsuda2019-07-312-3/+3
| |
* | Merge pull request #36708 from ↵Kasper Timm Hansen2019-07-315-12/+37
|\ \ | | | | | | | | | | | | rails/has-one-polymorphic-touch-dont-cache-association-result-inside-create-transaction Polymorphic has_one touch: Don't cache association result inside crea…
| * | Polymorphic has_one touch: Reset association cache result after create ↵Kasper Timm Hansen2019-07-315-12/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | transaction In case of a polymorphic association there's no automatic inverse_of to assign the inverse record. So to get the record there needs to be a query executed, however, if the query fires within the transaction that's trying to create the associated record, no record can be found. And worse, the nil result is cached on the association so after the transaction commits the record can't be found. That's what happens if touch is enabled on a polymorphic has_one association. Consider a Comment with a commentable association that needs to be touched. For `Comment.create(commentable: Post.new)`, the existing code essentially does `commentable.send(:comment)` within the create transaction for the comment and thus not finding the comment. Now we're purposefully clearing the cache in case we've tried accessing the association within the transaction and found no object. Before: ``` kaspth-imac 2.6.3 ~/code/rails/activerecord master *= ARCONN=postgresql bin/test test/cases/associations/has_one_associations_test.rb -n /commit/ Using postgresql Run options: -n /commit/ --seed 46022 D, [2019-07-19T03:30:37.864537 #96022] DEBUG -- : Chef Load (0.2ms) SELECT "chefs".* FROM "chefs" WHERE "chefs"."employable_id" = $1 AND "chefs"."employable_type" = $2 LIMIT $3 [["employable_id", 1], ["employable_type", "DrinkDesignerWithPolymorphicTouchChef"], ["LIMIT", 1]] D, [2019-07-19T03:30:37.865013 #96022] DEBUG -- : Chef Create (0.2ms) INSERT INTO "chefs" ("employable_id", "employable_type") VALUES ($1, $2) RETURNING "id" [["employable_id", 1], ["employable_type", "DrinkDesignerWithPolymorphicTouchChef"]] D, [2019-07-19T03:30:37.865201 #96022] DEBUG -- : TRANSACTION (0.1ms) RELEASE SAVEPOINT active_record_1 D, [2019-07-19T03:30:37.874136 #96022] DEBUG -- : TRANSACTION (0.1ms) ROLLBACK D, [2019-07-19T03:30:37.874323 #96022] DEBUG -- : TRANSACTION (0.1ms) ROLLBACK F Failure: HasOneAssociationsTest#test_polymorphic_has_one_with_touch_option_on_create_wont_cache_assocation_so_fetching_after_transaction_commit_works [/Users/kaspth/code/rails/activerecord/test/cases/associations/has_one_associations_test.rb:716]: --- expected +++ actual @@ -1 +1 @@ -#<Chef id: 1, employable_id: 1, employable_type: "DrinkDesignerWithPolymorphicTouchChef", department_id: nil, employable_list_type: nil, employable_list_id: nil> +nil ``` After: ``` kaspth-imac 2.6.3 ~/code/rails/activerecord master *= ARCONN=postgresql bin/test test/cases/associations/has_one_associations_test.rb -n /commit/ Using postgresql Run options: -n /commit/ --seed 46022 D, [2019-07-19T03:30:22.479387 #95973] DEBUG -- : Chef Create (0.3ms) INSERT INTO "chefs" ("employable_id", "employable_type") VALUES ($1, $2) RETURNING "id" [["employable_id", 1], ["employable_type", "DrinkDesignerWithPolymorphicTouchChef"]] D, [2019-07-19T03:30:22.479574 #95973] DEBUG -- : TRANSACTION (0.1ms) RELEASE SAVEPOINT active_record_1 D, [2019-07-19T03:30:22.482051 #95973] DEBUG -- : Chef Load (0.1ms) SELECT "chefs".* FROM "chefs" WHERE "chefs"."employable_id" = $1 AND "chefs"."employable_type" = $2 LIMIT $3 [["employable_id", 1], ["employable_type", "DrinkDesignerWithPolymorphicTouchChef"], ["LIMIT", 1]] D, [2019-07-19T03:30:22.482317 #95973] DEBUG -- : TRANSACTION (0.1ms) ROLLBACK D, [2019-07-19T03:30:22.482437 #95973] DEBUG -- : TRANSACTION (0.1ms) ROLLBACK . Finished in 0.088498s, 11.2997 runs/s, 22.5994 assertions/s. 1 runs, 2 assertions, 0 failures, 0 errors, 0 skips ``` Notice the select now fires after the commit.
* | | Merge pull request #36819 from yamato-payforward/payforward-branchRyuta Kamizono2019-07-311-1/+1
|\ \ \ | | | | | | | | fix a typo
| * | | fix a typoyamato-payforward2019-07-311-1/+1
|/ / /
* | | Merge pull request #36818 from hc0208/fix_typo_in_data_remote_jsRyuta Kamizono2019-07-311-1/+1
|\ \ \ | | | | | | | | Fix typo submited → submitted
| * | | Fix typo submited → submitted [ci skip]hc02082019-07-311-1/+1
| | | |
* | | | Accessing ivar with Symbols might be just a very little bit better than with ↵Akira Matsuda2019-07-314-8/+8
| | | | | | | | | | | | | | | | fstrings
* | | | Avoid creating new Array when looking up already registered detailAkira Matsuda2019-07-311-1/+1
| | | |
* | | | Reduce Hash object creation when normalizing request envAkira Matsuda2019-07-311-2/+3
| | | |
* | | | Cache tags_text to avoid computing tags each time when loggingAkira Matsuda2019-07-311-5/+10
| | | |
* | | | Reduce object allocations in Middleware::StaticAkira Matsuda2019-07-311-11/+12
| | | |
* | | | Reduce unnecessary String creation by not `to_s`ing until nothing matchesAkira Matsuda2019-07-311-1/+1
| | | |
* | | | Reduce String allocation when finding controller classAkira Matsuda2019-07-311-1/+1
| | | |
* | | | No need to dup the payload for an instrumentationAkira Matsuda2019-07-311-1/+1
| | | |
* | | | Speedup and reduce Array creation when constantizing a non-namespaced stringAkira Matsuda2019-07-311-25/+29
|/ / /
* | | Merge pull request #36815 from emp823/masterRyuta Kamizono2019-07-311-1/+1
|\ \ \ | | | | | | | | Fix typo in autoload documentation [ci skip]
| * | | Fix typo in autoload documentation [ci skip]Erik Pearson2019-07-301-1/+1
|/ / /
* | | Bump Trix to ^1.2.0Javan Makhmali2019-07-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an attachment button to the Trix toolbar that improves overall file upload usability, especially on mobile devices where files can't be dragged / dropped. References: - https://github.com/basecamp/trix/releases/tag/1.2.0 - https://github.com/basecamp/trix/pull/619 - https://github.com/basecamp/trix/issues/582
* | | Merge pull request #36813 from haruyuki97/haruyuki97/fix-comment-in-url-helperPrem Sichanugrist2019-07-301-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Fix a/an usage on `phone_to` documentation. [ci skip]
| * | | fix a typo [ci skip]haruyuki972019-07-301-1/+1
| | | |
* | | | Merge pull request #36812 from nigh7m4r3/readme-comma-usagePrem Sichanugrist2019-07-301-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix comma usage on project's README.md [ci skip]
| * | | | Update README.mdRifatul Islam Chayon2019-07-301-2/+2
|/ / / / | | | | | | | | A very minor change of comma usage.
* | | | Merge pull request #36805 from ↵Ryuta Kamizono2019-07-303-24/+65
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | kamipo/user_supplied_joins_order_should_be_preserved Preserve user supplied joins order as much as possible
| * | | | Preserve user supplied joins order as much as possibleRyuta Kamizono2019-07-303-24/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, string joins are always applied as last joins part, and Arel join nodes are always applied as leading joins part (since #36304), it makes people struggled to preserve user supplied joins order. To mitigate this problem, preserve the order of string joins and Arel join nodes either before or after of association joins. Fixes #36761. Fixes #34328. Fixes #24281. Fixes #12953.
* | | | | Add `silence_warnings` for defining 'not_' prefix enum elementsRyuta Kamizono2019-07-301-1/+3
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To suppress the following warnings in tests. ``` ~/rails/activerecord/lib/active_record/scoping/named.rb:190: warning: method redefined; discarding old not_sent ~/rails/activerecord/lib/active_record/scoping/named.rb:190: warning: previous definition of not_sent was here ```
* | | | Merge pull request #36804 from vzvu3k6k/ttVipul A M2019-07-301-1/+1
|\ \ \ \ | | | | | | | | | | [ci skip] Fix unclosed tags in `Inflector` docs
| * | | | [ci skip] Fix unclosed tags in `Inflector` docsvzvu3k6k2019-07-301-1/+1
| | | | |
* | | | | Merge pull request #36782 from jhawthorn/move_database_exists_to_adapterJohn Hawthorn2019-07-298-18/+13
|\ \ \ \ \ | | | | | | | | | | | | Move DatabaseAlreadyExists detection to DB adapter
| * | | | | Move DatabaseAlreadyExists detection to DB adapterJohn Hawthorn2019-07-298-18/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously it was the responsibility of the database tasks to translate the invalid statement from creating a duplicate database into an ActiveRecord::Tasks::DatabaseAlreadyExists error. It's actually easier for us to do this detection inside of the adapter, where we already do a case statement on the return code to translate the error. This commit introduces ActiveRecord::DatabaseAlreadyExists, a subclass of StatementInvalid, and updates both AbstractMysqlAdapter and PostgresqlAdapter to return this more specific exception in that case. Because this is a subclass of the old exception, StatementInvalid, it should be backwards compatible with any code expecting that from create_database. This works for both create_database and exectute("CREATE DATABASE")
* | | | | | Expand CHANGELOG for #36800 [ci skip]Ryuta Kamizono2019-07-301-2/+2
| |/ / / / |/| | | |
* | | | | Call raise with parentheses like a normal method call with argumentsCarlos Antonio da Silva2019-07-291-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | Using `(raise FooError, "error")` is like forcing a "new scope" around the `raise` call, it's simpler to just wrap the `raise` arguments with parentheses just like any other method call would.
* | | | Merge pull request #36800 from jamespearson/matches_regex_mysqlRyuta Kamizono2019-07-303-0/+58
|\ \ \ \ | | | | | | | | | | Enabled matches_regex for MySql
| * | | | Enabled matches_regex for MySqlJames Pearson2019-07-293-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously matches_regex was only availble on PostgreSql, this will enable it for MySql Usage example: users = User.arel_table; users = User.arel_table; User.where(users[:email].matches_regexp('(.*)\@gmail.com')) Update activerecord/test/cases/arel/visitors/mysql_test.rb Co-Authored-By: Ryuta Kamizono <kamipo@gmail.com>
* | | | | Merge pull request #36198 from oneiros/actionmailer_ssl_optionRyuta Kamizono2019-07-292-2/+2
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | Documentation for ActionMailer's SMTP over SSL/TLS option [ci skip]
| * | | | Improve documentation of `:ssl/:tls` option [ci skip]David Roetzel2019-07-292-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing bullet point to make clear this is actually a separate option from `:openssl_verify_mode`. Add `:ssl/:tls`-option to guides as well [ci skip]
* | | | | Tiny documentation fix [ci skip]Robin Dupret2019-07-291-3/+3
| | | | |
* | | | | Merge pull request #36798 from meganemura/remove-redundant-empty-lineRyuta Kamizono2019-07-291-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Remove redundant empty line when we don't use system test
| * | | | | Remove redundant empty line when we don't use system testmeganemura2019-07-291-1/+1
|/ / / / /
* | | | | Use match? where we don't need MatchDataAkira Matsuda2019-07-2946-67/+67
| | | | |
* | | | | Let the generated initializers/backtrace_silencers.rb code use Regexp#match?Akira Matsuda2019-07-294-4/+4
| | | | |
* | | | | Revert "Use assert_match / assert_no_match for asserting match"Akira Matsuda2019-07-292-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit e9651deea4145f62224af56af027bfbb3e45e4cd. Now we're having both `=~` and `match?` for these objects, and it's nicer to have explicit tests for both of them
* | | | | Add AS::TimeZone#match?Akira Matsuda2019-07-295-3/+19
| | | | |
* | | | | Add AS::Multibyte::Chars#match?Akira Matsuda2019-07-292-1/+7
| | | | |
* | | | | Add Mime::Type#match? that doesn't create MatchDataAkira Matsuda2019-07-292-1/+13
| | | | |
* | | | | Suppress Ruby warning: :warning: non-nil $, will be deprecatedAkira Matsuda2019-07-291-2/+6
| | | | |
* | | | | Merge pull request #36031 from st0012/guard-35982Rafael França2019-07-281-1/+16
|\ \ \ \ \ | | | | | | | | | | | | Add test case to guard the query count for relation cache (for #35982)
| * | | | | Assert query counts in cache relation testst00122019-07-281-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | This is to guard the change in #35982
generated by cgit v1.2.3 (git 2.25.1) at 2024-06-28 10:41:04 +0000