Content Lock should typehint ModuleHandler interface rather than a specific implementation

Posting the static patch because using the MR doesn't allow pinning to a specific commit, so anyone can submit pretty much anything and inject it into our codebase IIRC

#12 works for me.

Thanks.

Thanks for the patch

The time service in the Contentlock class should be added to make the patch complete:

 /**
   * The time service.
   *
   * @var \Drupal\Component\Datetime\TimeInterface $time
   */
  protected $time;

I added the missing property.

Given these changes touch a lot of files it'd be great to get them in to avoid any nasty rerolls.

+1 : Lets get this merged!

Following the comment in #12, here is a new static version of this patch.

Fixed up the MR to adhere to coding standards.

I'm not 100% convinced that everything needs an interface - like a swappable content lock service sounds like a nightmare to me and even supporting decoration is interesting. But my concerns are minor put against the fact the module is incompatible with modules that decorate well known core services.

Automatically closed - issue fixed for 2 weeks with no activity.