Not working with commerce v2.17

The $current_user persists already.
The problem is coming from a missing module dependency in .info.yml file:

+  - commerce_payment:commerce_payment

I create a patch for.

I'm not sure how to create patches, but i think i've found something that works.

In /commerce_paypal/src/Plugin/Commerce/PaymentGateway/ExpressCheckout.php,

append $container->get('current_user') to return statement of public static function create()

... like ...

  public static function create(ContainerInterface $container, array $configuration, $plugin_id, $plugin_definition) {
    return new static(
      $configuration,
      $plugin_id,
      $plugin_definition,
      $container->get('entity_type.manager'),
      $container->get('plugin.manager.commerce_payment_type'),
      $container->get('plugin.manager.commerce_payment_method_type'),
      $container->get('datetime.time'),
      $container->get('logger.factory'),
      $container->get('http_client'),
      $container->get('commerce_price.rounder'),
      $container->get('commerce_paypal.ipn_handler'),
      $container->get('module_handler'),
      $container->get('event_dispatcher'),
      $container->get('current_user') // THIS LINE
    );
  }

OK So I can see what has happened here.

The constructor for the Express Checkout Subscriptions has one more argument than the parent class, yet it still tries to use the parent classes construct method.

This patch allowed me to create the payment method.