Change author on clone

You can subscribe to the POST_CLONE event and update the author to the one that is logged in.

  public static function getSubscribedEvents() {
    $events[EntityCloneEvents::POST_CLONE] = ['updateClone'];

    return $events;
  }

  public function updateClone(EntityCloneEvent $event) {
    $clone = $event->getClonedEntity();

    // Set UID to current user.
    $clone->uid = \Drupal::currentUser()->id();
    $clone->save();
  }

Automatically changing the author on cloning will be required to give the user full access.
Otherwise you have to change the author manuelly as administrator.

Please make this a part of the settings of this module - this would really help a lot!

I've added a patch with the following concept behind this:
- I add permissions "allow user to take ownership of cloned entity" to all the content types that have the ownership trait (meaning they can have owners)
- If user has this permission he will see a checkbox on the clone form, if ticked, the owner will be changed to drupal's current user.

* Note: patch will only apply to the latest dev version, because of some other changes on the form in a commit before. Doesn't apply to beta 3.

Reuploading patch because of a copy&paste mistake when creating patch.

There were some issues with the patch, clone form was checking for incorrect permissions based on entity bundle instead of type.

I also fixed the TypeError with injected AccountProxyInterface service, replaced deprecated FILE_EXISTS_RENAME constant and minor fixes to method descriptions.

I have applied and tested the patch. It applied without errors and worked as described from maticb. I'm changing status to "Reviewed".

Added an extra checkbox to the entity_clone settings:
Whether the "Take ownership" option should be checked by default on the entity clone form.

Hi. I have also applied and tested patch from #8. It applied without errors and worked as described. I'm changing status to "RTBC".

Rerolling #8

I'd have more confidence in committing this if someone could test the reroll.

I had changed my mind and was about to merge this because tests are passing, but then I noticed the code duplication. EntityClonePermissions::entityTypeHasOwnerTrait() is basically identical to EntityCloneForm::entityHasOwnerTrait(). Why not simply have the latter just get the type and then pass it to the former? There's no need to duplicate the entire thing.

Updated patch.

Hello, I just tried applying the patch #16 with

git apply 3078811-16.patch

and it failed with the following errors:

error: patch failed: src/EntityClone/Content/ContentEntityCloneBase.php:32
error: src/EntityClone/Content/ContentEntityCloneBase.php: patch does not apply

I believe it needs a reroll.

Re-rolled patch.

Re-rolled patch, please review. It's basically the same that #18 re-rolled, but I renamed some classes for clarity.

Needs review for #18

Hello, I have just checked patch #18. Things are in order after the reroll. and besides that, it seems to work fine. I'm moving it to RTBC

Thanks all!