Problem/Motivation

drupal-check results on commit hash: e9b3b4de22140de73b96093d0f19629aa930d9fe



 ------ ------------------------------------------------ 
  Line   image_url_formatter.module                      
 ------ ------------------------------------------------ 
  66     Call to deprecated method l() of class Drupal.  
 ------ ------------------------------------------------ 

 ------ ---------------------------------------------------------------------------------- 
  Line   src/Plugin/Field/FieldFormatter/ImageUrlFormatter.php                             
 ------ ---------------------------------------------------------------------------------- 
  221    Call to deprecated method urlInfo() of class Drupal\Core\Entity\EntityInterface.  
 ------ ---------------------------------------------------------------------------------- 

 [ERROR] Found 2 errors

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#5 interdiff_3_5.diff1.36 KBnikolas.tatianenko
#5 drupal_9_deprecated_code-3042736-5.patch7.72 KBnikolas.tatianenko
#3 drupal_9_deprecated_code-3042736-3.patch6.89 KBSergiu Stici
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mcdwayne created an issue. See original summary.

Sergiu Stici’s picture

Sergiu Stici CreditAttribution: Sergiu Stici at FFW commented

..

Sergiu Stici’s picture

Sergiu Stici CreditAttribution: Sergiu Stici at FFW commented
Status: Active » Needs review
FileSize
drupal_9_deprecated_code-3042736-3.patch6.89 KB

Here is the patch, please review.

Dinesh18’s picture

Dinesh18 CreditAttribution: Dinesh18 at Singapore Press Holdings commented
Status: Needs review » Reviewed & tested by the community

Manually reviewed the patch and it covers the two errors which are mentioned in the descriptions.
Along with uses a short syntax array which is really good.
+1 to RTBC

nikolas.tatianenko’s picture

nikolas.tatianenko CreditAttribution: nikolas.tatianenko at EPAM Systems commented
Status: Reviewed & tested by the community » Needs review
Issue tags: +Drupal 9 porting weekend, +epam-contrib
FileSize
drupal_9_deprecated_code-3042736-5.patch7.72 KB
interdiff_3_5.diff1.36 KB

Checking with latest Drupal 9-rc1 and rector
Rector 0.8.x-dev@8896f0a
Config file: rector.yml

6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] Rector is done!

nikolas.tatianenko’s picture

nikolas.tatianenko CreditAttribution: nikolas.tatianenko at EPAM Systems commented

Checking with latest Drupal 9-rc1 and rector
Rector 0.8.x-dev@8896f0a
Config file: rector.yml

6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] Rector is done!

alexandra.vecher’s picture

alexandra.vecher CreditAttribution: alexandra.vecher at EPAM Systems commented
Status: Needs review » Reviewed & tested by the community

I'v checked.
Rector 0.8.x-dev@3465583
Config file: rector.yml

6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] Rector is done!

g089h515r806’s picture

g089h515r806 CreditAttribution: g089h515r806 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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