CommentFileSizeAuthor
#2 3097883.patch21.13 KBlongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Active » Needs review
FileSize
3097883.patch21.13 KB
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented

Should we include Drupal:l() and Drupal::url() here as well? Not technically in that component but closely related.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented

@Berdir those were already done in #3097534: Remove all @deprecated code in \Drupal static service container

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

2 traits and interfaces are deprecated 2 years ago, suppose it not very destructive

+++ /dev/null
@@ -1,32 +0,0 @@
- * Part of the deprecation means that applies() is now called on runtime instead
- * of compile time.

+++ /dev/null
@@ -1,90 +0,0 @@
- * @deprecated in drupal:8.0.0 and is removed from drupal:9.0.0.
- *   Use \Drupal\Core\Link instead.

+++ /dev/null
@@ -1,29 +0,0 @@
- * @deprecated in drupal:8.4.0 and is removed from drupal:9.0.0. Instead,
- * you should use \Drupal\Core\Routing\FilterInterface.

+++ /dev/null
@@ -1,106 +0,0 @@
- * @deprecated in drupal:8.0.0 and is removed from drupal:9.0.0.
- *   Use \Drupal\Core\Url instead.

Check and no other mentions of this removals left

  • catch committed 7031d8f on 9.0.x 
    Issue #3097883 by longwave, andypost: Remove all @deprecated code in \...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed 7031d8f and pushed to 9.0.x. Thanks!

Status: Fixed » Closed (fixed)

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