| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
This issue was introduced with d849f42 to solve #19782. However, we can
solve #19782 without causing the issue. It is enough to save only when
necessary.
Fixes #27338.
|
|
|
|
|
|
| |
Some methods were added to public API in
5b14129d8d4ad302b4e11df6bd5c7891b75f393c and they should be not part of
the public API.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With the changes in #25337, double save bugs are pretty much impossible,
so we can just lift this restriction with pretty much no change. There
were a handful of cases where we were relying on specific quirks in
tests that had to be updated. The change to has_one associations was due
to a particularly interesting test where an autosaved has_one
association was replaced with a new child, where the child failed to
save but the test wanted to check that the parent id persisted to `nil`.
I think this is almost certainly the wrong behavior, and I may change
that behavior later. But ultimately the root cause was because we never
remove the parent in memory when nullifying the child. This makes #23197
no longer needed, but it is what we'll do to fix some issues on 5.0
Close #23197
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When the association is autosaved we were storing the details with
string keys. This was creating inconsistency with other details that are
added using the `Errors#add` method. It was also inconsistent with the
`Errors#messages` storage.
To fix this inconsistency we are always storing with symbols. This will
cause a small breaking change because in those cases the details could
be accessed as strings keys but now it can not.
The reason that we chose to do this breaking change is because `#details`
should be considered a low level object like `#messages` is.
Fix #26499.
[Rafael Mendonça França + Marcus Vieira]
|
|
|
|
|
|
| |
assert [1, 3].includes?(2) fails with unhelpful "Asserting failed" message
assert_includes [1, 3], 2 fails with "Expected [1, 3] to include 2" which makes it easier to debug and more obvious what went wrong
|
|
|
|
|
|
|
|
|
|
|
|
| |
"models/comment"`
to address BasicsTest#test_readonly_attributes failure #26368
It reproduces only when both of these conditions are satisfied:
- Other test files `autosave_association_test.rb` or `where_test.rb`
which executes `require "models/comment"` then `require "models/post"`
- When `autosave_association_test.rb` or `where_test.rb` executed before `base_test.rb`
|
|
|
|
|
|
|
|
| |
Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces
Fix all violations in the repository.
|
| |
|
| |
|
|
|
|
|
| |
The current code base is not uniform. After some discussion,
we have chosen to go with double quotes by default.
|
|
|
|
| |
- marked_for_destroyal -> marked_for_destruction
|
| |
|
|\
| |
| |
| |
| |
| | |
Errors can be indexed with nested attributes
Close #8638
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
`has_many` can now take `index_errors: true` as an
option. When this is enabled, errors for nested models will be
returned alongside an index, as opposed to just the nested model name.
This option can also be enabled (or disabled) globally through
`ActiveRecord::Base.index_nested_attribute_errors`
E.X.
```ruby
class Guitar < ActiveRecord::Base
has_many :tuning_pegs
accepts_nested_attributes_for :tuning_pegs
end
class TuningPeg < ActiveRecord::Base
belongs_to :guitar
validates_numericality_of :pitch
end
```
- Old style
- `guitar.errors["tuning_pegs.pitch"] = ["is not a number"]`
- New style (if defined globally, or set in has_many_relationship)
- `guitar.errors["tuning_pegs[1].pitch"] = ["is not a number"]`
[Michael Probber, Terence Sun]
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As per the docs, `mark_for_destruction` should do nothing if `autosave`
is not set to true. We normally persist associations on a record no
matter what if the record is a new record, but we were always skipping
records which were `marked_for_destruction?`.
Fixes #20882
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This code is so fucked. Things that cause this bug not to replicate:
- Defining the validation before the association (we end up calling
`uniq!` on the errors in the autosave validation)
- Adding `accepts_nested_attributes_for` (I have no clue why. The only
thing it does that should affect this is adds `autosave: true` to the
inverse reflection, and doing that manually doesn't fix this).
This solution is a hack, and I'm almost certain there's a better way to
go about it, but this shouldn't cause a huge hit on validation times,
and is the simplest way to get it done.
Fixes #20874.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We deprecate the support for passing an argument to force reload in
6eae366d0d2e5d5211eeaf955f56bd1dc6836758. That led to several
deprecation warning when running Active Record test suite.
This commit silence the warnings by properly calling `#reload` on the
association proxy or on the association object instead. However, there
are several places that `ActiveSupport::Deprecation.silence` are used as
those tests actually tests the force reload functionality and will be
removed once `master` is targeted next minor release (5.1).
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
In 1f006c an option was added called :class to allow passing anonymous
classes to association definitions. Since using :class instead of
:class_name is a fairly common typo even amongst experienced developers
this can result in hard to debug errors arising in raise_on_type_mismatch?
To fix this we're renaming the option from :class to :anonymous_class as
that is a more correct description of what the option is for. Since this
was an internal, undocumented option there is no need for a deprecation.
Fixes #19659
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To me it seems like this should only be the case if `autosave: true` is
set on the association. However, when implemented that way, it caused
issues with has many associations, where we have explicit tests stating
that child records are updated when the parent is new, even if autosave
is not set (presumably to update the parent id, but other changed
attributes would be persisted as well).
It's quirky, but at least we should be consistently quirky. This
constitutes a minor but subtle change in behavior, and therefore should
not be backported to 4.2 and earlier.
Fixes #19782
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I’m renaming all instances of `use_transcational_fixtures` to
`use_transactional_tests` and “transactional fixtures” to
“transactional tests”.
I’m deprecating `use_transactional_fixtures=`. So anyone who is
explicitly setting this will get a warning telling them to use
`use_transactional_tests=` instead.
I’m maintaining backwards compatibility—both forms will work.
`use_transactional_tests` will check to see if
`use_transactional_fixtures` is set and use that, otherwise it will use
itself. But because `use_transactional_tests` is a class attribute
(created with `class_attribute`) this requires a little bit of hoop
jumping. The writer method that `class_attribute` generates defines a
new reader method that return the value being set. Which means we can’t
set the default of `true` using `use_transactional_tests=` as was done
previously because that won’t take into account anyone using
`use_transactional_fixtures`. Instead I defined the reader method
manually and it checks `use_transactional_fixtures`. If it was set then
it should be used, otherwise it should return the default, which is
`true`. If someone uses `use_transactional_tests=` then it will
overwrite the backwards-compatible method with whatever they set.
|
|
|
|
|
|
| |
The test added in 85465ed3e6c582d25f0c8fafe21f7a2c182c2f67 was passing
when the file was run on its own, but failing when the entire suite was
run since this test modifies the class and doesn't clean up.
|
|
|
|
|
|
|
| |
When associations checked for autosave have a cycle, and
none of them is dirty, then changed_for_autosave? will be an
infinite loop. We now remember if we're in the check and
will short circuit the recursion.
|
| |
|
|
|
|
|
| |
`Computer` class needs to be require
See #17217 for more details
|
|
|
|
|
|
|
|
|
|
|
|
| |
068f092ced8483e557725542dd919ab7c516e567 registered autosave callbacks
as `after_save` callbacks. This caused the regression described in #17209.
Autosave callbacks should be registered as `after_update` and
`after_create` callbacks, just like before.
This is a partial revert of 068f092ced8483e557725542dd919ab7c516e567.
Fixes #17209.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
alan/only_save_changed_has_one_objects""
This reverts commit e94e6c27af495a2460c811bb506459f1428dec6b.
Conflicts:
activerecord/CHANGELOG.md
The original commit was reverted only to be safe since #14407 were reported.
We don't have any proof we added a regression with the original commit
so reverting it now will give us more problem.
Closes #14407
|
|
|
|
|
|
|
|
|
| |
This reverts commit 6e3ab3e15faf782f6a937ccf5574a4fb63e3e353, reversing
changes made to 39e07b64ce3f4bb55e60ba0266e677f8e4f4893a.
Conflicts:
activerecord/CHANGELOG.md
activerecord/test/cases/autosave_association_test.rb
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
Save has_one associations only if record has changes
Conflicts:
activerecord/CHANGELOG.md
|
| |
| |
| |
| |
| |
| | |
Prevents save related callbacks such as `after_commit` being
triggered when `has_one` objects are already persisted and have no
changes.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
It should not save the parent record when the nested attributes are
invalid.
Test case to cover #8194.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This commit fixes two regressions introduced in cafe31a078 where
newly created finder methods #second, #third, #forth, and #fifth
caused a NoMethodError error on reload associations and where we
were pulling the wrong element out of cached associations.
Examples:
some_book.authors.reload.second
# Before
# => NoMethodError: undefined method 'first' for nil:NilClass
# After
# => #<Author id: 2, name: "Sally Second", ...>
some_book.first.authors.first
some_book.first.authors.second
# Before
# => #<Author id: 1, name: "Freddy First", ...>
# => #<Author id: 1, name: "Freddy First", ...>
# After
# => #<Author id: 1, name: "Freddy First", ...>
# => #<Author id: 2, name: "Sally Second", ...>
Fixes #13783.
|
|/
|
|
| |
This fixes test order issues.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
commit 02d3b059608c30e98136fde78bc710928f080566
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 30 15:31:39 2013 -0700
habtm works in terms of hm:t
commit 71ac336bbb41f5047a4ee307883a95eca7195742
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 30 15:27:07 2013 -0700
passing before_add callbacks
commit d846a7bf9872a79c3aa8082917abe806278fa159
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 17:57:53 2013 -0700
reducing diff against master
commit 96bd97de47d61a71c368ae367bc59a2dbec3c9ab
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 17:52:35 2013 -0700
fixing more tests
commit 0620399fc231df87c5f08664db1c37e5c1fa5a05
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 17:32:52 2013 -0700
self-referential tables in hm:t are allowed to have the same pk
commit 48eb90e27921d10b6ba3e400ab2c784ed75d5ec4
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 17:03:19 2013 -0700
translating more options
commit 5cace7b2cb546fd6b096543bfc49c4b7197ad21a
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 17:01:02 2013 -0700
handling more hm:t cases
commit 69985ca2cabff2c3f58f5d0a7eb12d7b414c1a01
Merge: d417ec8 3e0a60e
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 27 16:57:12 2013 -0700
Merge branch 'master' into rmhbtm
* master: (21 commits)
adding a test to demonstrate how to use STI subclasses on the far right side of a hm:t association along with preloading.
Fixed grammatical typo in configuring.md guide.
Getting Started Guide: Fix code container in Chapter 5.2 [ci-skip]
Getting Started Guide: Update sentence in Chapter 5.8 [ci-skip]
Add new line after create action on CommentsController code example
Adds template dependencies rake task from cache_digests gem.
[ci skip] Update scaffold output and change some words.
[ci skip] escape unintended url in docs
Getting Started Guide: post.rb -> Post
[ci skip] Add missing periods and update link name and some wording.
quote `false` reference in querying guide.
Getting Started Guide: update link_to string argument to use single-quote mark, following document style [ci skip]
Fix small typo in docs
changelog entry
fix .find when inverse is set
update changelog for #12359
Make sure inverse_of is visible on the has_many callbacks
Getting Started Guide: posts_controller -> PostsController [ci skip]
[ci skip] Correct the explanation of the example for find_or_create_by when used with create_with in ActiveRecord Querying guide
added "id: false" to HABTM join table example
...
commit d417ec82e8f83c32124d1c1a19824d023cfdf015
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 25 16:49:08 2013 -0700
another case is passing
commit 5c68280500962e4b2b6819dd863ebe8b398e5834
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 25 16:21:22 2013 -0700
this seems to work
commit e458c5e55c04a2444e96aca1ff192be42bc4ce7f
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 25 16:13:07 2013 -0700
add another case
commit fc6203b0d49c847b8efb1cc33d358897625f2115
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 25 15:51:45 2013 -0700
delete center records on habtm
commit 9af5156098f6bc8f8ce8eb559a51137960b4938b
Merge: 3a60b03 e2fd64f
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 25 11:33:13 2013 -0700
Merge branch 'master' into rmhbtm
* master: (100 commits)
remove initialize method
extract association resetting to a method
hash insertion order doesn't matter anymore, so only loop over the owners once
always populate the preloaded records instance variable so we can remove the @associated_records_by_owner ivar
keep preloaded records in a list rather than extract from a hash
Getting Started Guide: Hello Rails! -> Hello, Rails! and wrap code tag
push slice loading to it's own method so we can remove the type casting code
Add CHANGELOG entry for #12344
Add regression test to #12343
Fix typo in number_to_human docs: you -> your
guarantee that `klass` is not nil inside the preloader objects
[Documentation] Add a missing validation to I18n docs
Use the given name in html_options for the hidden field in collection_check_boxes
eliminate unused ivar
all records have a preloaded, so eliminate that conditional
eliminate the `loaded?` conditional
push preloaded test up to the factory method so we can eliminate conditionals from the individual preloaded classes
assign_attributes should return if argument is blank.
No need the else clause
Use join to concat the both side of the AST
...
commit 3a60b038a40532397b6c204dfb09d6d43a9336ac
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 17:50:11 2013 -0700
start with a clean slate
commit f30d3631af11ea6144d3ae7a068a7c0072e93a82
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 17:30:59 2013 -0700
make sure the class name goes on the rhs belongs_to
commit f7516b724014504ddb2e706fea1b5438dc5332c3
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 16:31:40 2013 -0700
remove unused variable
commit 61ffc5b9854dc2fe83ee502b17ba8028270ff8a7
Merge: 6cf41cd 460eb83
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 16:25:17 2013 -0700
Merge branch 'master' into rmhbtm
* master:
support objects with blank string primary keys
ActiveRecord::Base#<=> has been removed. Primary keys may not be in order, or even be numbers, so sorting by id doesn't make sense. Please use `sort_by` and specify the attribute you wish to sort with. For example, change:
do what the superclass does in the case that objects do not match
commit 6cf41cd98a82e6f4fe6d868ad323df3d72a9748f
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 15:06:28 2013 -0700
heating up habtm cache
commit d7f6c3aa491f27ba71fdb2b9d0b9d1780664f4dc
Merge: c68c904 56bfd8a
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 18 14:07:51 2013 -0700
Merge branch 'master' into rmhbtm
* master: (58 commits)
Fix an issue where router can't recognize downcased url encoding path.
There's no need to do this
Remove tzinfo dependency from Action Pack
[ci skip] Improve readability of 4.3's NOTE in migration.md.
Removes unused code related to DatabaseTasks.
[ci skip] Consistency wording of 9.6 in form_helpers.md
[ci skip] Update plugins.md
Removing ActiveSupport::Concern, it's not needed
Fixing comment typo in ActionController::Base
Don't require using application_name before options
Collapse where constraints to one where constraint
Custom flash should be defined only for the class that defines it and it's subclasses.
Fix typos: the indefinite articles(a -> an)
Missing destroy command
Update 3_2_release_notes.md
Add CHANGELOG entry for #11698
Add CHANGELOG entry for #12149
Use the Rails binary when generating task
Remove unnecessary loop
"generates" applies to "collection radio" so it should be singular
...
commit c68c904866ef9562c3bd9b54574206a416184414
Merge: 0f5d8e0 71cf717
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 11 17:45:53 2013 -0700
Merge branch 'master' into rmhbtm
* master:
these are not real developer objects, so counting them doesn't make sense. Let's load the object to ensure it's an array and count the array.
Remove conditional adding a new method
Fix inverted conditional
Remove invalid comment
Check if the SQL is not a prepared statement
Whitespaces
Revert "Add meta tag with charset information to application layout."
Avoid empty transaction from setting has_one association on new record.
Reduce Duration#inspect to a single series of transformations
Relation#merge should not lose readonly(false) flag.
Reduce allocations when extracting AR models
Perf: avoid dupes add fallback logic for coders
commit 0f5d8e0febd3128cf4121ff36f1764b9284d9f7d
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 11 11:43:29 2013 -0700
everything works with extensions
commit d003c103b5908fb3a6427f39bddd1748ef2c2576
Merge: 5768c38 7e0cac1
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Wed Sep 11 11:23:59 2013 -0700
Merge branch 'master' into rmhbtm
* master:
fix deleting join models with no pk
remove sentence err
[ci skip] 'previous version of Rails' is gramatically incorrect
Add meta tag with charset information to application layout.
add a comment for sanity of other people to come
commit 5768c38d53fd66a97814faaea8e07c70722b310f
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Tue Sep 10 15:11:05 2013 -0700
habtms with a scope seem to be working
commit 5ee9108d95c544d4befd682a72139383d0780d68
Merge: d5478e6 e64b5da
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Tue Sep 10 11:55:48 2013 -0700
Merge branch 'master' into rmhbtm
* master:
ask the association for records rather than calling `send`
commit d5478e64bbf80337ec35462368edabb373feeb74
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Tue Sep 10 11:54:39 2013 -0700
ask the association for records rather than calling `send`
commit 93020bc1ad51363c3f400370f91c9494690dcea8
Merge: 11b3d5f d68419a
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Tue Sep 10 11:30:03 2013 -0700
Merge branch 'master' into rmhbtm
* master:
Use Ruby 2.0 caller_locations instead of caller if available
Update Rails 3.2.x guide link [ci skip]
Be sure to restore the default I18n.locale after changed its value in a test
Fixes typo in Object#try!
More unused associations in AR test models
:scissors: [ci skip]
change function def self.table_name to self.table_name
Clean up unused associations in AR test model
Reset ActionView::Base.logger instead of AC::Base.logger
Refactor handling of action normalization
Don't mutate the Base settings by merge!ing the given value
Make AC standalone rendering work
use assert_empty in activemodel conditional validation test cases
Removed unused modules and classes
Removed unnecessary require
Remove helper fixtures not used in any test
Back AV description in gemspec
Fix order dependent test
grab executable from rubygems
Fixed API task file
commit 11b3d5fa45b57fc4e7dddb09be583498b120b185
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 16:19:48 2013 -0700
change query count since we are using hm:t associations
commit f59daebedab3ed13f31c99244ff71a4a5d6e554b
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 15:35:07 2013 -0700
delegate compute_type to a real AR class
commit c84a40d2ed76e5843b994c5a2b9e29ced3816511
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 15:29:51 2013 -0700
define callbacks with the builder
commit d08b1b6b3efc9ed8b0d5476892f048fbffb39e40
Merge: acebec1 0c5d0be
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 15:17:40 2013 -0700
Merge branch 'master' into rmhbtm
* master:
let the class cache object clean up user input
make @bitsweat happy. :heart:
commit acebec128e108ac2b4855e540d8764629670cb83
Merge: 2de68a4 e1cbd42
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 14:47:30 2013 -0700
Merge branch 'master' into rmhbtm
* master:
stop using deprecated api in the tests
commit 2de68a464641f76067743957f889ac87dff395a0
Merge: 7504df9 1385ae1
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 14:34:37 2013 -0700
Merge branch 'master' into rmhbtm
* master:
Remove BasicRendering tests
Remove remaining coupling with AV in MimeResponds
Remove BasicRendering and remove template functionality from AbsC::Rendering
Improves a sentence in guides/security
[ci skip] Change link name of Rails i18n wiki.
Typo in Changelog.
Fix fixtures regression that required table names to map to classes only, not class names
Use MiniTest::Unit::TestCase instead of Minitest::Test
Use Ruby on Rails Coding Conventions for code examples in the guides
commit 7504df92f21ed3d5da7c2760d760f026728ed04d
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Mon Sep 9 14:34:02 2013 -0700
fake class name should be a valid class name
commit 6609620ea86dc0fb7c4bbfb0db950f6e3fc75b56
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:52:03 2013 -0700
move another habtm
commit 2c95a36e2c3dfe92f2930f3cca44bc4452732a23
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:50:18 2013 -0700
use the habtm name to generate the rhs name on the join model
commit bd963f720b1db19b0bec186bc33bef9203d8b011
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:47:29 2013 -0700
don't hardcode the name
commit 461759e2caf66f23dca4eff988648bf769a2b533
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:45:27 2013 -0700
we do not need to specify the fk
commit 9c223f01db6e36adbb7570e2aa1bcaec1d142c87
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:44:08 2013 -0700
just call the left side :left_side
commit 5661622a82154eff877fe0993bfffad13dacad7a
Author: Aaron Patterson <aaron.patterson@gmail.com>
Date: Fri Sep 6 15:40:36 2013 -0700
initial habtm implementation is working
|
|
|
|
| |
Closes #11450
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
fixes bug introduced by #3329
These are the conditions necessary to reproduce the bug:
- For an association, autosave => true.
- An association record is being destroyed
- A new association record is being created.
- There is a unique index one of the association's fields.
- The record being created has the same value as the record being
destroyed on the indexed field.
Before, the deletion of records was postponed until after all
insertions/saves. Therefore the new record with the identical value in
the indexed field caused a non-unique value error to be thrown at the database
level.
With this fix, the deletions happen first, before the insertions/saves.
Therefore the record with the duplicate value is gone from the database
before the new record is created, thereby avoiding the non-uniuqe value
error.
|
| |
|
|
|
|
|
| |
This reverts commit 637a7d9d357a0f3f725b0548282ca8c5e7d4af4a, reversing
changes made to 5937bd02dee112646469848d7fe8a8bfcef5b4c1.
|
| |
|
| |
|
|
|
|
| |
This reflects the fact that it now impact inserts as well as updates.
|