<p>Leaving aside the discussion whether DBIC should disallow "ugly" use cases or not - I think you are running the test on an old version of DBIC or something like this.<br>
Bottom line is - no fix can go in without a test case, for 2 reasons.<br>
1) There is no other definitive way to verify that the patch indeed does what it is supposed to do (i.e. <code>prove -l test.t</code> should pass <code>prove test.t</code> should fail)<br>
2) There is no other definitive way to ensure a different edit of the code in question later on will not reintroduce the same breakage</p>

<p>I already did the tricky part for you in <a href="https://github.com/dbsrgits/dbix-class/commit/bd35f5fa3" class="commit-link"><tt>bd35f5f</tt></a>. Please modify this particular test <a href="https://github.com/dbsrgits/dbix-class/blob/master/t/resultset/update_delete.t#L326">https://github.com/dbsrgits/dbix-class/blob/master/t/resultset/update_delete.t#L326</a> (lifted from your comment above) so it actually <em>fails</em> without your patch. Then we can talk about the correct way of fixing this.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br>Reply to this email directly or <a href='https://github.com/dbsrgits/dbix-class/pull/18#issuecomment-14612940'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/4W3BWTo7EIi_7vx28Xzl_gG6ChOv85OqWofxoMwLyWuSlqgzOh1YfrZQLjQmVh4H.gif' height='1' width='1'></p>