- 13 2月, 2016 1 次提交
-
-
由 Gaurav Sharma 提交于
since starting with Rails 5.x(beta) we prefer to use rails as the replacement of rake commands, may be change log will be the same
-
- 11 2月, 2016 5 次提交
-
-
由 Sean Griffin 提交于
In Rails 5, we're much more restrictive about when we do or don't cache a prepared statement. In particular, we never cache when we are sending an IN statement or a SQL string literal However, in the case of Adequate Record, we are *always* sending a raw SQL string, and we *always* want to cache the result. Fixes #23507 /cc @tgxworld
-
由 yuuji.yaginuma 提交于
This removes the following warnings. ``` activerecord/lib/active_record/relation/finder_methods.rb:252: warning: ambiguous first argument; put parentheses or a space even after `-' operator activerecord/lib/active_record/relation/finder_methods.rb:258: warning: ambiguous first argument; put parentheses or a space even after `-' operator activerecord/lib/active_record/relation/finder_methods.rb:268: warning: ambiguous first argument; put parentheses or a space even after `-' operator activerecord/lib/active_record/relation/finder_methods.rb:274: warning: ambiguous first argument; put parentheses or a space even after `-' operator ```
-
由 Aaron Patterson 提交于
This commit walks the reflection tree and builds the scope chain functionally. It also removes the chain cache since the cache doesn't seem to have any impact on performance (I'd prefer to only cache at proven bottlenecks)
-
由 Brian Christian 提交于
-
由 Brian Christian 提交于
-
- 10 2月, 2016 2 次提交
-
-
由 Brian Christian 提交于
-
由 Diego Silva 提交于
When changing the PK for a record which has a uniqueness validation on some other attribute, Active Record should exclude itself from the validation based on the PK value stored on the DB (id_was) instead of its new value (id).
-
- 08 2月, 2016 2 次提交
-
-
由 Ryuta Kamizono 提交于
Follow up to #23508.
-
由 Ryuta Kamizono 提交于
A return value of `schema_type` is used by: 1. primary key type: using as `symbol.inspect` 2. normal column type: using as `symbol.to_s` It is better to return symbol.
-
- 07 2月, 2016 2 次提交
-
-
由 Vijay Dev 提交于
-
由 Mehmet Emin İNAÇ 提交于
With this addition, you can add a column into the table like: ``` create_table(:numeric_types) do |t| t.numeric :foo, precision: 10, scale: 2, default: 2.0 end ``` The result of the migration above is same with: ``` create_table(:numeric_types) do |t| t.decimal :foo, precision: 10, scale: 2, default: 2.0 end ```
-
- 06 2月, 2016 4 次提交
-
-
由 Ryuta Kamizono 提交于
Fixes #22980.
-
由 Ryuta Kamizono 提交于
It appears first in `lib/active_record.rb`.
-
由 Sean Griffin 提交于
This reverts commit 99801c6a. Ultimately it doesn't matter whether `add_index` or `t.index` are used in the schema dumper in any meaningful way. There are gems out there which hook into the old behavior for things like indexing materialized views. Since the reverted commit doesn't seem to add much benefit, there's no reason for us to break these gems.
-
- 05 2月, 2016 2 次提交
-
-
由 Jeremy Baker 提交于
-
由 Jeremy Baker 提交于
-
- 04 2月, 2016 7 次提交
-
-
由 Jeremy Baker 提交于
If you set the DATABASE_URL environment variable to `mydatabase` by accident, you end up getting a series of errors that are hard to trace. For example: ``` warning: already initialized constant ActiveRecord::Base::OrmAdapter ``` Turns out the cascade of errors is due to the error raised by `.tr` being called on `nil`. This commit makes sure that `scheme` is set before calling `.tr` on it. My previous iteration used `@uri.scheme.try(:tr, '-', '_')` but using the `&&` logical operator is a fair bit faster: http://stackoverflow.com/questions/26655032/try-vs-performance With this change, the error message becomes much more understandable: ``` FATAL: database "mydatabase" does not exist (ActiveRecord::NoDatabaseError) ```
-
由 Akira Matsuda 提交于
-
由 Ryuta Kamizono 提交于
-
由 Matthew Draper 提交于
-
由 Ryuta Kamizono 提交于
Currently some features uses `information_schema` (e.g. foreign key support). `information_schema` introduced since MySQL 5.0.
-
由 Matthew Draper 提交于
This still isn't as separated as I'd like, but it at least moves most of the burden of alias mapping in one place.
-
- 03 2月, 2016 6 次提交
-
-
由 Akira Matsuda 提交于
-
由 Prathamesh Sonpatki 提交于
-
由 Himesh 提交于
Changed id-writer to save join table records based on association primary key
-
由 Remo Mueller 提交于
-
由 Sean Griffin 提交于
There was a typo in the variable name leading to infinite recursion
-
由 Sean Griffin 提交于
We had previously updated this to attempt to map over whatever was passed in, so that additional types like range and array could benefit from this behavior without the time zone converter having to deal with every known type. However, the default behavior of a type is to just yield the given value to `map`, which means that if we don't actually know how to handle a value, we'll just recurse infinitely. Since both uses of `map` in this case occur in cases where we know receiving the same object will recurse, we can just break on reference equality. Fixes #23241.
-
- 02 2月, 2016 6 次提交
-
-
由 yui-knk 提交于
`#initialize` of `HasManyReflection`, `HasOneReflection` and `BelongsToReflection` only pass all arguments to `super` by passed order. These overriding can be removed.
-
由 PareshGupta 提交于
-
由 Rafael Mendonça França 提交于
And improve changelongs. [ci skip]
-
由 Sean Griffin 提交于
-
由 Jeremy Daer 提交于
When a `before_commit` callback raises, the database is rolled back but AR's record of the current transaction is not, leaving the connection in a perpetually broken state that affects all future users of the connection: subsequent requests, jobs, etc. They'll think a transaction is active when none is, so they won't BEGIN on their own. This manifests as missing `after_commit` callbacks and broken ROLLBACKs. This happens because `before_commit` callbacks fire before the current transaction is popped from the stack, but the exception-handling path they hit assumes that the current transaction was already popped. So the database ROLLBACK is issued, but the transaction stack is left intact. Common cause: deadlocked `#touch`, which is now implemented with `before_commit` callbacks. What's next: * We shouldn't allow active transaction state when checking in or out from the connection pool. Verify that conns are clean. * Closer review of txn manager sad paths. Are we missing other spots where we'd end up with incorrect txn state? What's the worst that can happen if txn state drifts? How can we guarantee it doesn't and contain the fallout if it does? Thanks for @tomafro for expert diagnosis!
-
由 Sean Griffin 提交于
This reverts commit 9f3730a5, reversing changes made to 2637fb75. There are additional issues with this commit that need to be addressed before this change is ready (see #23377). This is a temporary revert in order for us to have more time to address the issues with that PR, without blocking the release of beta2.
-
- 01 2月, 2016 3 次提交
-
-
由 Yasuo Honda 提交于
for those who already migrated to Rails 5.0.0 beta
-
由 Yasuo Honda 提交于
to support Oracle database which only supports 30 byte identifier length
-
由 PareshGupta 提交于
-