5/5 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ -----------------------------------------------------------------------------------
Line src/Controller/VatNumberController.php
------ -----------------------------------------------------------------------------------
78 Call to deprecated method strtoupper() of class Drupal\Component\Utility\Unicode:
in Drupal 8.6.0, will be removed before Drupal 9.0.0. Use
mb_strtoupper() instead.
78 Call to deprecated method substr() of class Drupal\Component\Utility\Unicode:
in Drupal 8.6.0, will be removed before Drupal 9.0.0. Use
mb_substr() instead.
79 Call to deprecated method strtoupper() of class Drupal\Component\Utility\Unicode:
in Drupal 8.6.0, will be removed before Drupal 9.0.0. Use
mb_strtoupper() instead.
79 Call to deprecated method substr() of class Drupal\Component\Utility\Unicode:
in Drupal 8.6.0, will be removed before Drupal 9.0.0. Use
mb_substr() instead.
------ -----------------------------------------------------------------------------------

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joy29 created an issue. See original summary.

joy29’s picture

patch for drupal 9 compatibility

joy29’s picture

Assigned: joy29 » Unassigned
Status: Active » Needs review
daften’s picture

Status: Needs review » Reviewed & tested by the community

Looks good

maartendeblock’s picture

I can also confirm the patch works.

daften’s picture

I've created two patches, one that encompasses #3129833 and can be applied to the branch. A second one that will apply with composer-patches and using the new lenient composer repository (see https://www.drupal.org/docs/develop/using-composer/using-drupals-lenient-composer-endpoint). This way it can be easier to include the module in a D9 upgrade.

I'm leaving the issue in RTBC, since nothing really changed.

tim-diels’s picture

Thanks all, will include the needed credits from other issue.

tim-diels’s picture

Status: Reviewed & tested by the community » Fixed

Fixed and will be released shortly.

Status: Fixed » Closed (fixed)

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