Rob, yup, I saw the commit. Been doing a lot of testing today to make sure those changes don't fubar anything else, so far it looks good.
BTW, I noticed a logic problem a few weeks ago with the way I originally implemented the merging. Basically, it ignores repeated data, i.e. data which is the same as the first row in the set of rows being merged.
This is a Bad Thing. For instance, if you have an element's set of joined data which was:
England
France
England
Germany
Italy
... after merging, it'll be ...
England
France
Germany
Italy
I don't know wtf I was thinking when I wrote that code. Anyway, we need to rethink the logic for how we actually merge the data down, probably by comparing the foreign key on the related data to the selected 'from element' param on the parent's join data.
My only excuse is I wrote that WiP merging coding back in 1.0.x days when I was MUCH less familiar with Fabrik than I am now!
-- hugh