]> Dogcows Code - chaz/p5-DBIx-Class-ResultSet-RecursiveUpdate/commit
partial fix for #69624
authorCharles McGarvey <chazmcgarvey@brokenzipper.com>
Thu, 22 Mar 2012 17:03:16 +0000 (11:03 -0600)
committerCharles McGarvey <chazmcgarvey@brokenzipper.com>
Thu, 22 Mar 2012 17:03:16 +0000 (11:03 -0600)
commit982b1ba172104d6cd67807e78dddbe71b3074394
tree254f0f689bb163f283a6fa0a09dbf964ac6fe8bd
parent7a1370a25e46f41b28349f9fb225e3b8caa6791c
partial fix for #69624

This uncomments a call to _delete_empty_auto_increment.  True to its name,
this function removes NULL auto-increments from the column info so that
they are not INSERTed.  The call to this method was commented out in
93ad3243, but I cannot find anyone who remembers why, and re-enabling the
call does not break any other tests on postgres, mysql, or sqlite.

However, this change alone does not cause the test to pass; there are still
issues with databases which enforce foreign constraints (tested on
postgres, mysql, and sqlite with the foreign_keys pragma).  I'm not sure if
the test itself is broken, but maybe it should at least be marked as TODO.
lib/DBIx/Class/ResultSet/RecursiveUpdate.pm
This page took 0.024095 seconds and 4 git commands to generate.