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.
------ -----------------------------------------------------------------------------------
Comment | File | Size | Author |
---|---|---|---|
#6 | vat_number-3073409-6-for-lenient-composer.patch | 1.27 KB | daften |
#6 | vat_number-3073409-6.patch | 1.17 KB | daften |
| |||
#2 | vat_number-3073409-1.patch | 912 bytes | joy29 |
Comments
Comment #2
joy29 CreditAttribution: joy29 commentedpatch for drupal 9 compatibility
Comment #3
joy29 CreditAttribution: joy29 commentedComment #4
daften CreditAttribution: daften at Dropsolid commentedLooks good
Comment #5
maartendeblock CreditAttribution: maartendeblock commentedI can also confirm the patch works.
Comment #6
daften CreditAttribution: daften at Dropsolid commentedI'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.
Comment #8
tim-dielsThanks all, will include the needed credits from other issue.
Comment #9
tim-dielsFixed and will be released shortly.