Replace DateFormatter with DateFormatterInterface in DI

Open on Drupal.org →

Created on 3 January 2024, over 1 year ago

Updated 22 March 2024, about 1 year ago

Problem/Motivation

Hi, when decorating the date.formatter service, this module stops working because it expects a specific class in dependency injection.

Steps to reproduce

Define a decorator for the date.formatter service, then go to /admin/people/profiles

Proposed resolution

Replace the DateFormatter class with the DateFormatterInterface interface in dependency injection.

🐛 Bug report

Status

Fixed

Version

1.0

Component

Code

Created by

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!22Replace DateFormatter with DateFormatterInterface in DI
Closed
Unnamed author
updated about 1 year ago

Comments & Activities

Issue created by @kksandr
Merge request !22Issue #3412148 by kksandr: fix dependency injection → (Closed) created by Unnamed author
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7
last update over 1 year ago
23 pass
Status changed to Needs review over 1 year ago6:37pm 3 January 2024
Comment over 1 year ago →
kksandr
Comment about 1 year ago →
System Message

jsacksick → committed d9a238d7 on 8.x-1.x authored by kksandr →
Issue #3412148 by kksandr: Replace DateFormatter with...
Status changed to Fixed about 1 year ago8:01am 13 February 2024
Comment about 1 year ago →
🇮🇱Israel jsacksick
Committed, thanks!
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.
Comment about 1 year ago →
System Message
kksandr → closed merge request !22
Comment about 1 year ago →
System Message
kksandr → reopened merge request !22
Comment about 1 year ago →
System Message
kksandr → closed merge request !22

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024