This project is archived and is in readonly mode.

association_collection#create acts as association_collection#create!

#2934

ActiveRecord has_many :through's association_collection#create seems to act as association_collection#create!, raising errors instead of merely building the AR object instance with the appropriate errors.

  #example
  http://github.com/stefanpenner/association_collection_create_bug/

Example

  #results in an error being raised.
  group.users.create(..with..invalid..user..attributes..)

Specs

  #Failing specs are included.
  rake

Workaround

  #Hackish Workaround.
  @user = group.users.create(..with..invalid..user..attributes..) rescue     User.build(..with..invalid..user..attributes..)

Reported by stefanpenner · July 21st, 2009 @ 09:13 PM

Activity

Michael Koziarski
- Tag changed from activerecord to activerecord, bugmash
August 3rd, 2009 @ 06:17 AM
railsbob
Verified

I was able to reproduce this error with the help of mentioned example.

August 8th, 2009 @ 12:21 PM
railsbob
I have attached a patch for the failing test case.
- failing_test_association_collection_create.diff 4.58 KB
August 8th, 2009 @ 06:01 PM
railsbob
The following patch fixes the error.
- fixed_association_collection_create_exception.diff 7.07 KB
August 8th, 2009 @ 06:15 PM
Dan Croak
-1 patches do not apply cleanly in 2-3-stable

August 8th, 2009 @ 08:09 PM
José Valim
Just to add a notice, I asked @railsbob to rebase the patch and improve test coverage. Here is a example of tests for has_many: http://pastie.org/576732

August 8th, 2009 @ 08:16 PM
railsbob
Sorry, please ignore the previous patch (whitespace error). This one applies cleanly.
- fixed_association_collection_create.diff 5.82 KB
August 8th, 2009 @ 09:04 PM
railsbob
As discussed with lifo, the previous patch is modifying << method. I am attaching another patch, which removes << from create method.
- 2934_fixed.diff 6 KB
August 8th, 2009 @ 11:48 PM
Rizwan Reza
verified

+1 The patch works cleanly on master but does not in 2-3-stable.

August 8th, 2009 @ 11:54 PM
Dan Croak
verified 2934_fixed.diff does not apply cleanly to 2-3-stable.

August 10th, 2009 @ 02:18 AM
Dan Croak
I've attached a patch that works for 2-3-stable. It was modified from railsbob's 2934_fixed.diff patch for master. I don't think it is possible to have a single patch that will work for both, at least without serious refactoring of one of the tests.
- 2934-association_collection-create-no-longer-acts.patch 6.1 KB
August 10th, 2009 @ 02:33 AM
Jeremy Kemper
- State changed from new to committed
- Tag changed from activerecord, bugmash to activerecord
- Milestone cleared.
August 10th, 2009 @ 02:52 AM
Repository

(from [33c054d7e04672201e120a0f8da38166650f3a94]) has_many :through create should not raise validation errors

[#2934 association_collection#create acts as association_collection#create! state:committed]

Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/33c054d7e04672201e120a0f8da381...

August 10th, 2009 @ 02:52 AM
Repository

(from [e06a0b03c8ba29f4b05a35560645814ac88aefbe]) has_many :through create should not raise validation errors

[#2934 association_collection#create acts as association_collection#create! state:committed]

Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/e06a0b03c8ba29f4b05a3556064581...

August 10th, 2009 @ 02:52 AM