| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\
| |
| | |
Raise `RecordNotDestroyed` when children can't be replaced
|
| | |
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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`.
|
| |
|
|\
| |
| | |
Fix some minor typos
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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).
|
|\ \
| | |
| | | |
avoiding calling of #option_value_selected? two times
|
| | | |
|
|\ \ \
| |_|/
|/| | |
`delgated` => `delegated`
|
| | | |
|
|\ \ \
| | | |
| | | | |
Better error message for typos in assert_response argument.
|
|/ / /
| | |
| | |
| | |
| | | |
This commit makes it really easy to debug errors due to typos like
"assert_response :succezz".
|
|\ \ \
| | | |
| | | | |
Added `rewhere` usage to AR querying guides
|
|/ / / |
|
|\ \ \
| | | |
| | | | |
Raise RuntimeErrors with inspectable and MRI-independent messages.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Conflicts:
activesupport/lib/active_support/core_ext/hash/deep_merge.rb
activesupport/lib/active_support/core_ext/hash/keys.rb
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
---
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
|
| | | | |
| | | | |
| | | | | |
syntax error joining tables
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
Code style adherence
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
[ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
[ci skip] Enumerable.instance_methods &
Rails::Configuration::MiddlewareStackProxy.instance_methods # =>
[:as_json]
MiddlewareStackProxy does not share any notable methods with Enumerable.
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
[ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
According to our guideline, we leave 1 space between `#` and `=>`, so we
want `# =>` instead of `#=>`.
Thanks to @fxn for the suggestion.
[ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | | |
[ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | | |
[ci skip]
|
| | | | |
| | | | |
| | | | | |
Highlighted code
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Updated allow_failures for Rubinius 2.2.1.
|
|/ / / / / |
|
|\ \ \ \ \
| |_|/ / /
|/| | | | |
Rubinius Gemfile and travis fixes
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Before Rubinius 2.2.0, some parts of the stdlib were lazily loaded, even
if not included in the Gemfile. Now it's 100% required to include the
gem 'rubysl', otherwise the stdlib is not available entirely, breaking
everything.
Also, the rubysl-test-unit gem was depending on minitest ~> 4.7, thus causing
conflicts with Rails's dependency on minitest 5. That is not the case
anymore since rubysl-test-unit 2.0.2, so it is completely safe to
include it fully.
When Travis is going to update RVM to 1.24, it'll be safe to use rbx-2,
thus picking new versions automatically.
|
|\ \ \ \
| | | | |
| | | | | |
Test for when controller with skip-assets
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Added test when task generator called on revoke
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Helper should be a module in Rails guides [ci skip]
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Using activerecord-jdbcsqlite3-adapter
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
PluginGeneratorTest needs to have
activerecord-jdbcsqlite3-adapter to pass under JRuby
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Cleaned up app generator test
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Don't call #as_json on children of Array and Hash with nil when no arguments are passed
|