Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove backwards compatibility layer in GroupSubscribeForm #397

Closed
3 tasks
pfrenssen opened this issue Jul 3, 2018 · 0 comments · Fixed by #530
Closed
3 tasks

Remove backwards compatibility layer in GroupSubscribeForm #397

pfrenssen opened this issue Jul 3, 2018 · 0 comments · Fixed by #530

Comments

@pfrenssen
Copy link
Contributor

pfrenssen commented Jul 3, 2018

Do not work on this before Drupal 8.6.0 is released, this is planned for September 2018

In Drupal 8.6.0 a change was made to the ContentEntityForm class: one of the constructor arguments was changed from EntityManagerInterface to EntityRepositoryInterface.

In #395 a workaround was added in GroupSubscribeForm (which extends ContentEntityForm) to ensure that the form is compatible both with Drupal 8.6 and earlier versions without throwing deprecation warnings.

Once Drupal 8.5.x is deprecated we do not need this B/C layer any more and it can be removed.

  • Add the EntityRepositoryInterface type hint to the $entity_repository argument in GroupSubscribeForm::__construct(), and update the docblock.
  • Remove the B/C layer from GroupSubscribeForm::create() and update the documentation.
  • Update the version requirement of Drupal core to ~8.6 in composer.json.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant