<blockquote>
<p>I am especially wary of the removal of 'filter'<br>
it is not removed. It were merged as noted here (see -247): ## 'filter' should disappear and get merged in with 'single' above!</p>
</blockquote>

<p>Actually, in code, I have changed, there are few point we must pay attention:<br>
1. Create related/child row only after parent (see +255, +272-280)<br>
2. Filter obj is placed into $inflated hash (-258). In new code that done here: +274, +278<br>
3. create_related_rows for 'multi', 'filter', 'single' is same. Except:<br>
 a) 'multi' can not be 'in_storage' otherwise exception: +197<br>
 b) +200-206 are same for 'single' and 'filter' except 'set_from_related' are executed only for 'single'<br>
 c) Now create_related_rows return one object or array of objects. I repeat that behaviour, BUT for future clean code you must not destinguish one or many in both cases must be arrayref</p>

<p>MY RESOLUTION: code is safe and you must not worry about it ;-) <strong>Merge or not it is your decision.</strong> Look at test - all passed.</p>

<blockquote>
<p>test for almost every behavioral change.<br>
This is not behavioral change. It is leaved same. It is better to say this patch is BUGFIX for not noted yet bug: <strong>You can not add slave rows without master row not exist!!!</strong> But code, I have fixed, violate this DB integrity rule.</p>
</blockquote>

<p>And actually this patch does not resolve my issue, but this one step to fixit. As I noted here:<br>
<a href="https://rt.cpan.org/Public/Bug/Display.html?id=104375#txn-1495569">https://rt.cpan.org/Public/Bug/Display.html?id=104375#txn-1495569</a> I loose clue. When I try to get access to parent/master row from children the code at ResutlSource:sub _resolve_condition:1773<br>
return wrong condition UNRESOLVABLE_CONDITION, maybe because it do not know that row in memory yet. And I do not understand the terrible code there.</p>

<blockquote>
<p>Please, if time permits adapt the test from<br>
Yes, I will do that. But in other pull request. As I mentioned this patch does not resolve my issue.<br>
Thank you for your examples and links. They help me. ;-)</p>
</blockquote>

<p>PS. I had desire to report about some method renaming and removing, about removing of some part of documentation and add doc for well formed methods (that were renamed). But I guess you will reject that as mine other proposal. So I threw out this idea. It's a pity.</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/78#issuecomment-102372876">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AASeAgr8ybx2WUuIgBYndWoMwXa1T-Pyks5oJc3tgaJpZM4EYUtO.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
  <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
    <link itemprop="url" href="https://github.com/dbsrgits/dbix-class/pull/78#issuecomment-102372876"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>