Hi killroyboy,

I'll have to mark this ticket as "works for me". I'm unable to reproduce the issue on svn branch 1.0.0.

I added coverage to all possibilities I've imagined, without success. Coverage to this ticket was added in r4870. Your ticket reports the same issue of ticket #1381. Please provide a failing test case, otherwise I cannot address the issue.

Regards,

What is the importance of the cleanData method in the Doctrine_Record::load method? And why was it added in RC1? That is what broke my code. If I comment out the cleanData method, everything works fine.

Working on a failing test case.

I think I've figured out why cleanData caused the problem, but don't know how to fix it. Apparently, $this->_data isn't supposed to contain my calculated fields. They should be in $this->_values.

I do have _some_ "calculated" fields that are working. It appears that the ones that are not working are the ones that are manipulated/added in external functions where I pass the $event->data array to a function that is within a non-doctrine class and then return the data array. I've tried passing by reference as well as value (and returning the array into $event->data). Neither one seems to result in the calculated fields ending up in the $_values array inside the object.

Any ideas?

I can't figure out the failing test case. Everything seems to be working as expected when I build a test case. So there is something I am doing in my code that is making it not work.

Coverage for #1395 -- can't get it to fail

I think you should attach your code and let's see if we can find what's going on.

Cheers,

I would love to do that, but I don't think my employer would like it too much.

The heart of the issue is how fields get from the $_data array into the $_values array. Because the $event->data array properly gets updated even after the listener. cleanData then strips out the fields not set directly inside the preHydrate.

What makes Doctrine move data from $_data to $_values?

Milestone New deleted

If you can't attach your code can you create a failing test case with the same type of code to replicate the problem?

That's the problem. I can't get the test case to fail. I don't know what my code is doing differently. I've tried several approaches to mimic my code, but nothing seems to be working.

Until I can figure this out, we are stuck on BETA-1 (unless I comment out the cleanData line in Doctrine_Record::load)

Roman, I added coverage for this in r4975.

Can you clarify why the cleanData is needed? When I remove it, all the tests still pass. Just leave your comments and then re-assign it back to me.

In r4995 this was fixed. Thanks for the report and sorry for the regression/inconvenience.

If the tests pass i think this change is fine.