Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove extra indirection for testing exceptions and messages | Carlos Antonio da Silva | 2014-03-20 | 1 | -28/+12 |
| | | | | | | assert_raise + assert_equal on the returned exception message work just fine, there is no need for extra work, specially like this overly complicated helper. | ||||
* | Improve touch docs with extra attributes passed in [ci skip] | Carlos Antonio da Silva | 2014-03-20 | 2 | -11/+10 |
| | |||||
* | Merge pull request #14423 from yakko/persistence-touches-many | Carlos Antonio da Silva | 2014-03-20 | 3 | -2/+27 |
|\ | | | | | | | | | | | | | ActiveRecord#touch should accept multiple attributes Conflicts: activerecord/CHANGELOG.md | ||||
| * | ActiveRecord#touch should accept multiple attributes #14423 | Thiago Pinto | 2014-03-19 | 3 | -2/+27 |
| | | |||||
* | | only dump schema information if migration table exists. Closes #14217 | Yves Senn | 2014-03-20 | 2 | -1/+9 |
|/ | |||||
* | Add counter cache test for class_name | Arthur Neves | 2014-03-17 | 1 | -0/+11 |
| | | | | Backport test from #14410 | ||||
* | test should only pass if the pool.size+1 checkout fails | Aaron Patterson | 2014-03-17 | 1 | -3/+2 |
| | | | | | Previously, any of the connection checkouts could have failed, and this test would pass. | ||||
* | use a latch to avoid busy loops | Aaron Patterson | 2014-03-17 | 1 | -3/+4 |
| | |||||
* | Reap connections based on owning-thread death | Matthew Draper | 2014-03-18 | 7 | -68/+66 |
| | | | | | | | | | | | | | | | | .. not a general timeout. Now, if a thread checks out a connection then dies, we can immediately recover that connection and re-use it. This should alleviate the pool exhaustion discussed in #12867. More importantly, it entirely avoids the potential issues of the reaper attempting to check whether connections are still active: as long as the owning thread is alive, the connection is its business alone. As a no-op reap is now trivial (only entails checking a thread status per connection), we can also perform one in-line any time we decide to sleep for a connection. | ||||
* | Teach PostgreSQLAdapter#reset! to actually reset | Matthew Draper | 2014-03-18 | 2 | -1/+37 |
| | | | | It wasn't doing anything beyond clearing the statement cache. | ||||
* | `where.not` adds `references` for `includes`. | Yves Senn | 2014-03-17 | 4 | -0/+26 |
| | | | | Closes #14406. | ||||
* | format ActiveRecord CHANGELOG. [ci skip] | Yves Senn | 2014-03-17 | 1 | -8/+9 |
| | |||||
* | Extend fixture label replacement to allow string interpolation | Eric Steele | 2014-03-15 | 4 | -1/+19 |
| | | | | | | | | | | | | | | Allows fixtures to use their $LABEL as part of a string instead of limiting use to the entire value. mark: first_name: $LABEL username: $LABEL1973 email: $LABEL@$LABELmail.com users(:mark).first_name # => mark users(:mark).username # => mark1973 users(:mark).email # => mark@markmail.com | ||||
* | Merge pull request #14380 from ↵ | Yves Senn | 2014-03-15 | 49 | -61/+51 |
|\ | | | | | | | | | tgxworld/use_teardown_helper_method_in_activerecord Use teardown helper method. | ||||
| * | Use teardown helper method. | Guo Xiang Tan | 2014-03-14 | 49 | -61/+51 |
| | | | | | | | | | | | | | | | | Follow-Up to https://github.com/rails/rails/pull/14348 Ensure that SQLCounter.clear_log is called after each test. This is a step to prevent side effects when running tests. This will allow us to run them in random order. | ||||
* | | Obey `validate: false` option for habtm | Carlos Antonio da Silva | 2014-03-14 | 4 | -1/+39 |
| | | | | | | | | Fixes #14383. | ||||
* | | Merge pull request #14387 from huoxito/touch-api-doc | Arthur Nogueira Neves | 2014-03-14 | 1 | -2/+2 |
|\ \ | | | | | | | Update callbacks executed on AR::Base#touch [skip ci] | ||||
| * | | Update callbacks executed on AR::Base#touch [skip ci] | Washington Luiz | 2014-03-14 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | As of https://github.com/rails/rails/pull/12031 after_commit and after_rollback are also executed | ||||
* | | | Allow custom JDBC urls | schneems | 2014-03-14 | 2 | -2/+8 |
|/ / | | | | | | | mitigates #14323 | ||||
* / | Avoid duplicated conditionals | Carlos Antonio da Silva | 2014-03-14 | 1 | -2/+1 |
|/ | |||||
* | No need to binds be optional | Rafael Mendonça França | 2014-03-13 | 1 | -2/+2 |
| | |||||
* | Merge pull request #14372 from arthurnn/fix_query_cache_binds | Rafael Mendonça França | 2014-03-13 | 4 | -12/+24 |
|\ | | | | | Make select_all on query cache accept a Relation without binds. | ||||
| * | Make select_all on query cache accept a Relation without binds. | Arthur Neves | 2014-03-13 | 4 | -12/+24 |
| | | | | | | | | | | [fixes #14361] [related #13886] | ||||
* | | Add test for deprecation warning for passing an AR object to `find`. | Lauro Caetano | 2014-03-13 | 1 | -0/+6 |
| | | |||||
* | | Test deprecation warning for passing an ActiveRecord object to | Lauro Caetano | 2014-03-13 | 1 | -0/+6 |
|/ | | | | `exists?` | ||||
* | passing an instance of an AR object to `find` is deprecated | Aaron Patterson | 2014-03-13 | 5 | -6/+13 |
| | | | | please pass the id of the AR object by calling `.id` on the model first. | ||||
* | passing an ActiveRecord object to `exists?` is deprecated. | Aaron Patterson | 2014-03-13 | 5 | -4/+14 |
| | | | | | Pass the id of the object to the method by calling `.id` on the AR object. | ||||
* | make tests pass on Ruby 2.2 | Aaron Patterson | 2014-03-13 | 1 | -4/+4 |
| | | | | | | Apparently we've been using a buggy feature for the past 6 years: https://bugs.ruby-lang.org/issues/9593 | ||||
* | Use Sqlite3 adapter in examples | Julian Simioni | 2014-03-12 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | Two bits of example code use sqlite as an adapter, which doesn't exist. Using the code verbatim will raise a LoadError exception: ActiveRecord::Base.establish_connection( "adapter" => "sqlite", "database" => "db.sqlite" ) # => LoadError: Could not load 'active_record/connection_adapters/sqlite_adapter'... Considering this is code a lot of people new to Rails might be running, it's especially confusing. Closes #14367 [ci skip] | ||||
* | Merge pull request #14364 from why-el/better-docs-for-update-attribute | Arthur Nogueira Neves | 2014-03-12 | 1 | -0/+2 |
|\ | | | | | Enhance docs for update_attribute [skip ci] | ||||
| * | Enhance docs for update_attribute [ci-skip] | Mohamed Wael Khobalatte | 2014-03-12 | 1 | -0/+2 |
| | | |||||
* | | Change usec to 0 on tests that compare seconds | Arthur Neves | 2014-03-12 | 1 | -3/+3 |
| | | | | | | | | | | Avoid rounding problems with `.usec` method rounding the seconds when the field doesn't persist the `.usec` piece. | ||||
* | | Unit test for mysql quote time usec | Arthur Neves | 2014-03-12 | 1 | -0/+7 |
|/ | |||||
* | Merge pull request #13040 from kamipo/case_sensitive_comparison | Rafael Mendonça França | 2014-03-12 | 6 | -6/+62 |
|\ | | | | | | | | | | | | | Only use BINARY for mysql case sensitive uniqueness check when column has a case insensitive collation. Conflicts: activerecord/CHANGELOG.md | ||||
| * | Only use BINARY for mysql case sensitive uniqueness check when column has a ↵ | Ryuta Kamizono | 2013-11-26 | 6 | -6/+62 |
| | | | | | | | | case insensitive collation. | ||||
* | | Changelog entry for mysql56 microseconds | Arthur Neves | 2014-03-12 | 1 | -0/+4 |
| | | |||||
* | | Test microsecond on mysql 5.6 | Arthur Neves | 2014-03-12 | 3 | -2/+11 |
| | | |||||
* | | MySQL 5.6 and later supports microsecond precision in datetime. | Tatsuhiko Miyagawa | 2014-03-12 | 1 | -0/+8 |
| | | | | | | | | | | You might want to branch it to include this only for 5.6, but passing these values to < 5.6 doesn't cause issues either. | ||||
* | | Remove mocking on save, when not necessary | Arthur Neves | 2014-03-11 | 1 | -3/+1 |
| | | |||||
* | | Merge pull request #14345 from arthurnn/revert_14052 | Godfrey Chan | 2014-03-11 | 2 | -3/+16 |
|\ \ | | | | | | | Fixes STI when 2+ levels deep. | ||||
| * | | Fixes STI when 2+ levels deep. | Arthur Neves | 2014-03-10 | 2 | -3/+16 |
| | | | | | | | | | | | | | | | | | | PR #14052 Added a regression where it was only looking for methods in one level up, So when the method was defined in a 2+ levels up the inheritance chain, the method was not found as defined. | ||||
* | | | `change_table` supports `citext`. Follow up to #12523. | Yves Senn | 2014-03-11 | 2 | -8/+31 |
| | | | |||||
* | | | register OID for PostgreSQL citex datatype [Troy Kruthoff & Lachlan Sylvester] | lsylvester | 2014-03-11 | 6 | -2/+89 |
|/ / | | | | | | | citext makes it possible to use AR Hash finders for case-insensitive matching as sql UPPER/LOWER functions are not needed. | ||||
* | | Merge pull request #8313 from alan/only_save_changed_has_one_objects | Rafael Mendonça França | 2014-03-10 | 3 | -2/+24 |
|\ \ | | | | | | | | | | | | | | | | | | | Save has_one associations only if record has changes Conflicts: activerecord/CHANGELOG.md | ||||
| * | | Save has_one associations only if record has changes | Alan Kennedy | 2013-10-31 | 3 | -2/+24 |
| | | | | | | | | | | | | | | | | | | Prevents save related callbacks such as `after_commit` being triggered when `has_one` objects are already persisted and have no changes. | ||||
* | | | current_adapter? accepts multiple arguments | Rafael Mendonça França | 2014-03-10 | 1 | -2/+2 |
| | | | |||||
* | | | mysql adapter also fails with a number bigger than int | Rafael Mendonça França | 2014-03-10 | 1 | -2/+5 |
| | | | |||||
* | | | Change the assertions depending in the database adapter | Rafael Mendonça França | 2014-03-10 | 1 | -14/+9 |
| | | | | | | | | | | | | This will avoid the confusing flunk logic | ||||
* | | | Do proper adapter check | Rafael Mendonça França | 2014-03-10 | 1 | -2/+2 |
| | | | |||||
* | | | Whitespaces | Rafael Mendonça França | 2014-03-10 | 1 | -2/+2 |
| | | |