Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Get rid of unused TransactionError constant | Carlos Antonio da Silva | 2014-01-15 | 1 | -3/+0 |
| | | | | | | This has been added 9 years ago in a677da209b16f43198b3485dda89dce862fb9bfb, and removed 6 years ago in 38f8252e2d0a109d1b833d6b289cd989e7bfffe4. | ||||
* | Avoid converting :on option to array twice when defining commit/rollback ↵ | Carlos Antonio da Silva | 2014-01-15 | 1 | -3/+2 |
| | | | | callbacks | ||||
* | Unify changelog entries about single quotes [ci skip] | Carlos Antonio da Silva | 2014-01-15 | 1 | -5/+1 |
| | |||||
* | Merge pull request #13713 from chulkilee/use-single-quotes | Carlos Antonio da Silva | 2014-01-15 | 8 | -20/+24 |
|\ | | | | | Use single quotes in generated files | ||||
| * | Use single quotes in generated files | Chulki Lee | 2014-01-14 | 8 | -20/+24 |
| | | |||||
* | | rm dup code | Aaron Patterson | 2014-01-14 | 1 | -13/+0 |
| | | | | | | | | https://github.com/rails/rails/blob/e64a83cdbd3ce836f3339c600c1006b20c23d8fd/activerecord/test/cases/relation/merging_test.rb#L80-91 | ||||
* | | directly create the ARel AST | Aaron Patterson | 2014-01-14 | 1 | -6/+4 |
| | | | | | | | | rather than allocating a new Relation, just make the AST directly | ||||
* | | Updated comment to mention the enum mapping class method [ci skip] | Godfrey Chan | 2014-01-14 | 1 | -2/+3 |
| | | |||||
* | | "string" isn't a valid column type for ActiveRecord | Aaron Patterson | 2014-01-14 | 1 | -1/+1 |
|/ | | | | | Conflicts: activerecord/test/cases/adapters/sqlite3/sqlite3_adapter_test.rb | ||||
* | don't establish a new connection when testing with `sqlite3_mem`. | Yves Senn | 2014-01-14 | 1 | -15/+21 |
| | | | | | | | This fixes broken `rake test_sqlite3_mem` suite for Active Record. The problem is that that the old database with the schema is lost when establishing a new connection. Upon reconnting we are left with a blank database and tests down the line start failing. | ||||
* | Merge pull request #13350 from ccutrer/sqlite-partial-indexes | Yves Senn | 2014-01-14 | 3 | -1/+25 |
|\ | | | | | sqlite >= 3.8.0 supports partial indexes | ||||
| * | sqlite >= 3.8.0 supports partial indexes | Cody Cutrer | 2014-01-14 | 3 | -1/+25 |
|/ | |||||
* | Merge pull request #13710 from ujjwalt/hotfix/sti | Andrew White | 2014-01-14 | 5 | -8/+42 |
|\ | | | | | Active Record's subclass_from_attributes shouldn't assume :type is for STI unless there is a type column. | ||||
| * | Don't try to get the subclass if the inheritance column doesn't exist | Ujjwal Thaakar | 2014-01-14 | 5 | -8/+42 |
|/ | | | | | | | The `subclass_from_attrs` method is called even if the column specified by the `inheritance_column` setting doesn't exist. This prevents setting associations via the attributes hash if the association name clashes with the value of the setting, typically `:type`. This worked previously in Rails 3.2. | ||||
* | Enum mappings are now exposed via class methods instead of constants. | Godfrey Chan | 2014-01-14 | 3 | -10/+31 |
| | | | | | | | | | | | | | | | | Example: class Conversation < ActiveRecord::Base enum status: [ :active, :archived ] end Before: Conversation::STATUS # => { "active" => 0, "archived" => 1 } After: Conversation.statuses # => { "active" => 0, "archived" => 1 } | ||||
* | Merge pull request #13705 from mess110/controller_generator_route_quotes | Carlos Antonio da Silva | 2014-01-14 | 5 | -11/+15 |
|\ | | | | | Single quotes for controller generated routes. This is more consistent with all other Rails generated code. | ||||
| * | single quotes for controller generated routes | Cristian Mircea Messel | 2014-01-14 | 5 | -11/+15 |
| | | | | | | | | | | | | | | | | | | | | Write routes in route.rb with single quotes get 'welcome/index' instead of get "welcome/index" | ||||
* | | Merge pull request #13706 from prathamesh-sonpatki/issue-13125 | Yves Senn | 2014-01-14 | 1 | -3/+10 |
|\ \ | | | | | | | Fix fields_for documentation with index option [ci skip] | ||||
| * | | Fix fields_for documentation with index option [ci skip] | Prathamesh Sonpatki | 2014-01-14 | 1 | -3/+10 |
| |/ | | | | | | | | | | | | | - fields_for documentation with index option was wrong. - It does not work with passing model as it is. - Changed the example by passing id of the address object. - Fixes #13125. | ||||
* | | Merge pull request #13709 from akshay-vishnoi/typo | Yves Senn | 2014-01-14 | 1 | -1/+1 |
|\ \ | | | | | | | [ci skip] Grammar correction | ||||
| * | | [ci skip] Grammar correction | Akshay Vishnoi | 2014-01-14 | 1 | -1/+1 |
| | | | |||||
* | | | quick pass through Active Record CHANGELOG. [ci skip] | Yves Senn | 2014-01-14 | 1 | -7/+9 |
| | | | |||||
* | | | Merge pull request #13669 from chulkilee/name-error-name | Yves Senn | 2014-01-14 | 3 | -2/+8 |
|\ \ \ | | | | | | | | | Set NameError#name | ||||
| * | | | Set NameError#name | Chulki Lee | 2014-01-13 | 3 | -2/+8 |
| | | | | |||||
* | | | | Merge pull request #13707 from udaykadaboina/docModification | Damien Mathieu | 2014-01-13 | 1 | -2/+2 |
|\ \ \ \ | |_|_|/ |/| | | | [ci skip] Added alias to CSRF | ||||
| * | | | [ci skip] Added alias to CSRF | Uday Kadaboina | 2014-01-14 | 1 | -2/+2 |
|/ / / | |||||
* | | | unscope should remove bind values associated with the where | Aaron Patterson | 2014-01-13 | 2 | -0/+11 |
| | | | |||||
* | | | reverse_order_value= is not private, so no need to send | Aaron Patterson | 2014-01-13 | 1 | -1/+1 |
| | | | | | | | | | | | | It's not public API, but is not private scoped. | ||||
* | | | avoid more dynamic symbols | Aaron Patterson | 2014-01-13 | 1 | -1/+1 |
| | | | |||||
* | | | no need to to_sym | Aaron Patterson | 2014-01-13 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #13704 from waynn/patch-4 | Rafael Mendonça França | 2014-01-13 | 1 | -2/+2 |
|\ \ \ | | | | | | | | | "serie" => "series" | ||||
| * | | | "serie" => "series" | Waynn Lue | 2014-01-13 | 1 | -2/+2 |
| | | | | |||||
* | | | | recover from test runs that leave the database in a bad state | Aaron Patterson | 2014-01-13 | 1 | -1/+1 |
| | | | | |||||
* | | | | updates screenshot data [ci skip] | Xavier Noria | 2014-01-13 | 1 | -1/+1 |
|/ / / | |||||
* / / | revises a few things in the getting started guide [ci skip] | Xavier Noria | 2014-01-13 | 5 | -121/+129 |
|/ / | | | | | | | More to come. | ||||
* | | Merge pull request #13703 from schneems/schneems/rail-secrets-key | Rafael Mendonça França | 2014-01-13 | 2 | -3/+3 |
|\ \ | | | | | | | Favor canonical environment variables for secrets | ||||
| * | | Favor canonical environment variables for secrets | schneems | 2014-01-13 | 2 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Prefixing an environment variable with `RAILS_` should be used when there is otherwise a conflict, such as `RAILS_ENV` or if it is being used for a very Rails specific value. If we remove the prefix of `RAILS_` in the case of `RAILS_SECRET_KEY_BASE` then we can push for a pseudo standard among other frameworks that will accept a common environment key `SECRET_KEY_BASE` to keep your app secure. This is beneficial for containerized deployments such as docker, Heroku, etc. So that the container need to know one less thing about your app (it can just set it no-matter what language or framework you are using). This change also allows the defaults to be consistent with the way the secret key is accessed so `secrets.secret_key_base` is aliased to `SECRET_KEY_BASE` instead of `RAILS_SECRET_KEY_BASE`. | ||||
* | | | Merge pull request #13702 from kuldeepaggarwal/docs-fix | Rafael Mendonça França | 2014-01-13 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | removed extra comma [ci skip] | ||||
| * | | removed extra comma [ci skip] | Kuldeep Aggarwal | 2014-01-14 | 1 | -1/+1 |
|/ / | |||||
* | | Only lookup `config.log_level` for stdlib `::Logger`. Closes #11665. | Yves Senn | 2014-01-13 | 3 | -2/+26 |
| | | | | | | | | | | | | | | | | This prevents Rails from assigning meaningless log levels to third party loggers like log4r. If `Rails.logger` is not `kind_of?(::Logger)` we simply assign the `config.log_level` as is. This bug was introduced by #11665. | ||||
* | | Merge pull request #13699 from rystraum/dependent-restrict-removal | Rafael Mendonça França | 2014-01-13 | 1 | -0/+4 |
|\ \ | | | | | | | Updated Changelog to reflect removal of :dependent => :restrict | ||||
| * | | Updated Changelog to reflect removal of :dependent => :restrict | Rystraum Gamonez | 2014-01-14 | 1 | -0/+4 |
|/ / | | | | | | | | | Based on https://github.com/rails/rails/issues/2502, the option has been removed. | ||||
* | | Merge pull request #13693 from gaurish/typo | Yves Senn | 2014-01-13 | 7 | -6/+39 |
|\ \ | | | | | | | [JRuby] Fix incorrect RUBY_ENGINE value | ||||
| * | | standardize on jruby_skip & rbx_skip | Gaurish Sharma | 2014-01-13 | 7 | -6/+39 |
| | | | | | | | | | | | | | | | | | | This Adds helpers(jruby_skip & rbx_skip). In Future, Plan is to use these helpers instead of calls directly to RUBY_ENGINE/RbConfig/JRUBY_VERSION | ||||
* | | | fix bug in becomes! when changing from base to subclass. Closes #13272. | Yves Senn | 2014-01-13 | 3 | -1/+22 |
|/ / | |||||
* | | highlight http://localhost:3000 in README.md. Closes #13643. [ci skip] | Yves Senn | 2014-01-13 | 1 | -1/+2 |
| | | | | | | | | | | This has no effect on the API docs. The main API page is taken from: railties/RDOC_MAIN.rdoc | ||||
* | | doc proc/lambda arg on inclusion validation. Closes #13689. [ci skip] | Yves Senn | 2014-01-13 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #13687 from gaurish/spring-has-no-love-for-jruby | Yves Senn | 2014-01-12 | 1 | -0/+2 |
|\ \ | | | | | | | Skip Spring App Generator tests on JRuby | ||||
| * | | Skip Spring App Generator tests on JRuby | Gaurish Sharma | 2014-01-13 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | Spring makes extensive use of Process.fork, so won't be able to provide JRuby Support, hence skip these tests on JRuby. | ||||
* | | | Merge pull request #13646 from jenseng/json_escape | Godfrey Chan | 2014-01-12 | 1 | -12/+12 |
|\ \ \ | | | | | | | | | Clarify behavior of json_escape [ci skip] |