silverstripe-big-o/sapphire — comment on commit 64db811d653d6e2de2c83f9f36b0f8168a964555
File:  line 0
By sminnee:

I thought we had already fixed this. In any case, looks good.

The fact that `merge()` modifies the object itself brings up another point, though: shouldn't merge and other manipulators return a new object rather than modifying an existing one?

Then `$pages = $pages->merge($livePages)` would be easy to implement on DataList (albeit still inefficient).