aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #19521 from mudphone/test_fixture_docsArthur Nogueira Neves2015-03-251-2/+2
|\ | | | | [skip ci] Minor edits for clarity in the test fixture guide.
| * [skip ci] Minor edits for clarity, to make smooth, & separate clauses.Kyle Oba2015-03-251-2/+2
|/ | | | | | * Add conjunction for clarity. * Add comma between independent clauses: create command, and generation of fixtures.
* Merge pull request #19496 from andreynering/docs-migrations-changeKasper Timm Hansen2015-03-251-0/+8
|\ | | | | Documenting remove_column as a reversible migration method
| * Documenting remove_column as a reversible migration method [ci skip]Andrey Nering2015-03-251-0/+8
|/
* Merge pull request #19503 from jasoncodes/no-psqlrcYves Senn2015-03-253-3/+7
|\ | | | | | | Avoid loading user's psqlrc when loading test structure
| * Avoid loading user's psqlrc when loading test structureJason Weathered2015-03-251-1/+1
| |
* | Merge pull request #19507 from jonatack/ar-validations-guide-fixesKasper Timm Hansen2015-03-251-3/+5
|\ \ | | | | | | [skip ci] Active Record Validations guide fixes
| * | [ci skip] Active Record Validations guide fixesJon Atack2015-03-251-3/+5
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove deprecated `[]=` - Fix duplicate `errors#add` example. The second code example was originally `[]=`, replace it with `[] <<`. - Improve explanations for `errors#add` and `errors#full_messages` Follow-up to PR #19457 (closed after borking my git history). Apologies for the duplicate PR. cc @kaspth
* | Merge pull request #19504 from y-yagi/fix_digest_explainClaudio B.2015-03-241-2/+1
|\ \ | |/ |/| update `config.assets.digest` default value in guides [ci skip]
| * update `config.assets.digest` default value in guides [ci skip]yuuji.yaginuma2015-03-251-2/+1
|/
* Merge pull request #19502 from kaspth/assert-select-failure-outputRafael Mendonça França2015-03-241-0/+24
|\ | | | | [ci skip] Update 4.2 Release notes with more assert_select information.
| * [ci skip] Update 4.2 Release notes with more assert_select information.Kasper Timm Hansen2015-03-241-0/+24
| | | | | | | | | | Adds documentation for substitutions and why Regexp substitions look different when assertion fails.
* | Merge pull request #19499 from swalkinshaw/patch-1Arthur Nogueira Neves2015-03-241-7/+2
|\ \ | |/ |/| Update ActiveSupport::Subscriber docs
| * Update ActiveSupport::Subscriber docsScott Walkinshaw2015-03-241-7/+2
| | | | | | For consistency purposes with the changes done in https://github.com/rails/rails/pull/12285
* | Add changelog entry for #19271Arthur Neves2015-03-241-0/+7
| |
* | Revert "Merge pull request #19500 from ccutrer/dry_sti_subclass_finding"Rafael Mendonça França2015-03-241-19/+19
| | | | | | | | | | | | | | This reverts commit 5cfa6a8ab997089c3012a82052c8c317b2e095f5, reversing changes made to bfd5bf8313e6ea0bb2eccb68ee5076bb63f0b2db. Reason: This broken travis build.
* | Merge pull request #19500 from ccutrer/dry_sti_subclass_findingRafael Mendonça França2015-03-241-19/+19
|\ \ | |/ |/| DRY up STI subclass logic
| * DRY up STI subclass logicCody Cutrer2015-03-241-19/+19
|/ | | | | | | | | the newer method used for discriminating new records did not use the older and more robust method used for instantiating existing records, but did have a better post-check to ensure the sublass was in the hierarchy. so move the descendants check to find_sti_class, and then simply call find_sti_class from subclass_from_attributes
* Merge pull request #19452 from ↵Carlos Antonio da Silva2015-03-243-1/+19
|\ | | | | | | | | | | pinglamb/fix-referencing-wrong-alias-when-joining-tables-of-has-many-through-association Fix referencing wrong aliases while joining tables of has many through association
| * Fix referencing wrong aliases while joining tables of has many throughpinglamb2015-03-223-1/+19
| | | | | | | | | | | | | | | | | | | | | | association While joining table of has_many :through association, ActiveRecord will use the actual table name instead of through-join alias. It results with a wrong SQL and exception is raised. This only happens when calculation methods like #count is called. This issue is affecting Rails 4.1.x and 4.2.x as well.
* | Merge pull request #19493 from larskanis/add_infinity_testYves Senn2015-03-241-0/+9
|\ \ | | | | | | PostgreSQL, Add test case for "Infinity" string assignment to float columns
| * | PostgreSQL, Add test case for "Infinity" string assignment to float columns.Lars Kanis2015-03-221-0/+9
| |/ | | | | | | This is implemented in Type::Float, but not tested, so far.
* | Merge pull request #19488 from yui-knk/fix/comment2Yves Senn2015-03-241-1/+1
|\ \ | | | | | | [ci skip] Remove unacceptable method name
| * | [ci skip] Remove unacceptable method nameyui-knk2015-03-241-1/+1
| | |
* | | Merge pull request #19491 from tgxworld/fix_incorrect_descriptionYves Senn2015-03-241-1/+1
|\ \ \ | |/ / |/| | Fix incorrect description for `assert_nothing_raised`. [ci skip]
| * | Fix incorrect description for `assert_nothing_raised`.Guo Xiang Tan2015-03-241-1/+1
|/ /
* | Merge pull request #19485 from tgxworld/small_doc_fixRafael Mendonça França2015-03-241-1/+1
|\ \ | | | | | | Small doc fix. [CI SKIP]
| * | Small doc fix. [CI SKIP]Guo Xiang Tan2015-03-241-1/+1
| | |
* | | Merge pull request #19481 from davidcornu/activesupport/normalize-to-sentenceArthur Nogueira Neves2015-03-232-1/+7
|\ \ \ | | | | | | | | Make sure Array#to_sentence always returns a String
| * | | Make sure Array#to_sentence always returns a StringDavid Cornu2015-03-232-1/+7
| |/ /
* | | Merge pull request #19482 from mperham/masterRafael Mendonça França2015-03-232-7/+22
|\ \ \ | |/ / |/| | Pass wrapped class name to Sidekiq for logging purposes
| * | Pass wrapped class name to Sidekiq for logging purposesMike Perham2015-03-232-7/+22
|/ / | | | | | | | | | | | | Sidekiq logs the name of the job class being performed. Because ActiveJob wraps the class, this means every job logs as an AJ::JobWrapper instead of the actual class name. Will help fix mperham/sidekiq#2248
* | Merge pull request #19475 from tgxworld/no_one_has_to_suck_anymoreRafael Mendonça França2015-03-232-13/+0
|\ \ | | | | | | Remove alias for `i_suck_and_my_tests_are_order_dependent`.
| * | Remove alias for `i_suck_and_my_tests_are_order_dependent`.Guo Xiang Tan2015-03-242-13/+0
|/ /
* | Fix failing tests for #19474Andrew White2015-03-231-2/+2
| |
* | Fix ActionDispatch::PublicExceptions returning string rack statusRyan Tomayko2015-03-231-2/+2
| | | | | | | | | | | | | | | | The status returned in the rack [status, headers, body] array was a string, which can cause problems with middleware that assumes the status will be a Fixnum. This likely never surfaced because other middleware to_i the status returned from downstream apps before passing it on.
* | Merge pull request #19473 from nerdinand/duration-to_i-documentationYves Senn2015-03-231-0/+24
|\ \ | | | | | | Add documentation for Duration#to_i for clarification
| * | Add documentation for Duration#to_i for clarificationnerdinand2015-03-231-0/+24
|/ /
* | Merge pull request #19471 from ↵Yves Senn2015-03-232-30/+0
|\ \ | | | | | | | | | | | | nerdinand/remove-activesupport-integer-example-doc Remove reference to Numeric#from_now, as it is no longer supported [ci skip]
| * | Remove reference to Numeric#from_now, as it is no longer supportednerdinand2015-03-232-30/+0
|/ /
* | Merge pull request #19449 from Gaurav2728/gaurav-unavailable_linkRafael Mendonça França2015-03-234-7/+4
|\ \ | | | | | | remove old unavailable link with relevant fix patch 1
| * | remove old unavailable link with relevant fixGaurav Sharma2015-03-234-7/+4
| | |
* | | Merge pull request #19469 from Gaurav2728/gaurav-mysql_missing_linksRafael Mendonça França2015-03-231-1/+1
|\ \ \ | |/ / |/| | update mysql link that has been replaced
| * | update mysql link that has been replacedGaurav Sharma2015-03-231-1/+1
|/ /
* | Provide a more truthful #inspectMatthew Draper2015-03-231-0/+4
| | | | | | | | | | | | This is obviously all very internal, but sometimes you have to look at it... and when you do, it'll save a lot of confusion if it doesn't lie about its identity.
* | Make sure to persist a newly-nil serialized valueMatthew Draper2015-03-233-1/+20
| | | | | | | | | | | | | | The subtype will (quite reasonably) ignore the possibility that it has `changed_in_place?` by becoming nil. Fixes #19467
* | Simplify setting button form optionsCarlos Antonio da Silva2015-03-221-2/+3
| | | | | | | | No need to merge hashes when simply setting options does the job.
* | Fix arguments order on assertionCarlos Antonio da Silva2015-03-221-1/+1
| | | | | | | | The expected value comes first. Related to #19465.
* | Merge pull request #19465 from radanskoric/fix-errors-deleteRafael Mendonça França2015-03-222-7/+13
|\ \ | | | | | | Fix ActiveModel::Errors#delete return value to stay backward compatible
| * | Fix ActiveModel::Errors#delete return value to stay backward compatibleRadan Skoric2015-03-222-7/+13
|/ / | | | | | | | | | | | | | | | | | | | | | | Rails 5.0 changes to ActiveModel::Errors include addition of `details` that also accidentally changed the return value of `delete`. Since there was no test for that behavior it went unnoticed. This commit adds a test and fixes the regression. Small improvements to comments have also been made. Since `get` is getting deprecated it is better to use `[]` in other methods' code examples. Also, in the module usage example, `def Person.method` was replaced with a more commonly used `def self.method` code style.