Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | Expanded coverage on JSON encoding | Godfrey Chan | 2013-11-26 | 1 | -2/+12 | |
| | | | | ||||||
| * | | | Added some failing tests where the JSON encoder is not resolving as_json ↵ | Godfrey Chan | 2013-11-26 | 1 | -2/+12 | |
| | | | | | | | | | | | | | | | | correctly | |||||
* | | | | More typo fixes | Akira Matsuda | 2013-11-27 | 2 | -2/+2 | |
|/ / / | ||||||
* | | | Minor typo fixes | Akira Matsuda | 2013-11-27 | 4 | -6/+6 | |
| | | | ||||||
* | | | Add branch to arel in Gemfile so that we can use local repos [ci skip] | Carlos Antonio da Silva | 2013-11-26 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #13018 from heruku/scoping_fix | Yves Senn | 2013-11-26 | 4 | -1/+19 | |
|\ \ \ | | | | | | | | | changed update counter to act on unscoped model | |||||
| * | | | changed update counter to act on unscoped model | heruku | 2013-11-26 | 4 | -1/+19 | |
|/ / / | ||||||
* | | | Merge pull request #13049 from senny/remove_firebird_tests | Rafael Mendonça França | 2013-11-26 | 3 | -148/+0 | |
|\ \ \ | |/ / |/| | | remove leftover firebird adapter tests. | |||||
| * | | remove leftover firebird adapter tests. | Yves Senn | 2013-11-26 | 3 | -148/+0 | |
| | | | ||||||
* | | | sync example and documentation for strong_parameters hash whitelisting. | Yves Senn | 2013-11-26 | 1 | -5/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] This is a follow up to #12609 and integrates #12158. Thanks @aspiers. Closes #12158. | |||||
* | | | tyop | Akira Matsuda | 2013-11-26 | 1 | -1/+1 | |
|/ / | ||||||
* / | `rename_index`: add the new index before removing the old one. | Yves Senn | 2013-11-26 | 3 | -8/+29 | |
|/ | | | | | | | | | | | This prevents the following error when a MySQL index on a foreign key column is renamed: ``` ActiveRecord::StatementInvalid: Mysql2::Error: Cannot drop index 'index_engines_on_car_id': needed in a foreign key constraint: DROP INDEX `index_engines_on_car_id` ON `engines` ``` refs: #13038. | |||||
* | Merge pull request #13042 from brianstorti/fix-dependent-destroy-12812 | Rafael Mendonça França | 2013-11-25 | 5 | -1/+25 | |
|\ | | | | | Raise `RecordNotDestroyed` when children can't be replaced | |||||
| * | move changelog entry to the top | Brian Thomas Storti | 2013-11-25 | 1 | -6/+6 | |
| | | ||||||
| * | Raise `RecordNotDestroyed` when children can't be replaced | Brian Thomas Storti | 2013-11-25 | 5 | -1/+25 | |
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #12812 Raise `ActiveRecord::RecordNotDestroyed` when a child marked with `dependent: destroy` can't be destroyed. The following code: ```ruby class Post < ActiveRecord::Base has_many :comments, dependent: :destroy end class Comment < ActiveRecord::Base before_destroy do return false end end post = Post.create!(comments: [Comment.create!]) post.comments = [Comment.create!] ```` would result in a `post` with two `comments`. With this commit, the same code would raise a `RecordNotDestroyed` exception, keeping the `post` with the same `comment`. | |||||
* | require the files we test | Aaron Patterson | 2013-11-25 | 1 | -0/+1 | |
| | ||||||
* | Merge pull request #13034 from vipulnsward/some_typos | Rafael Mendonça França | 2013-11-25 | 2 | -2/+2 | |
|\ | | | | | Fix some minor typos | |||||
| * | Fix some minor typos [ci skip] | Vipul A M | 2013-11-26 | 2 | -2/+2 | |
| | | ||||||
* | | Revert "Merge pull request #13027 from akshay-vishnoi/f-refactor" | Carlos Antonio da Silva | 2013-11-25 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | This reverts commit f4a5a9ea4d183f4102796215d4502c46dbe3e52b, reversing changes made to 7ccb482181ee6c47c765406009018a15172812de. Reason: The logic is different, the first call to #option_value_selected? is for the :selected option (the argument is the "selected" variable), the second call is for the :disabled option (the argument is the "disabled" variable). | |||||
* | | Merge pull request #13027 from akshay-vishnoi/f-refactor | Yves Senn | 2013-11-25 | 1 | -1/+1 | |
|\ \ | | | | | | | avoiding calling of #option_value_selected? two times | |||||
| * | | avoiding calling of #option_value_selected? two times | Akshay Vishnoi | 2013-11-25 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #13029 from akshay-vishnoi/documentation | Yves Senn | 2013-11-25 | 1 | -4/+4 | |
|\ \ \ | |_|/ |/| | | `delgated` => `delegated` | |||||
| * | | `delgated` => `delegated` | Akshay Vishnoi | 2013-11-25 | 1 | -4/+4 | |
| | | | ||||||
* | | | Merge pull request #13025 from pwnall/nice_test | Yves Senn | 2013-11-25 | 3 | -0/+19 | |
|\ \ \ | | | | | | | | | Better error message for typos in assert_response argument. | |||||
| * | | | Better error message for typos in assert_response argument. | Victor Costan | 2013-11-25 | 3 | -0/+19 | |
|/ / / | | | | | | | | | | | | | This commit makes it really easy to debug errors due to typos like "assert_response :succezz". | |||||
* | | | Merge pull request #13026 from vipulnsward/document_rewhere | Yves Senn | 2013-11-24 | 1 | -0/+26 | |
|\ \ \ | | | | | | | | | Added `rewhere` usage to AR querying guides | |||||
| * | | | Added `rewhere` usage to AR querying guides | Vipul A M | 2013-11-25 | 1 | -0/+26 | |
|/ / / | ||||||
* | | | Merge pull request #13020 from razielgn/raise-inspectable-messages | Carlos Antonio da Silva | 2013-11-24 | 2 | -4/+4 | |
|\ \ \ | | | | | | | | | Raise RuntimeErrors with inspectable and MRI-independent messages. | |||||
| * | | | Raise RuntimeErrors with inspectable and MRI-independent messages. | Federico Ravasio | 2013-11-24 | 2 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Previous behaviour was MRI-dependent, now we're making sure the message is correctly shown: something that can be relyied upon across every Ruby implementation. | |||||
* | | | | Merge remote-tracking branch 'docrails/master' | Xavier Noria | 2013-11-24 | 29 | -183/+203 | |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activesupport/lib/active_support/core_ext/hash/deep_merge.rb activesupport/lib/active_support/core_ext/hash/keys.rb | |||||
| * | | | | a couple of copy-edits before merging [ci skip] | Xavier Noria | 2013-11-24 | 1 | -3/+2 | |
| | | | | | ||||||
| * | | | | API Documentation Guidelines link correction [ci skip] | Harshad Sabne | 2013-11-23 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Improve reading / style of hashes in AR guide | Carlos Antonio da Silva | 2013-11-19 | 1 | -5/+5 | |
| | | | | | ||||||
| * | | | | Revert last two commits mistakenly changing join/include syntax | Carlos Antonio da Silva | 2013-11-19 | 1 | -7/+7 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --- Revert "syntax error joining/including models" This reverts commit ac35f72f30c47298a20a90c8c537db888b463bbb. --- Revert "syntax error joining tables" This reverts commit c365986b48c3e8bc8c7f3fa6a8521616ed5dc138. --- Comments: https://github.com/rails/docrails/commit/c365986b48c3e8bc8c7f3fa6a8521616ed5dc138#commitcomment-4630684 | |||||
| * | | | | syntax error joining/including models | Sergio | 2013-11-18 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | syntax error joining/including models | |||||
| * | | | | syntax error joining tables | Sergio | 2013-11-18 | 1 | -5/+5 | |
| | | | | | | | | | | | | | | | syntax error joining tables | |||||
| * | | | | The option is called encode_big_decimal_as_string [ci-skip] | Godfrey Chan | 2013-11-15 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Update action_controller_overview.md [ci skip] | Harshad Sabne | 2013-11-15 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | Code style adherence | |||||
| * | | | | Fix broken Wikipedia link. | Harshad Sabne | 2013-11-13 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Remove outdated guide on using rack builder w/o rails middleware | Alex Johnson | 2013-11-13 | 1 | -20/+0 | |
| | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| * | | | | Correct guide relating MiddlewareStackProxy with Enumerable | Alex Johnson | 2013-11-13 | 1 | -5/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] Enumerable.instance_methods & Rails::Configuration::MiddlewareStackProxy.instance_methods # => [:as_json] MiddlewareStackProxy does not share any notable methods with Enumerable. | |||||
| * | | | | [ci skip] Replace #=> with # => | Alex Johnson | 2013-11-12 | 3 | -17/+17 | |
| | | | | | ||||||
| * | | | | Fix RailsServer#start content in initialization guide | Alex Johnson | 2013-11-12 | 1 | -17/+23 | |
| | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| * | | | | Change syntax format for example returned values | Prem Sichanugrist | 2013-11-11 | 19 | -107/+107 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to our guideline, we leave 1 space between `#` and `=>`, so we want `# =>` instead of `#=>`. Thanks to @fxn for the suggestion. [ci skip] | |||||
| * | | | | Add section for command_tasks in initialization guide | Alex Johnson | 2013-11-08 | 1 | -12/+42 | |
| | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| * | | | | Update commands.rb content in initialization guide | Alex Johnson | 2013-11-08 | 1 | -2/+8 | |
| | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| * | | | | Update README.rdoc [ci skip] | Harshad Sabne | 2013-11-08 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | Highlighted code | |||||
* | | | | | Merge pull request #13021 from razielgn/rbx-gemfile-and-travis-fixes | Rafael Mendonça França | 2013-11-24 | 1 | -1/+1 | |
|\ \ \ \ \ | | | | | | | | | | | | | Updated allow_failures for Rubinius 2.2.1. | |||||
| * | | | | | Updated allow_failures for Rubinius 2.2.1. | Federico Ravasio | 2013-11-24 | 1 | -1/+1 | |
|/ / / / / | ||||||
* | | | | | Merge pull request #13019 from razielgn/rbx-gemfile-and-travis-fixes | Rafael Mendonça França | 2013-11-24 | 2 | -5/+3 | |
|\ \ \ \ \ | |_|/ / / |/| | | | | Rubinius Gemfile and travis fixes |