Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add doc for `:touch` option of AR::Base#save | claudiob | 2014-12-30 | 1 | -0/+8 |
| | | | | | | | | | ActiveRecord::Base `save` and `save!` take an option boolean `:touch` parameter since #18225 (stems from #18202). This commit document that parameter. [ci skip] | ||||
* | Compare http request type with test equivalent methods, and application ↵ | Zachary Scott | 2014-12-29 | 1 | -2/+2 |
| | | | | | | use-cases. Briefly explain difference between Functional and Request type tests. [ci skip] | ||||
* | Merge pull request #18246 from aprescott/document-include_all_helpers-setting | Rafael Mendonça França | 2014-12-29 | 1 | -0/+2 |
|\ | | | | | Document the include_all_helpers config setting | ||||
| * | Document the include_all_helpers config setting. | Adam Prescott | 2014-12-29 | 1 | -0/+2 |
| | | | | | | | | | | This config setting has been available for quite a while but was not documented. | ||||
* | | Merge pull request #18244 from cristianbica/remove-object-itself | Rafael Mendonça França | 2014-12-29 | 3 | -25/+0 |
|\ \ | |/ |/| | Removed Object#itself as it's implemented in ruby 2.2 | ||||
| * | Removed Object#itself as it's implemented in ruby 2.2 | Cristian Bica | 2014-12-29 | 3 | -25/+0 |
| | | |||||
* | | Rephrase functional testing in its relation to Rails Controller Unit Testing. | Zachary Scott | 2014-12-29 | 1 | -1/+1 |
|/ | | | | [ci skip] | ||||
* | Merge pull request #18243 from JoshCheek/mailer_guide_code_example | Carlos Antonio da Silva | 2014-12-29 | 1 | -1/+3 |
|\ | | | | | Guide for Action Mailer Basics: Un-inline if statement | ||||
| * | Un-inline if statement | Josh Cheek | 2014-12-29 | 1 | -1/+3 |
|/ | | | | | The single line was long enough that it rendered on two lines, causing the example to look like syntactically invalid code. | ||||
* | Merge pull request #18240 from shunsukeaida/add_tzinfo_data_to_plugin_gemfile | Rafael Mendonça França | 2014-12-29 | 1 | -0/+4 |
|\ | | | | | plugin's gemfile also needs tzinfo-data in Windows | ||||
| * | plugin's gemfile also needs tzinfo-data in Windows | ShunsukeAida | 2014-12-30 | 1 | -0/+4 |
| | | |||||
* | | Remove some warning with Ruby 2.2 | Rafael Mendonça França | 2014-12-29 | 1 | -1/+1 |
| | | |||||
* | | Ensure `first!` and friends work on loaded associations | Sean Griffin | 2014-12-29 | 3 | -0/+12 |
| | | | | | | | | Fixes #18237 | ||||
* | | Require Arel 7.0.0.alpha | Sean Griffin | 2014-12-29 | 1 | -1/+1 |
| | | |||||
* | | Remove all cases of manuallly wrapping `Arel::Nodes::Quoted` | Sean Griffin | 2014-12-29 | 7 | -34/+15 |
| | | | | | | | | | | | | | | | | | | | | This is no longer required now that we are injecting a type caster object into the Arel table, with the exception of uniqueness validations. Since it calls `ConnectionAdapter#type_cast`, the value has already been cast for the database. We don't want Arel to attempt to cast it further, so we need to continue wrapping it in a quoted node. This can potentially go away when this validator is refactored to make better use of `where` or the predicate builder. | ||||
* | | Pass a type caster when aliasing tables for joins | Sean Griffin | 2014-12-29 | 4 | -10/+15 |
| | | |||||
* | | Fix failing tests | Sean Griffin | 2014-12-29 | 1 | -8/+4 |
| | | |||||
* | | Extract an explicit type caster class | Sean Griffin | 2014-12-29 | 8 | -39/+69 |
| | | |||||
* | | Rely on the injectable type caster for `arel_table` | Sean Griffin | 2014-12-29 | 6 | -74/+22 |
| | | | | | | | | | | | | | | This API will require much less consuming code to change to accomodate the removal of automatic type casting from Arel. As long as the predicates are constructed using the `arel_table` off of an AR subclass, there will be no changes that need to happen. | ||||
* | | Merge pull request #18097 from k0kubun/readonly-eager_load | Rafael Mendonça França | 2014-12-29 | 3 | -1/+11 |
|\ \ | | | | | | | `eager_load` preserves readonly flag for associations | ||||
| * | | `eager_load` preserves readonly flag for associations | Takashi Kokubun | 2014-12-30 | 3 | -1/+11 |
| | | | |||||
* | | | Add nodoc to touch at no_touching.rb | Rafael Mendonça França | 2014-12-29 | 1 | -1/+1 |
| |/ |/| | | | | | | | | | It is already docummented at persistence.rb. [ci skip] | ||||
* | | Just check if the buffer exists before changing it | Rafael Mendonça França | 2014-12-29 | 2 | -6/+4 |
| | | |||||
* | | Merge pull request #18236 from vipulnsward/dont-eval-frozen | Rafael Mendonça França | 2014-12-29 | 2 | -1/+12 |
|\ \ | |/ |/| | | | Fix html_safe string access modifying frozen values | ||||
| * | When trying to access a character on a string buffer object via `:[]`, if ↵ | Vipul A M | 2014-12-29 | 2 | -1/+12 |
|/ | | | | | | | | | | | | | | | | the object being accessed currently returns `html_safe?` as true, we used to set `@html_safe` variable as true on new object created. When doing something like x = 'Hello'.html_safe x[/a/, 1] would throw an error on ruby 2.2, since when nothign gets matched nil is returned by the code and it tries to set `@html_safe` value to true, which would error since starting 2.2 nil is frozen. This change adds a safety net to avoid setting `@html_safe = true` on frozen objects. Fixes #18235 | ||||
* | Merge pull request #18232 from yousuketto/doc-for-bundler-require | Yves Senn | 2014-12-29 | 1 | -1/+1 |
|\ | | | | | adjust args of Bundler.require in guides [ci skip] | ||||
| * | adjust args of Bundler.require in guides [ci skip] | Yosuke | 2014-12-29 | 1 | -1/+1 |
|/ | | | | | | Arguments of Bundler.require of generator has been changed from rails4.0.4. https://github.com/rails/rails/blob/v4.0.4/railties/lib/rails/generators/rails/app/templates/config/application.rb#L16 | ||||
* | Merge pull request #18231 from untidy-hair/improve_a_test | Guillermo Iguaran | 2014-12-28 | 1 | -1/+1 |
|\ | | | | | Expectation comes first | ||||
| * | Expectation comes first | Yukio Mizuta | 2014-12-28 | 1 | -1/+1 |
|/ | |||||
* | Minor documentation edits [ci skip] | Robin Dupret | 2014-12-28 | 6 | -19/+19 |
| | |||||
* | Merge pull request #18209 from claudiob/bump-to-2-2 | Guillermo Iguaran | 2014-12-28 | 10 | -12/+12 |
|\ | | | | | Require Ruby 2.2 for Rails 5.0 | ||||
| * | Require Ruby 2.2 for Rails 5.0 | claudiob | 2014-12-26 | 10 | -12/+12 |
| | | | | | | | | | | Stems from [this comment](https://github.com/rails/rails/pull/18203#issuecomment-68138096) by @robin850 and by the blog post http://weblog.rubyonrails.org/2014/12/19/Rails-4-2-final | ||||
* | | Merge pull request #18213 from agrobbin/moar-concerns | Sean Griffin | 2014-12-28 | 3 | -4/+34 |
|\ \ | | | | | | | Actually autoload all second-level directories called `app/*/concerns` | ||||
| * | | actually autoload all second-level directories called `app/*/concerns` | Alex Robbin | 2014-12-26 | 3 | -4/+34 |
| | | | |||||
* | | | docs, cleanup Active Record CHANGELOG. [ci skip] | Yves Senn | 2014-12-28 | 1 | -4/+4 |
| | | | |||||
* | | | Merge pull request #18229 from benprew/patch-2 | Sean Griffin | 2014-12-28 | 1 | -4/+2 |
|\ \ \ | | | | | | | | | Update example test documentation | ||||
| * | | | Update example test documentation | Ben Prew | 2014-12-28 | 1 | -4/+2 |
|/ / / | | | | | | | Example does not work with session headers, should use request headers. [ci skip] | ||||
* | | | Merge pull request #18226 from kamipo/fix_warning_interpreted_as_argument_prefix | Guillermo Iguaran | 2014-12-28 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Fix warning: `*' interpreted as argument prefix | ||||
| * | | | Fix warning: `*' interpreted as argument prefix | Ryuta Kamizono | 2014-12-28 | 1 | -1/+1 |
|/ / / | |||||
* | | | Merge pull request #18225 from DanOlson/update-without-changing-timestamps | Sean Griffin | 2014-12-27 | 5 | -10/+48 |
|\ \ \ | | | | | | | | | Provide :touch option to save() to accommodate saving without updating t... | ||||
| * | | | Provide :touch option to save() to accommodate saving without updating ↵ | Dan Olson | 2014-12-27 | 5 | -10/+48 |
|/ / / | | | | | | | | | | timestamps. [#18202] | ||||
* | | | Provide a better error message for unsupported classes in `serialize` | Sean Griffin | 2014-12-27 | 3 | -0/+24 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We only support classes which provide a no-args constructor to use as a default value. We can provide a more helpful error message if we catch this when `serialize` is called, rather than letting it error when you try to assign the attribute. Fixes #18224 | ||||
* | | | Merge pull request #18220 from kamipo/bigint_primary_key_support | Sean Griffin | 2014-12-27 | 6 | -17/+71 |
|\ \ \ | | | | | | | | | Add bigint primary key support for MySQL. | ||||
| * | | | Add bigint primary key support for MySQL. | Ryuta Kamizono | 2014-12-28 | 5 | -7/+36 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example: create_table :foos, id: :bigint do |t| end | ||||
| * | | | Support for any type primary key. | Ryuta Kamizono | 2014-12-28 | 4 | -10/+35 |
|/ / / | |||||
* | | | add autoload for `ForeignKeyDefinition`. | Yves Senn | 2014-12-27 | 1 | -0/+1 |
| | | | | | | | | | | | | /cc @sgrif | ||||
* | | | reduce duplication in `ConnectionAdapters::Table` docs. [ci skip] | Yves Senn | 2014-12-27 | 1 | -27/+38 |
| | | | | | | | | | | | | | | | | | | Most of the documentation very closely mirrors the matching docs from `SchemaStatements`. I reduced duplicated copy and added links to the underlying methods for the user to follow. | ||||
* | | | Add a note about non-buffering servers for AC::Live | Aditya Kapoor | 2014-12-27 | 1 | -0/+5 |
| | | | | | | | | | | | | [ci skip] | ||||
* | | | Merge pull request #18221 from kamipo/refactor_table_definition_primary_key | Sean Griffin | 2014-12-27 | 1 | -4/+2 |
|\ \ \ | | | | | | | | | Refactor `PostgreSQL::TableDefinition#primary_key` | ||||
| * | | | Refactor `PostgreSQL::TableDefinition#primary_key` | Ryuta Kamizono | 2014-12-27 | 1 | -4/+2 |
|/ / / | | | | | | | | | | | | | Because call the `column` method and set the `options[:primary_key]` is handled at `super`, here need only treat the `options[:default]`. |