Problem/Motivation

src/Plugin/Field/FieldWidget/BootstrapDatepickerBase.php calls deprecated entity.manager which breaks the widget upon selection.

This needs an update to use entity_type.manager

CommentFileSizeAuthor
#2 bootstrap_datepicker-fix-deprecated-manager-3220495-2.patch665 bytesgreg.zielinski

Issue fork bootstrap_datepicker-3220495

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

greg.zielinski created an issue. See original summary.

greg.zielinski’s picture

greg.zielinski commented
StatusFileSize
new bootstrap_datepicker-fix-deprecated-manager-3220495-2.patch665 bytes

This should fix it

imre.horjan’s picture

imre.horjan
Primary language Hungarian
Location Hungary
commented
Title: Replace depracated EntityManager with EntityTypeManager » [D9] Replace depracated EntityManager with EntityTypeManager
Version: 8.x-1.0-beta3 » 8.x-1.x-dev
Status: Active » Reviewed & tested by the community
Issue tags: +Drupal 9 compatibility

Works for me using the 8.x-1.x-dev branch.
Looks like RTBC

imre.horjan opened merge request !3

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: bootstrap_datepicker-fix-deprecated-manager-3220495-2.patch, failed testing. View results

imre.horjan’s picture

imre.horjan
Primary language Hungarian
Location Hungary
commented
Status: Needs work » Reviewed & tested by the community

I've just created a Merge Request for this one as well.

imre.horjan’s picture

imre.horjan
Primary language Hungarian
Location Hungary
commented

Hi Martijn,
This code change is included in the latest patch of https://www.drupal.org/project/bootstrap_datepicker/issues/3147249#comme...
I couldn't find that issue in the queue because that's closed already.
Could you please merge this or apply the patch provided in the D9 compatibility issue?
Many thanks

  • imre.horjan committed 402faaa on 8.x-1.x 
    Issue #3220495 by greg.zielinski: Replace depracated EntityManager with...
martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Status: Reviewed & tested by the community » Fixed

Thank you for all the work you are doing Imre !

Martijn de Wit credited Tomefa.

Martijn de Wit credited akhoury.

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented

added credits from #3147249: Drupal 9 compatibility

Status: Fixed » Closed (fixed)

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