aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #26943 from colleenmcguckin/testing_typoKasper Timm Hansen2016-10-311-2/+2
|\ | | | | fix typo in Rails Test Runner section
| * fix typo in Rails Test Runner sectioncolleenmcguckin2016-10-311-2/+2
|/
* Ensure duration parsing is consistent across DST changesAndrew White2016-10-313-2/+54
| | | | | | | | | | | | | | | | | | Previously `ActiveSupport::Duration.parse` used `Time.current` and `Time#advance` to calculate the number of seconds in the duration from an arbitrary collection of parts. However as `advance` tries to be consistent across DST boundaries this meant that either the duration was shorter or longer depending on the time of year. This was fixed by using an absolute reference point in UTC which isn't subject to DST transitions. An arbitrary date of Jan 1st, 2000 was chosen for no other reason that it seemed appropriate. Additionally, duration parsing should now be marginally faster as we are no longer creating instances of `ActiveSupport::TimeWithZone` every time we parse a duration string. Fixes #26941.
* Merge pull request #26838 from Cohen-Carlisle/activerecord-slice-array-argsGuillermo Iguaran2016-10-313-1/+15
|\ | | | | allow ActiveRecord::Core#slice to use array arg
| * allow ActiveRecord::Core#slice to use array argCohen Carlisle2016-10-313-1/+15
|/
* Merge pull request #23094 from yui-knk/fix_ar_test_commentArthur Nogueira Neves2016-10-311-5/+4
|\ | | | | Fix the order of `create_table` to match a comment about `:inverse_of…
| * Remove a confusing commentyui-knk2016-10-311-1/+0
| | | | | | | | | | These comment sometimes explain a face which does not match the face.
| * Fix the order of `create_table` to match a comment about `:inverse_of` optionsyui-knk2016-10-311-4/+4
| | | | | | | | | | In ActiveRecord test :men, :faces, :interests and :zines tables are used for `:inverse_of` test cases, not `:wheels`.
* | Merge pull request #26938 from riseshia/patch-1प्रथमेश Sonpatki2016-10-311-1/+0
|\ \ | | | | | | Remove `Rack::Runtime from console log [ci skip]
| * | Remove `Rack::Runtime from console log [ci skip]Shia2016-10-311-1/+0
|/ /
* | Merge pull request #26935 from y-yagi/fix_ruby_warningAndrew White2016-10-311-1/+1
|\ \ | | | | | | remove warning from big integer test
| * | remove warning from big integer testyuuji.yaginuma2016-10-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This removes the following warnings. ``` activemodel/test/cases/type/big_integer_test.rb:15: warning: ambiguous first argument; put parentheses or a space even after `-' operator ```
* | | Merge pull request #26936 from caramelcat/add-missing-spaceGuillermo Iguaran2016-10-301-1/+1
|\ \ \ | |_|/ |/| | Add missing space in Migrations guide [ci skip]
| * | Add missing space in Migrations guide [ci skip]Benny2016-10-311-1/+1
|/ /
* | Merge pull request #26932 from y-yagi/update_doc_about_assets_digestJon Moss2016-10-291-2/+2
|\ \ | | | | | | update doc about assets digest class [ci skip]
| * | update doc about assets digest class [ci skip]yuuji.yaginuma2016-10-301-2/+2
| |/ | | | | | | | | The default digest class changed to SHA256 in sprockets 3.0. Ref: https://github.com/sstephenson/sprockets/pull/647
* | Add more rubocop rules about whitespacesRafael Mendonça França2016-10-29380-2077/+2098
| |
* | Merge pull request #26926 from gsamokovarov/debug-exceptions-plain-loggersRafael França2016-10-292-1/+20
|\ \ | | | | | | Support plain loggers in DebugExceptions
| * | Support plain loggers in DebugExceptionsGenadi Samokovarov2016-10-282-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I have been seeing people setting `Logger` instances for `config.logger` and it blowing up on `rails/web-console` usage. Now, I doubt many folks are manually setting `ActionView::Base.logger`, but given that `DebugExceptions` is running in a pretty fragile environment already, having it crash (and being silent) in those cases can be pretty tricky to trace down. I'm proposing we verify whether the `ActionView::Base.logger` supports silencing before we try to do it, to save us the headache of tracing it down.
* | | Revert "Merge pull request #23698 from ↵Rafael Mendonça França2016-10-281-1/+11
| |/ |/| | | | | | | | | | | | | | | meinac/add_missing_types_into_ar_model_generator_usage" This reverts commit aaf561d26cf9a879ff40190b625155015c6225da, reversing changes made to 83776676e54b387fc52081ff31cfd7710bae9e03. Reason: It is not a good idea to connect to the database on generators
* | Fix the variable scoping issue I introduced in ↵Matthew Draper2016-10-281-2/+2
| | | | | | | | 007e50d8e5a900547471b6c4ec79d9d217682c5d
* | Merge PR #19759Arthur Neves2016-10-283-1/+24
|\ \ | | | | | | | | | Fix for has_and_belongs_to_many & has_many_through associations
| * | Fix for has_and_belongs_to_many & has_many_through associations while ↵Mehmet Emin İNAÇ2016-02-133-2/+24
| | | | | | | | | | | | | | | | | | | | | | | | partial_writes is false This will fix #19663 Also with this fix, active record does not fire unnecassary update queries while partial_writes is true
* | | Merge pull request #23698 from ↵Arthur Nogueira Neves2016-10-281-11/+1
|\ \ \ | |_|/ |/| | | | | | | | meinac/add_missing_types_into_ar_model_generator_usage Dynamically show available sql types [ci skip]
| * | Dynamically show available sql types [ci skip]Mehmet Emin İNAÇ2016-02-161-11/+1
| | | | | | | | | | | | Dynamically list available sql data types based on current database adapter.
* | | Hide commands from API site.Kasper Timm Hansen2016-10-2813-17/+17
| | | | | | | | | | | | | | | They're just barren on the site and confure more than guide, instead rely on the built in --help to guide users.
* | | Merge pull request #26688 from kamipo/remove_respond_to_indexesKasper Timm Hansen2016-10-282-18/+9
|\ \ \ | | | | | | | | Remove unnecessary `respond_to?(:indexes)` checking
| * | | Remove unnecessary `respond_to?(:indexes)` checkingRyuta Kamizono2016-10-032-18/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently all adapters (postgresql, mysql2, sqlite3, oracle-enhanced, and sqlserver) implemented `indexes` and schema dumper expects implemented `indexes`. https://github.com/rails/rails/blob/v5.0.0/activerecord/lib/active_record/schema_dumper.rb#L208 Therefore `respond_to?(:indexes)` checking is unnecessary.
* | | | [ci skip] Revise TagHelper only permitting symbols doc.Kasper Timm Hansen2016-10-281-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up to 454460e. Rebuild the sentence so the bit about only symbols allowed comes less out of left field and fits in better with the existing doc. Also remove the `(Legacy syntax)` construct. The sections are properly nested under `=== Options` with `====` and look fine on the generated API doc site.
* | | | Merge pull request #26919 from Erol/add-assets-version-optionKasper Timm Hansen2016-10-281-0/+2
|\ \ \ \ | | | | | | | | | | Add documentation for `config.assets.version`
| * | | | Add documentation for `config.assets.version` [skip ci]Erol Fornoles2016-10-281-0/+2
|/ / / /
* | | | Explicitly show --no-helper and --no-assets options in help messageAkira Matsuda2016-10-281-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | I'm sorry for causing #24168. I wasn't aware of --no-helper and --no-assets. So I'm adding them to the help message. I'm still not sure how to show `--no-test-framework` though.
* | | | Merge pull request #26917 from kamipo/fix_stopdoc_to_startdocRafael França2016-10-281-1/+1
|\ \ \ \ | | | | | | | | | | Fix :stopdoc: to :startdoc: [ci skip]
| * | | | Fix :stopdoc: to :startdoc: [ci skip]Ryuta Kamizono2016-10-281-1/+1
| | | | |
* | | | | Merge pull request #23770 from meinac/fix_habtm_symbol_class_nameArthur Nogueira Neves2016-10-272-2/+14
|\ \ \ \ \ | |/ / / / |/| | | | Fix HABTM associations join table resolver bug on constants and symbols
| * | | | Fix HABTM associations join table resolver bug on constants and symbolsMehmet Emin İNAÇ2016-10-272-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using Constant and symbol class_name option for associations are valid but raises exception on HABTM associations. There was a test case which tries to cover symbol class_name usage but doesn't cover correctly. Fixed both symbol usage and constant usage as well. These are all working as expected now; ``` has_and_belongs_to_many :foos, class_name: 'Foo' has_and_belongs_to_many :foos, class_name: :Foo has_and_belongs_to_many :foos, class_name: Foo ``` Closes #23767
* | | | | Permit loads while queries are runningMatthew Draper2016-10-275-25/+47
| | | | | | | | | | | | | | | | | | | | | | | | | A query may wait on a database-level lock, which could lead to a deadlock between threads.
* | | | | Merge pull request #26915 from jules2689/time-zone-conversion-inherited-bugSean Griffin2016-10-271-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | Fixes an issue where time_zone_conversion that causes an exception in…
| * | | | Fixes an issue where time_zone_conversion that causes an exception in ARs ↵Julian Nadeau2016-10-271-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | delegation Following off of https://github.com/rails/rails/issues/15945, I realized that super needs to be the first thing that is called in an AbstractModel's inherited method. I was receiving errors within the inherited method of time_zone_conversion, so I tested locally by moving super to the top of the method declaration. All exceptions went away.
* | | | removes requires already present in active_support/railsXavier Noria2016-10-277-9/+0
| | | |
* | | | Merge pull request #26899 from kamipo/use_regex_matchXavier Noria2016-10-2710-13/+13
|\ \ \ \ | | | | | | | | | | Use Regexp#match? rather than Regexp#===
| * | | | Use Regexp#match? rather than Regexp#===Ryuta Kamizono2016-10-2610-13/+13
| | | | | | | | | | | | | | | | | | | | Follow up to 99cf7558000090668b137085bfe6bcc06c4571dc.
* | | | | let Regexp#match? be globally availableXavier Noria2016-10-2726-25/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Regexp#match? should be considered to be part of the Ruby core library. We are emulating it for < 2.4, but not having to require the extension is part of the illusion of the emulation.
* | | | | Merge pull request #26907 from y-yagi/remove_unused_RedirectBackErrorArthur Nogueira Neves2016-10-261-8/+0
|\ \ \ \ \ | | | | | | | | | | | | remove unused `RedirectBackError` class
| * | | | | remove unused `RedirectBackError` classyuuji.yaginuma2016-10-271-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | Follow up to 333bfd896e87862cece95deb1ef88132d5f54ba8
* | | | | | Merge pull request #26906 from y-yagi/fix_initializer_nameJon Moss2016-10-261-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | fix initializer name [ci skip]
| * | | | | fix initializer name [ci skip]yuuji.yaginuma2016-10-271-1/+1
|/ / / / / | | | | | | | | | | | | | | | This changed in c046660
* | | | | Clear the correct query cacheSamuel Cochran2016-10-262-4/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This executor currently relies on `ActiveRecord::Base.connection` not changing between `prepare` and `complete`. If something else returns the current ActiveRecord connection to the pool early then this `complete` call will fail to clear the correct query cache and restore the original `query_cache_enabled` status. This has for example been happening in Sidekiq: https://github.com/mperham/sidekiq/pull/3166 We can just keep track of the connection as part of the exector state.
* | | | | Remove all Journey constant from public APIRafael Mendonça França2016-10-265-7/+17
|/ / / / | | | | | | | | | | | | | | | | | | | | There were never public API only there by mistake. [ci skip]
* | | | Merge pull request #26896 from j-dexx/update_habtm_docsVipul A M2016-10-261-3/+1
|\ \ \ \ | | | | | | | | | | Update HABTM documentation in guides [ci skip]
generated by cgit v1.2.3 (git 2.25.1) at 2024-12-28 04:48:17 +0000