Needs tests: Support cloning

I'm not sure if a field currently can even act on a clone. createDuplicate certainly doesn't invoke a method on fields.

node_clone might add a hook but then it only works with that.

Whatever we want to support is fine with me, just saying that the core API alone won't help us get there. I see @dawehner did a replicate D8 port. afaik node_clone is being worked on too.

Definitely interested to see these working together for a site I'll theoretically be working on soon-ish, hopefully I (or a coworker) will be able to help out make it work.

FYI there's WIP for a D8 port of Node Clone, and there's the newer Entity Clone module that has dev version available for it.

Attached patch provides integration with Replicate module. Test will fail since the D8 port was not pushed to the drupal.org yet, but it passes locally.

The tests will also fail because it can't find the Replicate module. In D7 you could just add a test_dependency[]= line to the info file, is that possible with D8?

Done.

This should fix the tests. Also confirmed uncommitted Replicate test is still passing.

Replicate integration test somehow sneaked into the commit.

Can we try what the commit suggests? (@requires module replicate annotation) If that works, then it will automatically start to work as soon as replicate is on d.o.

And it will also not fail locally when running all tests and you don't have that module.

Not sure if it will work, I think it's still broken with simpletest but maybe it works with phpunit...

Tried locally and it failed both with run-tests.sh and phpunit. Does patch look correct?

Discussed this with @Berdir and he suggested slightly different approach. This does look much nicer indeed.

+++ b/src/ParagraphsServiceProvider.php
@@ -0,0 +1,31 @@
+    $modules = $container->getParameter('container.modules');
+    // Check for installed REST & HAL modules, as HAL requires REST.
+    if (isset($modules['replicate']) ) {

:)

Dammit!

Now it looks fine to me.

We will still need to commit the test ;)

Test only patch.

Replicate has been pushed to d.o.

Search api broken, I think we have to wait on the next release, monitoring also has search api test fails.

The new test is passing, so it's not a problem with this issue. Didn't really review, so just needs review, not RTBC.

Yup, tests failed on HEAD, which proves #62.