lcars's profile

lcars commented at 3:49 PM
I agree changing it's backwards incompatible and really problematic. I just wonder if #attributes= is expected to behave the same (I'm not saying it shouldn't),...
lcars commented at 3:46 PM

Sorry, my test wasn't focused on the specific issue. Is it then also expected for #attributes= ti save records?

def test_assign_ids_via_attributes_without_savin...
lcars commented at 3:38 PM
added this test to autosave_association_test.rb

def test_assign_ids_without_saving
```
firm = Firm.new("name" => "Apple")
firm.save
firm.reload
firm.client_ids = [...
```
lcars commented at 1:06 PM
Not sure how to fix the .attributes = case. I assume that it's implied in the documentation that foo.bar_ids = immediately replaces the collection. Being this t...
lcars commented at 11:52 AM
Other than validation concerns, is .attributes= expected to update associations before .save ? It is not clear from documentation that .attributes implies .save...

lcars commented at 11:25 AM
Working on a patch right now, I should also note that I would treat this issue as a security bug as it allows validation to fails but still to update associatio...
lcars commented at 10:59 AM
This bug still affects attributes= method. It should be fixed there as well.

lcars created the ticket at 11:10 AM

Hi, we are experiencing random 'undefined method' errors while creating objects in an heavily threaded environment, here's an example:

caught exception in vendo...

lcars created the ticket at 6:13 PM

Hi,

The pasted code (around line 1337 of lib /active_record/associations.rb in activerecord-2.3.8) prevents the usage of non numeric ids (which afaik is perfect...