Ran drupal-check against f1dfc7c.

 ------ --------------------------------------------------------------------------
  Line   src/PluginForm/AcceptJs/PaymentMethodAddForm.php
 ------ --------------------------------------------------------------------------
  287    Call to deprecated method set() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method set() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method set() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method setExpiration() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method setId() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method setIssuedAt() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method setIssuer() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
  287    Call to deprecated method sign() of class Lcobucci\JWT\Builder:
         This method will be removed on v4
 ------ --------------------------------------------------------------------------
CommentFileSizeAuthor
#3 3118118-3.patch1.52 KBjsacksick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

FatGuyLaughing created an issue. See original summary.

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Issue tags: -Deprecation Removal +deprecated
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Active » Needs review
FileSize
3118118-3.patch1.52 KB

Attached patch should fix the deprecations.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented

To test, we need to setup Cardinal Commerce and make sure that fraud detection is available.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs review » Fixed

Committed.

  • jsacksick committed 810d20d on 8.x-1.x 
    Issue #3118118 by jsacksick, FatGuyLaughing: Lcobucci JWT Code...
bradjones1’s picture

bradjones1
Primary language English
CreditAttribution: bradjones1 at Not Vanilla, Inc. commented
Status: Needs review » Fixed

Not sure why the status didn't stick.

Status: Fixed » Closed (fixed)

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